dandi / dandisets

730 Dandisets, 807.1 TB total. DataLad super-dataset of all Dandisets from https://github.com/dandisets
10 stars 0 forks source link

Capture output from all subcommands and log on failure #348

Closed jwodder closed 1 year ago

jwodder commented 1 year ago

Part of #347.

Note that this won't affect commands run by DataLad; would changing this setting help?

codecov[bot] commented 1 year ago

Codecov Report

Patch coverage: 53.33% and project coverage change: -0.05 :warning:

Comparison is base (85af577) 76.36% compared to head (2b15d1a) 76.31%.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## draft #348 +/- ## ========================================== - Coverage 76.36% 76.31% -0.05% ========================================== Files 14 14 Lines 2458 2470 +12 Branches 541 544 +3 ========================================== + Hits 1877 1885 +8 - Misses 431 435 +4 Partials 150 150 ``` | [Impacted Files](https://codecov.io/gh/dandi/dandisets/pull/348?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=dandi) | Coverage Δ | | |---|---|---| | [tools/backups2datalad/aioutil.py](https://codecov.io/gh/dandi/dandisets/pull/348?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=dandi#diff-dG9vbHMvYmFja3VwczJkYXRhbGFkL2Fpb3V0aWwucHk=) | `72.85% <50.00%> (-2.02%)` | :arrow_down: | | [tools/backups2datalad/adataset.py](https://codecov.io/gh/dandi/dandisets/pull/348?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=dandi#diff-dG9vbHMvYmFja3VwczJkYXRhbGFkL2FkYXRhc2V0LnB5) | `79.72% <100.00%> (+0.06%)` | :arrow_up: | ... and [2 files with indirect coverage changes](https://codecov.io/gh/dandi/dandisets/pull/348/indirect-changes?src=pr&el=tree-more&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=dandi) Help us with your feedback. Take ten seconds to tell us [how you rate us](https://about.codecov.io/nps?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=dandi). Have a feature suggestion? [Share it here.](https://app.codecov.io/gh/feedback/?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=dandi)

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.

yarikoptic commented 1 year ago

Part of #347.

Note that this won't affect commands run by DataLad; would changing this setting help?

I think we should make it into smth like

export ${DATALAD_LOG_LEVEL:=WARNING}

so we overload only if not defined -- this way we can have a run which would define it to DEBUG

yarikoptic commented 1 year ago

thank you! I will again disable 000026 and will run it "manually" eventually. ATM I think it has no updates

yarikoptic commented 1 year ago

or actually I probably can just clone/hard reset back in time somewhere and rerun script in that location... will do whenever I get a chance