search

dandi / dandisets

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

Redo Zarr syncing up to 5 times if there was a checksum mystery mismatch #330

Closed jwodder closed 1 year ago

jwodder commented 1 year ago

Closes #305.

codecov[bot] commented 1 year ago

Codecov Report

Base: 75.86% // Head: 75.62% // Decreases project coverage by -0.24% :warning:

Coverage data is based on head (eb94a6f) compared to base (d284713). Patch coverage: 59.45% of modified lines in pull request are covered.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## draft #330 +/- ## ========================================== - Coverage 75.86% 75.62% -0.24% ========================================== Files 14 14 Lines 2424 2433 +9 Branches 382 383 +1 ========================================== + Hits 1839 1840 +1 - Misses 430 439 +9 + Partials 155 154 -1 ``` | [Impacted Files](https://codecov.io/gh/dandi/dandisets/pull/330?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=dandi) | Coverage Δ | | |---|---|---| | [tools/backups2datalad/zarr.py](https://codecov.io/gh/dandi/dandisets/pull/330?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=dandi#diff-dG9vbHMvYmFja3VwczJkYXRhbGFkL3phcnIucHk=) | `72.37% <58.33%> (-1.15%)` | :arrow_down: | | [tools/backups2datalad/adataset.py](https://codecov.io/gh/dandi/dandisets/pull/330?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=dandi#diff-dG9vbHMvYmFja3VwczJkYXRhbGFkL2FkYXRhc2V0LnB5) | `79.31% <100.00%> (+0.07%)` | :arrow_up: | | [tools/backups2datalad/consts.py](https://codecov.io/gh/dandi/dandisets/pull/330?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=dandi#diff-dG9vbHMvYmFja3VwczJkYXRhbGFkL2NvbnN0cy5weQ==) | `100.00% <100.00%> (ø)` | | | [tools/backups2datalad/\_\_main\_\_.py](https://codecov.io/gh/dandi/dandisets/pull/330?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=dandi#diff-dG9vbHMvYmFja3VwczJkYXRhbGFkL19fbWFpbl9fLnB5) | `63.44% <0.00%> (-0.43%)` | :arrow_down: | | [tools/backups2datalad/asyncer.py](https://codecov.io/gh/dandi/dandisets/pull/330?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=dandi#diff-dG9vbHMvYmFja3VwczJkYXRhbGFkL2FzeW5jZXIucHk=) | `78.72% <0.00%> (-0.28%)` | :arrow_down: | 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 at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.

yarikoptic commented 1 year ago

diff is too large to be able to see what has happened on github, looked locally at git log -p -w origin/gh-305 to ignore indentation - changes looked good. Thank you @jwodder, let's proceed with this in place!