taxprofiler / taxpasta

TAXnomic Profile Aggregation and STAndardisation
https://taxpasta.readthedocs.io/
Apache License 2.0
31 stars 6 forks source link

Allow ignoring profile errors when merging, message on empty profile #136

Closed Midnighter closed 10 months ago

Midnighter commented 10 months ago

Too late to add further testing right now.

codecov-commenter commented 10 months ago

Codecov Report

Patch coverage: 45.18% and project coverage change: +2.32% :tada:

Comparison is base (98713de) 85.87% compared to head (3715353) 88.19%.

:exclamation: Current head 3715353 differs from pull request most recent head 9cde5a0. Consider uploading reports for the commit 9cde5a0 to get more accurate results

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## dev #136 +/- ## ========================================== + Coverage 85.87% 88.19% +2.32% ========================================== Files 114 114 Lines 1791 1762 -29 Branches 325 303 -22 ========================================== + Hits 1538 1554 +16 + Misses 205 176 -29 + Partials 48 32 -16 ``` | [Files Changed](https://app.codecov.io/gh/taxprofiler/taxpasta/pull/136?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=None) | Coverage Δ | | |---|---|---| | [...rc/taxpasta/infrastructure/application/\_\_init\_\_.py](https://app.codecov.io/gh/taxprofiler/taxpasta/pull/136?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=None#diff-c3JjL3RheHBhc3RhL2luZnJhc3RydWN0dXJlL2FwcGxpY2F0aW9uL19faW5pdF9fLnB5) | `100.00% <ø> (ø)` | | | [src/taxpasta/infrastructure/cli/merge.py](https://app.codecov.io/gh/taxprofiler/taxpasta/pull/136?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=None#diff-c3JjL3RheHBhc3RhL2luZnJhc3RydWN0dXJlL2NsaS9tZXJnZS5weQ==) | `61.34% <28.57%> (+8.74%)` | :arrow_up: | | [src/taxpasta/infrastructure/cli/standardise.py](https://app.codecov.io/gh/taxprofiler/taxpasta/pull/136?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=None#diff-c3JjL3RheHBhc3RhL2luZnJhc3RydWN0dXJlL2NsaS9zdGFuZGFyZGlzZS5weQ==) | `47.88% <35.29%> (+15.59%)` | :arrow_up: | | [src/taxpasta/application/add\_tax\_info\_command.py](https://app.codecov.io/gh/taxprofiler/taxpasta/pull/136?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=None#diff-c3JjL3RheHBhc3RhL2FwcGxpY2F0aW9uL2FkZF90YXhfaW5mb19jb21tYW5kLnB5) | `41.86% <41.86%> (ø)` | | | [...axpasta/application/sample\_handling\_application.py](https://app.codecov.io/gh/taxprofiler/taxpasta/pull/136?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=None#diff-c3JjL3RheHBhc3RhL2FwcGxpY2F0aW9uL3NhbXBsZV9oYW5kbGluZ19hcHBsaWNhdGlvbi5weQ==) | `76.31% <57.89%> (ø)` | | | [src/taxpasta/application/\_\_init\_\_.py](https://app.codecov.io/gh/taxprofiler/taxpasta/pull/136?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=None#diff-c3JjL3RheHBhc3RhL2FwcGxpY2F0aW9uL19faW5pdF9fLnB5) | `100.00% <100.00%> (ø)` | | | [...rc/taxpasta/domain/model/tidy\_observation\_table.py](https://app.codecov.io/gh/taxprofiler/taxpasta/pull/136?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=None#diff-c3JjL3RheHBhc3RhL2RvbWFpbi9tb2RlbC90aWR5X29ic2VydmF0aW9uX3RhYmxlLnB5) | `100.00% <100.00%> (ø)` | | | [...rc/taxpasta/domain/model/wide\_observation\_table.py](https://app.codecov.io/gh/taxprofiler/taxpasta/pull/136?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=None#diff-c3JjL3RheHBhc3RhL2RvbWFpbi9tb2RlbC93aWRlX29ic2VydmF0aW9uX3RhYmxlLnB5) | `100.00% <100.00%> (ø)` | |

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