MolarVerse / PQAnalysis

PQAnalysis is a API/CLI python package for the analysis of MD simulations
https://molarverse.github.io/PQAnalysis/
MIT License
4 stars 2 forks source link

Bugfix/logger #36

Closed 97gamjak closed 2 months ago

97gamjak commented 2 months ago

Modified the logger class, such that the Exception type is also included in the standard print information and now one can also check the exception type given to the logger via pytest.

Furthermore, small refactorings of trajectory.py due to some pylint issues.

codecov[bot] commented 2 months ago

Codecov Report

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

Project coverage is 77.25%. Comparing base (cec6f78) to head (8694cd6).

Additional details and impacted files ```diff @@ Coverage Diff @@ ## dev #36 +/- ## ========================================== + Coverage 77.23% 77.25% +0.02% ========================================== Files 118 118 Lines 4190 4195 +5 ========================================== + Hits 3236 3241 +5 Misses 954 954 ``` | [Flag](https://app.codecov.io/gh/MolarVerse/PQAnalysis/pull/36/flags?src=pr&el=flags&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=MolarVerse) | Coverage Δ | | |---|---|---| | [unittests](https://app.codecov.io/gh/MolarVerse/PQAnalysis/pull/36/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=MolarVerse) | `77.25% <100.00%> (+0.02%)` | :arrow_up: | | [Files](https://app.codecov.io/gh/MolarVerse/PQAnalysis/pull/36?dropdown=coverage&src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=MolarVerse) | Coverage Δ | | |---|---|---| | [PQAnalysis/traj/trajectory.py](https://app.codecov.io/gh/MolarVerse/PQAnalysis/pull/36?src=pr&el=tree&filepath=PQAnalysis%2Ftraj%2Ftrajectory.py&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=MolarVerse#diff-UFFBbmFseXNpcy90cmFqL3RyYWplY3RvcnkucHk=) | `100.00% <100.00%> (ø)` | | | [PQAnalysis/utils/custom\_logging.py](https://app.codecov.io/gh/MolarVerse/PQAnalysis/pull/36?src=pr&el=tree&filepath=PQAnalysis%2Futils%2Fcustom_logging.py&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=MolarVerse#diff-UFFBbmFseXNpcy91dGlscy9jdXN0b21fbG9nZ2luZy5weQ==) | `72.44% <100.00%> (+1.48%)` | :arrow_up: |
github-actions[bot] commented 2 months ago

PYLINT REPORT

Your code has been rated at 9.87/10

Full report Raw metrics =========== |type |number |% |previous |difference | |----------|-------|------|---------|-----------| |code |6403 |38.60 |NC |NC | |docstring |7739 |46.66 |NC |NC | |comment |215 |1.30 |NC |NC | |empty |2230 |13.44 |NC |NC | Duplication =========== | |now |previous |difference | |-------------------------|------|---------|-----------| |nb duplicated lines |0 |NC |NC | |percent duplicated lines |0.000 |NC |NC | Messages by category ==================== |type |number |previous |difference | |-----------|-------|---------|-----------| |convention |10 |NC |NC | |refactor |36 |NC |NC | |warning |6 |NC |NC | |error |0 |NC |NC | Messages ======== |message id |occurrences | |-----------------------------|------------| |too-many-arguments |17 | |too-many-instance-attributes |10 | |invalid-name |9 | |fixme |6 | |too-many-locals |3 | |duplicate-code |2 | |too-many-statements |1 | |too-many-return-statements |1 | |too-many-branches |1 | |too-complex |1 | |missing-module-docstring |1 |