search

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

Hotfix RAM usage #70

Closed CodyCBakerPhD closed 2 months ago

CodyCBakerPhD commented 2 months ago

Only a problem during 000108

codecov[bot] commented 2 months ago

Codecov Report

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

Project coverage is 71.38%. Comparing base (b59ae1d) to head (7240e3e). Report is 1 commits behind head on main.

Additional details and impacted files [![Impacted file tree graph](https://app.codecov.io/gh/catalystneuro/dandi_s3_log_parser/pull/70/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/70?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=catalystneuro) ```diff @@ Coverage Diff @@ ## main #70 +/- ## ========================================== + Coverage 71.12% 71.38% +0.26% ========================================== Files 14 14 Lines 658 664 +6 ========================================== + Hits 468 474 +6 Misses 190 190 ``` | [Flag](https://app.codecov.io/gh/catalystneuro/dandi_s3_log_parser/pull/70/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/70/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=catalystneuro) | `71.38% <100.00%> (+0.26%)` | :arrow_up: | 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 with missing lines](https://app.codecov.io/gh/catalystneuro/dandi_s3_log_parser/pull/70?dropdown=coverage&src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=catalystneuro) | Coverage Δ | | |---|---|---| | [...\_s3\_log\_parser/\_map\_binned\_s3\_logs\_to\_dandisets.py](https://app.codecov.io/gh/catalystneuro/dandi_s3_log_parser/pull/70?src=pr&el=tree&filepath=src%2Fdandi_s3_log_parser%2F_map_binned_s3_logs_to_dandisets.py&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=catalystneuro#diff-c3JjL2RhbmRpX3MzX2xvZ19wYXJzZXIvX21hcF9iaW5uZWRfczNfbG9nc190b19kYW5kaXNldHMucHk=) | `97.67% <100.00%> (+0.11%)` | :arrow_up: |