mdanalysis
mdanalysis copied to clipboard
Enable parallelization for analysis.polymer
Fixes #4671
Changes made in this Pull Request:
- Enabled parallelization for PersistenceLength class
- added tests (including fixtures in analysis/conftest.py)
- update CHANGELOG
PR Checklist
- [ ] Tests?
- [x] Docs?
- [x] CHANGELOG updated?
- [x] Issue raised/referenced?
Developers certificate of origin
- [x] I certify that this contribution is covered by the LGPLv2.1+ license as defined in our LICENSE and adheres to the Developer Certificate of Origin.
📚 Documentation preview 📚: https://mdanalysis--4871.org.readthedocs.build/en/4871/
Hello @tanishy7777! Thanks for updating this PR. We checked the lines you've touched for PEP 8 issues, and found:
- In the file
package/MDAnalysis/analysis/polymer.py:
Line 322:80: E501 line too long (89 > 79 characters)
- In the file
testsuite/MDAnalysisTests/analysis/conftest.py:
Line 183:1: E302 expected 2 blank lines, found 1 Line 185:25: W292 no newline at end of file
Line 175:1: E302 expected 2 blank lines, found 1 Line 195:1: W391 blank line at end of file
Comment last updated at 2025-01-18 17:29:26 UTC
Superseded by PR #5074.
Thank you for your work here, @tanishy7777 !