search

datalad / datalad-xnat

Track XNAT projects with DataLad
Other
3 stars 9 forks source link

Rough outline of API changes #122

Closed mih closed 2 years ago

mih commented 2 years ago

Also sketches where and how interactive exploration would take place in init().

Also fixes datalad/datalad-xnat#63 by inspecting the config, instead of a directory in a dataset.

codecov-commenter commented 2 years ago

Codecov Report

Merging #122 (79932e0) into master (712b3c7) will increase coverage by 1.56%. The diff coverage is 73.33%.

:exclamation: Current head 79932e0 differs from pull request most recent head 57940a3. Consider uploading reports for the commit 57940a3 to get more accurate results Impacted file tree graph

@@            Coverage Diff             @@
##           master     #122      +/-   ##
==========================================
+ Coverage   86.52%   88.08%   +1.56%     
==========================================
  Files          11       11              
  Lines         371      361      -10     
==========================================
- Hits          321      318       -3     
+ Misses         50       43       -7
Impacted Files Coverage Δ
datalad_xnat/platform.py 81.72% <ø> (ø)
datalad_xnat/query_files.py 86.53% <ø> (ø)
datalad_xnat/tests/test_update.py 100.00% <ø> (ø)
datalad_xnat/tests/test_xnatcentral.py 100.00% <ø> (ø)
datalad_xnat/init.py 79.03% <60.00%> (+1.25%) :arrow_up:
datalad_xnat/update.py 94.11% <82.60%> (+7.45%) :arrow_up:
datalad_xnat/parser.py 86.66% <100.00%> (-0.84%) :arrow_down:

Continue to review full report at Codecov.

Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update 712b3c7...57940a3. Read the comment docs.

mih commented 2 years ago

Will merge this now to expose the breakage to more folks