adswa commented 2 years ago

This can't do much yet, but it is a start towards the xnat-query command. I will continue working on this, but putting it here in case someone depends on it, or is faster and wants to take over

TODO:

[x] make the command create all queries that are needed to build the urls for download
[x] construct the urls
[ ] cache the urls/save them in a temporary place
[x] make init and update use query internally
[ ] add tests

mih commented 2 years ago

Quick comment: if only for the convenience of not having to specify --credential over and over again, it would make sense to support a --dataset argument.

codecov-commenter commented 2 years ago

Codecov Report

Merging #82 (38941db) into master (d9ebc46) will decrease coverage by 0.92%. The diff coverage is 72.00%.

:exclamation: Current head 38941db differs from pull request most recent head 2fe1f76. Consider uploading reports for the commit 2fe1f76 to get more accurate results

@@            Coverage Diff             @@
##           master      #82      +/-   ##
==========================================
- Coverage   82.42%   81.49%   -0.93%     
==========================================
  Files           9       10       +1     
  Lines         256      281      +25     
==========================================
+ Hits          211      229      +18     
- Misses         45       52       +7

Impacted Files	Coverage Δ
datalad_xnat/__init__.py	`100.00% <ø> (ø)`
datalad_xnat/query.py	`72.00% <72.00%> (ø)`

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 d9ebc46...2fe1f76. Read the comment docs.

datalad / datalad-xnat

NF: Implement a basic xnat-query command #82

Codecov Report