datalad / datalad-ukbiobank

Resources for working with UKBiobank as a DataLad dataset
MIT License
6 stars 12 forks source link

Consolidate BIDS mapping code into a single invokation per update #40

Closed mih closed 4 years ago

mih commented 4 years ago

This prepares for gh-38 (maintain concurrent BIDS and non-BIDS branches)

codecov-commenter commented 4 years ago

Codecov Report

Merging #40 into master will increase coverage by 1.20%. The diff coverage is 45.45%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master      #40      +/-   ##
==========================================
+ Coverage   57.40%   58.60%   +1.20%     
==========================================
  Files           9        9              
  Lines         500      529      +29     
==========================================
+ Hits          287      310      +23     
- Misses        213      219       +6     
Impacted Files Coverage Δ
datalad_ukbiobank/update.py 76.63% <44.44%> (+1.95%) :arrow_up:
datalad_ukbiobank/ukb2bids.py 13.04% <50.00%> (-0.29%) :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 51049ee...52517d5. Read the comment docs.

mih commented 4 years ago

Force pushed to expose to extended test battery #41