DOC: (Follow-up to PR #5325) Add missing docstrings and update CHANGELOG#5331
DOC: (Follow-up to PR #5325) Add missing docstrings and update CHANGELOG#5331ParthUppal523 wants to merge 2 commits intoMDAnalysis:developfrom
Conversation
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## develop #5331 +/- ##
========================================
Coverage 93.83% 93.83%
========================================
Files 182 182
Lines 22492 22492
Branches 3199 3199
========================================
Hits 21106 21106
Misses 924 924
Partials 462 462 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
|
@BradyAJohnston , request you to review these changes. This is a follow-up PR to PR #5325 as I realized the docstring changes made in |
|
The linter is currently failing - you'll need to lint with If you have uvx black~=24.0 package/ |
|
Sure @BradyAJohnston , I have reformatted using Also based on our discussion in PR #5325, I have provided a fix for the |
Fixes #5317
This PR is a follow-up to my previous merged PR #5325. I realized that the changes made to
select_atoms()docstrings ingroups.pyandpackage/CHANGELOGwere accidentally omitted.Changes made in this Pull Request:
allkeyword inselect_atoms()package/CHANGELOGLLM / AI generated code disclosure
LLMs or other AI-powered tools (beyond simple IDE use cases) were used in this contribution: no
PR Checklist
package/CHANGELOGfile updated?package/AUTHORS? (If it is not, add it!)Developers Certificate of Origin
I certify that I can submit this code contribution as described in the Developer Certificate of Origin, under the MDAnalysis LICENSE.
📚 Documentation preview 📚: https://mdanalysis--5331.org.readthedocs.build/en/5331/