search

MDAnalysis / mdanalysis

MDAnalysis is a Python library to analyze molecular dynamics simulations.
https://mdanalysis.org
Other
1.3k stars 648 forks source link

[doc] fix RDF equation (and hole link) #4591

Closed orbeckst closed 4 months ago

orbeckst commented 4 months ago

Corrects error in the RDF equation as noted by @rodpollet https://github.com/MDAnalysis/mdanalysis/discussions/4437#discussioncomment-9021926

Changes made in this Pull Request:

PR Checklist

Developers certificate of origin

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

pep8speaks commented 4 months ago

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

Line 35:80: E501 line too long (82 > 79 characters)

github-actions[bot] commented 4 months ago

Linter Bot Results:

Hi @orbeckst! Thanks for making this PR. We linted your code and found the following:

There are currently no issues detected! 🎉

codecov[bot] commented 4 months ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 93.62%. Comparing base (45fb664) to head (61d05a9).

Additional details and impacted files ```diff @@ Coverage Diff @@ ## develop #4591 +/- ## =========================================== - Coverage 93.64% 93.62% -0.02% =========================================== Files 168 180 +12 Lines 21254 22333 +1079 Branches 3918 3918 =========================================== + Hits 19904 20910 +1006 - Misses 892 965 +73 Partials 458 458 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.