OpenFreeEnergy / gufe

grand unified free energy by OpenFE
https://gufe.readthedocs.io
MIT License
28 stars 8 forks source link

Clarify "DAG dependency order" in docstrings #244

Closed Yoshanuikabundi closed 11 months ago

Yoshanuikabundi commented 12 months ago

Fixes #223

pep8speaks commented 12 months ago

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

Line 66:80: E501 line too long (80 > 79 characters) Line 163:80: E501 line too long (80 > 79 characters)

codecov[bot] commented 12 months ago

Codecov Report

All modified lines are covered by tests :white_check_mark:

Comparison is base (dcc7896) 99.18% compared to head (ad6f345) 99.18%.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #244 +/- ## ======================================= Coverage 99.18% 99.18% ======================================= Files 36 36 Lines 1838 1838 ======================================= Hits 1823 1823 Misses 15 15 ``` | [Files](https://app.codecov.io/gh/OpenFreeEnergy/gufe/pull/244?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=OpenFreeEnergy) | Coverage Δ | | |---|---|---| | [gufe/protocols/protocoldag.py](https://app.codecov.io/gh/OpenFreeEnergy/gufe/pull/244?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=OpenFreeEnergy#diff-Z3VmZS9wcm90b2NvbHMvcHJvdG9jb2xkYWcucHk=) | `100.00% <ø> (ø)` | |

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