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

Update Zarr download URLs for dandi-archive#1394 #314

Closed jwodder closed 1 year ago

jwodder commented 1 year ago

https://github.com/dandi/dandi-archive/pull/1394 will change the API URLs used to download individual Zarr entries, so the code for registering URLs for entries in git-annex needs to be updated.

Blocked by https://github.com/dandi/dandi-archive/pull/1394.

codecov[bot] commented 1 year ago

Codecov Report

Base: 75.88% // Head: 75.80% // Decreases project coverage by -0.07% :warning:

Coverage data is based on head (449c6b7) compared to base (ce7946c). Patch coverage: 78.37% of modified lines in pull request are covered.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## draft #314 +/- ## ========================================== - Coverage 75.88% 75.80% -0.08% ========================================== Files 14 14 Lines 2368 2385 +17 Branches 377 378 +1 ========================================== + Hits 1797 1808 +11 - Misses 425 427 +2 - Partials 146 150 +4 ``` | [Impacted Files](https://codecov.io/gh/dandi/dandisets/pull/314?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=dandi) | Coverage Δ | | |---|---|---| | [tools/backups2datalad/syncer.py](https://codecov.io/gh/dandi/dandisets/pull/314/diff?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=dandi#diff-dG9vbHMvYmFja3VwczJkYXRhbGFkL3N5bmNlci5weQ==) | `78.78% <ø> (ø)` | | | [tools/backups2datalad/zarr.py](https://codecov.io/gh/dandi/dandisets/pull/314/diff?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=dandi#diff-dG9vbHMvYmFja3VwczJkYXRhbGFkL3phcnIucHk=) | `73.93% <50.00%> (-0.43%)` | :arrow_down: | | [tools/backups2datalad/adataset.py](https://codecov.io/gh/dandi/dandisets/pull/314/diff?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=dandi#diff-dG9vbHMvYmFja3VwczJkYXRhbGFkL2FkYXRhc2V0LnB5) | `77.46% <72.72%> (-0.32%)` | :arrow_down: | | [tools/backups2datalad/aioutil.py](https://codecov.io/gh/dandi/dandisets/pull/314/diff?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=dandi#diff-dG9vbHMvYmFja3VwczJkYXRhbGFkL2Fpb3V0aWwucHk=) | `72.63% <72.72%> (+0.14%)` | :arrow_up: | | [tools/backups2datalad/\_\_main\_\_.py](https://codecov.io/gh/dandi/dandisets/pull/314/diff?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=dandi#diff-dG9vbHMvYmFja3VwczJkYXRhbGFkL19fbWFpbl9fLnB5) | `63.46% <100.00%> (+0.17%)` | :arrow_up: | | [tools/backups2datalad/annex.py](https://codecov.io/gh/dandi/dandisets/pull/314/diff?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=dandi#diff-dG9vbHMvYmFja3VwczJkYXRhbGFkL2FubmV4LnB5) | `84.50% <100.00%> (+0.22%)` | :arrow_up: | | [tools/backups2datalad/asyncer.py](https://codecov.io/gh/dandi/dandisets/pull/314/diff?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=dandi#diff-dG9vbHMvYmFja3VwczJkYXRhbGFkL2FzeW5jZXIucHk=) | `79.00% <100.00%> (-0.39%)` | :arrow_down: | | [tools/backups2datalad/consts.py](https://codecov.io/gh/dandi/dandisets/pull/314/diff?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/datasetter.py](https://codecov.io/gh/dandi/dandisets/pull/314/diff?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=dandi#diff-dG9vbHMvYmFja3VwczJkYXRhbGFkL2RhdGFzZXR0ZXIucHk=) | `77.31% <100.00%> (+0.23%)` | :arrow_up: | | ... and [1 more](https://codecov.io/gh/dandi/dandisets/pull/314/diff?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 at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.

yarikoptic commented 1 year ago

should this be undrafted/merged to continue backing up zarrs?