-
Notifications
You must be signed in to change notification settings - Fork 39
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
Restrict NumPy to pre-2.0 #344
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #344 +/- ##
=======================================
Coverage 89.77% 89.77%
=======================================
Files 12 12
Lines 929 929
=======================================
Hits 834 834
Misses 95 95 ☔ View full report in Codecov by Sentry. |
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 tested the hot fix on my local machine by installing the fixed Casanovo package into a fresh conda environment (using pip install .
). All the tests pass and the CLI seems to work as expected.
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.
Tested on my local machine - CLI functionality works as expected and all tests pass.
* Restrict NumPy to pre-2.0 * Update changelog
* prediction output in model eval mode * eliminate eval command, introduce -e flag for predict command * adapted unit test to new model runner and model functionality * updated documentation * removed log and result files * Generate new screengrabs with rich-codex * Update paper reference (#361) * Bug report template (#360) * bug report template * punctuation, hardware description item * Restrict NumPy to pre-2.0 (#344) * Restrict NumPy to pre-2.0 * Update changelog * Update paper reference (#361) --------- Co-authored-by: Lilferrit <straub.gavin@gmail.com> * upgrade codecove to v4 (#364) * implemen eval mode at model runner level, fix unit test * CLI documentation * Generate new screengrabs with rich-codex * requested changes * Generate new screengrabs with rich-codex * evaluation test cases * file warnings, evaluation tests * fixed ubuntu specific test case bug * verify annotated mgf files * verify annotated mgf files * Generate new screengrabs with rich-codex * Save best model (#365) * save best model * save best model * updated unit tests * remove save top k config item * added save_top_k to deprecated config options * changelog entry * test case, formatting * requested changes * prediction output in model eval mode * eliminate eval command, introduce -e flag for predict command * adapted unit test to new model runner and model functionality * updated documentation * removed log and result files * implemen eval mode at model runner level, fix unit test * CLI documentation * Bug report template (#360) * bug report template * punctuation, hardware description item * Restrict NumPy to pre-2.0 (#344) * Restrict NumPy to pre-2.0 * Update changelog * Update paper reference (#361) --------- Co-authored-by: Lilferrit <straub.gavin@gmail.com> * requested changes * evaluation test cases * file warnings, evaluation tests * fixed ubuntu specific test case bug * verify annotated mgf files * AnnotatedSpectrumIndex type error * requested changes, changelog entry --------- Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com> Co-authored-by: Wout Bittremieux <bittremieux@users.noreply.github.com>
Fixes #339.
Hotfix until NumPy v2.0 will be supported after the DepthCharge upgrade.