catalystneuro / dandi_s3_log_parser

S3 log parsing for the DANDI Archive.
BSD 3-Clause "New" or "Revised" License
1 stars 2 forks source link

Naming adjustments and CLI exposure #41

Closed CodyCBakerPhD closed 2 months ago

CodyCBakerPhD commented 2 months ago

Parser is the name of the package; reduction is the main operation currently being performed

Added CLIs for non-batch functions

codecov[bot] commented 2 months ago

Codecov Report

Attention: Patch coverage is 41.02564% with 23 lines in your changes missing coverage. Please review.

Project coverage is 65.21%. Comparing base (a6ba38d) to head (93759f4). Report is 7 commits behind head on main.

Files Patch % Lines
src/dandi_s3_log_parser/_command_line_interface.py 0.00% 22 Missing :warning:
...c/dandi_s3_log_parser/_dandi_s3_log_file_parser.py 91.66% 1 Missing :warning:
Additional details and impacted files [![Impacted file tree graph](https://app.codecov.io/gh/catalystneuro/dandi_s3_log_parser/pull/41/graphs/tree.svg?width=650&height=150&src=pr&token=RIT59WJokn&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=catalystneuro)](https://app.codecov.io/gh/catalystneuro/dandi_s3_log_parser/pull/41?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=catalystneuro) ```diff @@ Coverage Diff @@ ## main #41 +/- ## ========================================== - Coverage 68.08% 65.21% -2.87% ========================================== Files 11 12 +1 Lines 423 483 +60 ========================================== + Hits 288 315 +27 - Misses 135 168 +33 ``` | [Flag](https://app.codecov.io/gh/catalystneuro/dandi_s3_log_parser/pull/41/flags?src=pr&el=flags&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=catalystneuro) | Coverage Δ | | |---|---|---| | [unittests](https://app.codecov.io/gh/catalystneuro/dandi_s3_log_parser/pull/41/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=catalystneuro) | `65.21% <41.02%> (-2.87%)` | :arrow_down: | Flags with carried forward coverage won't be shown. [Click here](https://docs.codecov.io/docs/carryforward-flags?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=catalystneuro#carryforward-flags-in-the-pull-request-comment) to find out more. | [Files](https://app.codecov.io/gh/catalystneuro/dandi_s3_log_parser/pull/41?dropdown=coverage&src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=catalystneuro) | Coverage Δ | | |---|---|---| | [src/dandi\_s3\_log\_parser/\_\_init\_\_.py](https://app.codecov.io/gh/catalystneuro/dandi_s3_log_parser/pull/41?src=pr&el=tree&filepath=src%2Fdandi_s3_log_parser%2F__init__.py&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=catalystneuro#diff-c3JjL2RhbmRpX3MzX2xvZ19wYXJzZXIvX19pbml0X18ucHk=) | `100.00% <100.00%> (ø)` | | | [src/dandi\_s3\_log\_parser/\_dandiset\_mapper.py](https://app.codecov.io/gh/catalystneuro/dandi_s3_log_parser/pull/41?src=pr&el=tree&filepath=src%2Fdandi_s3_log_parser%2F_dandiset_mapper.py&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=catalystneuro#diff-c3JjL2RhbmRpX3MzX2xvZ19wYXJzZXIvX2RhbmRpc2V0X21hcHBlci5weQ==) | `95.55% <100.00%> (ø)` | | | [...c/dandi\_s3\_log\_parser/\_dandi\_s3\_log\_file\_parser.py](https://app.codecov.io/gh/catalystneuro/dandi_s3_log_parser/pull/41?src=pr&el=tree&filepath=src%2Fdandi_s3_log_parser%2F_dandi_s3_log_file_parser.py&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=catalystneuro#diff-c3JjL2RhbmRpX3MzX2xvZ19wYXJzZXIvX2RhbmRpX3MzX2xvZ19maWxlX3BhcnNlci5weQ==) | `80.68% <91.66%> (-0.43%)` | :arrow_down: | | [src/dandi\_s3\_log\_parser/\_command\_line\_interface.py](https://app.codecov.io/gh/catalystneuro/dandi_s3_log_parser/pull/41?src=pr&el=tree&filepath=src%2Fdandi_s3_log_parser%2F_command_line_interface.py&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=catalystneuro#diff-c3JjL2RhbmRpX3MzX2xvZ19wYXJzZXIvX2NvbW1hbmRfbGluZV9pbnRlcmZhY2UucHk=) | `0.00% <0.00%> (ø)` | | ... and [1 file with indirect coverage changes](https://app.codecov.io/gh/catalystneuro/dandi_s3_log_parser/pull/41/indirect-changes?src=pr&el=tree-more&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=catalystneuro)