-
-
Notifications
You must be signed in to change notification settings - Fork 422
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Temporarily fetch refdata from GDrive #1267
Conversation
Codecov Report
@@ Coverage Diff @@
## master #1267 +/- ##
=======================================
Coverage 80.63% 80.63%
=======================================
Files 41 41
Lines 3424 3424
=======================================
Hits 2761 2761
Misses 663 663 Continue to review full report at Codecov.
|
displayName: "Fetch reference data" | ||
condition: false | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I just removed the plasma_reference
folder (does not exist in tardis-sn/tardis-refdata repo) and disabled the step with the false
condition. The rest of the lines were shifted by the YAML autoformatter.
pytest tardis --tardis-refdata=$(ref.data.home) --cov=tardis --cov-report=xml --cov-report=html | ||
source activate tardis | ||
pip install pytest-azurepipelines | ||
pytest tardis --tardis-refdata=$(ref.data.home) --cov=tardis --cov-report=xml --cov-report=html | ||
displayName: "TARDIS test" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
No changes here, just the YAML autoformatter.
displayName: "TARDIS test" | ||
|
||
- bash: | | ||
bash <(curl -s https://codecov.io/bash) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
YAML autoformatter.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
675665e
to
dcc3336
Compare
Co-authored-by: Wolfgang Kerzendorf <wkerzendorf@gmail.com>
* add GIT_LFS_SMUDGE * fixed smudge command * Minor changes * Fetch refdata from GDrive backup * Move fetch refdata steps to the top * Fix typo * Debugging pipeline * Move fetch refdata steps again. Install gdown within TARDIS env * [skip-ci] Minor changes * Update azure-pipelines/simple_test_framework.yml Co-authored-by: Wolfgang Kerzendorf <wkerzendorf@gmail.com> Co-authored-by: Kaushik Varanasi <kaushik.varanasi1@gmail.com> Co-authored-by: Wolfgang Kerzendorf <wkerzendorf@gmail.com>
Description
Retrieve TARDIS reference data from a backup stored in Google Drive.
Motivation and Context
We exceeded our LFS quota by mistake :(
How Has This Been Tested?
Testing pipeline is passing.
Screenshots (if appropriate):
Types of changes
Checklist: