OpenFreeEnergy / gufe

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

Adding NetworkPlanner #307

Closed RiesBen closed 3 weeks ago

RiesBen commented 5 months ago

The goal here is to add a generic NetworkPlanner. to be discussed.

pep8speaks commented 5 months ago

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

Line 21:80: E501 line too long (90 > 79 characters)

Line 8:1: E302 expected 2 blank lines, found 1 Line 18:50: E231 missing whitespace after ':' Line 18:80: E501 line too long (85 > 79 characters) Line 19:80: E501 line too long (97 > 79 characters) Line 29:22: E222 multiple spaces after operator Line 32:5: E303 too many blank lines (2) Line 32:66: E225 missing whitespace around operator Line 32:80: E501 line too long (82 > 79 characters) Line 36:80: E501 line too long (96 > 79 characters) Line 36:81: E225 missing whitespace around operator

Line 28:80: E501 line too long (111 > 79 characters) Line 40:80: E501 line too long (89 > 79 characters)

Line 9:80: E501 line too long (94 > 79 characters) Line 14:80: E501 line too long (87 > 79 characters)

Comment last updated at 2024-08-27 15:36:10 UTC
codecov[bot] commented 5 months ago

Codecov Report

Attention: Patch coverage is 0% with 8 lines in your changes missing coverage. Please review.

Project coverage is 98.55%. Comparing base (d5fe979) to head (a002cfb). Report is 2 commits behind head on main.

Files Patch % Lines
gufe/mapping/atom_mapping_scorer.py 0.00% 8 Missing :warning:
Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #307 +/- ## ========================================== - Coverage 98.94% 98.55% -0.40% ========================================== Files 36 37 +1 Lines 1996 2004 +8 ========================================== Hits 1975 1975 - Misses 21 29 +8 ```

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

RiesBen commented 3 weeks ago

was moved to Fe setup structure #346