OpenFreeEnergy / kartograf

This package contains tools for setting up hybrid-topology FE calculations
https://kartograf.readthedocs.io/
MIT License
25 stars 2 forks source link

Refactoring & Docs #10

Closed RiesBen closed 1 year ago

codecov[bot] commented 1 year ago

Codecov Report

Patch coverage: 98.41% and project coverage change: -0.18 :warning:

Comparison is base (7c25e6b) 92.30% compared to head (555498c) 92.13%.

:exclamation: Current head 555498c differs from pull request most recent head 3a51c0e. Consider uploading reports for the commit 3a51c0e to get more accurate results

Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #10 +/- ## ========================================== - Coverage 92.30% 92.13% -0.18% ========================================== Files 7 4 -3 Lines 481 445 -36 ========================================== - Hits 444 410 -34 + Misses 37 35 -2 ``` | [Impacted Files](https://app.codecov.io/gh/OpenFreeEnergy/kartograf/pull/10?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=OpenFreeEnergy) | Coverage Δ | | |---|---|---| | [src/kartograf/atom\_mapper.py](https://app.codecov.io/gh/OpenFreeEnergy/kartograf/pull/10?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=OpenFreeEnergy#diff-c3JjL2thcnRvZ3JhZi9hdG9tX21hcHBlci5weQ==) | `89.35% <97.22%> (+2.22%)` | :arrow_up: | | [src/kartograf/atom\_mapping\_scoring.py](https://app.codecov.io/gh/OpenFreeEnergy/kartograf/pull/10?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=OpenFreeEnergy#diff-c3JjL2thcnRvZ3JhZi9hdG9tX21hcHBpbmdfc2NvcmluZy5weQ==) | `95.20% <100.00%> (ø)` | | ... and [4 files with indirect coverage changes](https://app.codecov.io/gh/OpenFreeEnergy/kartograf/pull/10/indirect-changes?src=pr&el=tree-more&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=OpenFreeEnergy)

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.

RiesBen commented 1 year ago

things here are super minor, so i will just merge it, after the tests passed.