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

Feature/beartype exceptions #41

Closed 97gamjak closed 2 months ago

97gamjak commented 2 months ago

RDF analysis module fully type checked for all user relevant functions

Now also logger is implemented in the whole module - no more exceptions or warns.warnings

codecov[bot] commented 2 months ago

Codecov Report

Attention: Patch coverage is 84.84848% with 15 lines in your changes are missing coverage. Please review.

Project coverage is 77.52%. Comparing base (3585a95) to head (199734c). Report is 2 commits behind head on dev.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## dev #41 +/- ## ========================================== + Coverage 75.73% 77.52% +1.79% ========================================== Files 119 119 Lines 4237 4272 +35 ========================================== + Hits 3209 3312 +103 + Misses 1028 960 -68 ``` | [Flag](https://app.codecov.io/gh/MolarVerse/PQAnalysis/pull/41/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/41/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=MolarVerse) | `77.52% <84.84%> (+1.79%)` | :arrow_up: | | [Files](https://app.codecov.io/gh/MolarVerse/PQAnalysis/pull/41?dropdown=coverage&src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=MolarVerse) | Coverage Δ | | |---|---|---| | [PQAnalysis/analysis/rdf/rdf\_input\_file\_reader.py](https://app.codecov.io/gh/MolarVerse/PQAnalysis/pull/41?src=pr&el=tree&filepath=PQAnalysis%2Fanalysis%2Frdf%2Frdf_input_file_reader.py&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=MolarVerse#diff-UFFBbmFseXNpcy9hbmFseXNpcy9yZGYvcmRmX2lucHV0X2ZpbGVfcmVhZGVyLnB5) | `80.00% <100.00%> (+0.83%)` | :arrow_up: | | [PQAnalysis/analysis/rdf/rdf\_output\_file\_writer.py](https://app.codecov.io/gh/MolarVerse/PQAnalysis/pull/41?src=pr&el=tree&filepath=PQAnalysis%2Fanalysis%2Frdf%2Frdf_output_file_writer.py&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=MolarVerse#diff-UFFBbmFseXNpcy9hbmFseXNpcy9yZGYvcmRmX291dHB1dF9maWxlX3dyaXRlci5weQ==) | `35.71% <100.00%> (+11.71%)` | :arrow_up: | | [PQAnalysis/topology/selection.py](https://app.codecov.io/gh/MolarVerse/PQAnalysis/pull/41?src=pr&el=tree&filepath=PQAnalysis%2Ftopology%2Fselection.py&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=MolarVerse#diff-UFFBbmFseXNpcy90b3BvbG9neS9zZWxlY3Rpb24ucHk=) | `98.58% <100.00%> (ø)` | | | [PQAnalysis/traj/trajectory.py](https://app.codecov.io/gh/MolarVerse/PQAnalysis/pull/41?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/type\_checking.py](https://app.codecov.io/gh/MolarVerse/PQAnalysis/pull/41?src=pr&el=tree&filepath=PQAnalysis%2Ftype_checking.py&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=MolarVerse#diff-UFFBbmFseXNpcy90eXBlX2NoZWNraW5nLnB5) | `75.00% <100.00%> (+7.43%)` | :arrow_up: | | [PQAnalysis/utils/units.py](https://app.codecov.io/gh/MolarVerse/PQAnalysis/pull/41?src=pr&el=tree&filepath=PQAnalysis%2Futils%2Funits.py&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=MolarVerse#diff-UFFBbmFseXNpcy91dGlscy91bml0cy5weQ==) | `100.00% <100.00%> (ø)` | | | [PQAnalysis/utils/custom\_logging.py](https://app.codecov.io/gh/MolarVerse/PQAnalysis/pull/41?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==) | `75.49% <75.00%> (-1.05%)` | :arrow_down: | | [PQAnalysis/analysis/rdf/rdf.py](https://app.codecov.io/gh/MolarVerse/PQAnalysis/pull/41?src=pr&el=tree&filepath=PQAnalysis%2Fanalysis%2Frdf%2Frdf.py&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=MolarVerse#diff-UFFBbmFseXNpcy9hbmFseXNpcy9yZGYvcmRmLnB5) | `77.24% <80.88%> (+46.57%)` | :arrow_up: |
github-actions[bot] commented 2 months ago

PYLINT REPORT

Your code has been rated at 9.89/10 (previous run: 9.87/10, +0.02)

Full report Raw metrics =========== |type |number |% |previous |difference | |----------|-------|------|---------|-----------| |code |6527 |38.84 |6473 |+54.00 | |docstring |7783 |46.32 |7755 |+28.00 | |comment |221 |1.32 |226 |-5.00 | |empty |2273 |13.53 |2253 |+20.00 | Duplication =========== | |now |previous |difference | |-------------------------|------|---------|-----------| |nb duplicated lines |0 |0 |0 | |percent duplicated lines |0.000 |0.000 |= | Messages by category ==================== |type |number |previous |difference | |-----------|-------|---------|-----------| |convention |0 |10 |10 | |refactor |37 |36 |36 | |warning |7 |8 |8 | |error |0 |0 |0 | Messages ======== |message id |occurrences | |-----------------------------|------------| |too-many-arguments |18 | |too-many-instance-attributes |10 | |fixme |7 | |too-many-locals |3 | |duplicate-code |2 | |too-many-statements |1 | |too-many-return-statements |1 | |too-many-branches |1 | |too-complex |1 |