add --download-only option to openfe test#1814
Conversation
Codecov Report❌ Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #1814 +/- ##
==========================================
- Coverage 95.27% 92.98% -2.30%
==========================================
Files 201 204 +3
Lines 17580 17623 +43
==========================================
- Hits 16750 16386 -364
- Misses 830 1237 +407
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
4f569df to
86316d4
Compare
86316d4 to
28f2869
Compare
| return universe | ||
|
|
||
|
|
||
| RFE_OUTPUT = pooch.create( |
There was a problem hiding this comment.
note: we moved to from figshare to zenodo here:
openfe/src/openfe/tests/protocols/test_openmmutils.py
Lines 1037 to 1038 in f7561cd
but this was left as a stragger no-op.
mikemhenry
left a comment
There was a problem hiding this comment.
LGTM, a hash of the _registry.py files can now be used as the cache key, this will also make it easy to migrate things somewhere else since we only have one file to track.
|
🚨 API breaking changes detected! 🚨 |
this will allow us to:
Checklist
newsentry, or the changes are not user-facing.pre-commit.ci autofix.Manual Tests: these are slow so don't need to be run every commit, only before merging and when relevant changes are made (generally at reviewer-discretion).
Developers certificate of origin