-
Notifications
You must be signed in to change notification settings - Fork 646
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
Deprecate fetch mmtf #4639
Deprecate fetch mmtf #4639
Conversation
Hello @IAlibay! Thanks for updating this PR. We checked the lines you've touched for PEP 8 issues, and found:
Comment last updated at 2024-07-24 14:17:41 UTC |
Linter Bot Results:Hi @IAlibay! Thanks for making this PR. We linted your code and found the following: Some issues were found with the formatting of your code.
Please have a look at the Please note: The |
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## develop #4639 +/- ##
===========================================
- Coverage 93.61% 93.59% -0.03%
===========================================
Files 171 183 +12
Lines 21250 22320 +1070
Branches 3936 3937 +1
===========================================
+ Hits 19893 20890 +997
- Misses 898 971 +73
Partials 459 459 ☔ View full report in Codecov by Sentry. |
RDKit failures being fixed in #4640 |
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.
Content LGTM @IAlibay
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
Fixes #4634
Changes made in this Pull Request:
fetch_mmtf
methodPR Checklist
Developers certificate of origin
📚 Documentation preview 📚: https://mdanalysis--4639.org.readthedocs.build/en/4639/