mdanalysis icon indicating copy to clipboard operation
mdanalysis copied to clipboard

Enable parallelization for analysis.polymer

Open tanishy7777 opened this issue 11 months ago • 1 comments

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


📚 Documentation preview 📚: https://mdanalysis--4871.org.readthedocs.build/en/4871/

tanishy7777 avatar Dec 28 '24 10:12 tanishy7777

Hello @tanishy7777! Thanks for updating this PR. We checked the lines you've touched for PEP 8 issues, and found:

Line 322:80: E501 line too long (89 > 79 characters)

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

pep8speaks avatar Dec 28 '24 10:12 pep8speaks

Superseded by PR #5074.

Thank you for your work here, @tanishy7777 !

orbeckst avatar Jul 31 '25 17:07 orbeckst