search

TheLabbingProject / django_mri

A reusable Django app to manage MRI data.
https://django-mri.readthedocs.io/en/latest/
Apache License 2.0
7 stars 5 forks source link

Added a small fix to TheBase naming according to BIDS specifications #89

Closed GalKepler closed 3 years ago

GalKepler commented 3 years ago

It's a small fix that offers a temporary solution for our problem described in #72.

codecov[bot] commented 3 years ago

Codecov Report

Merging #89 (46b6ac7) into master (18fe9f8) will increase coverage by 0.11%. The diff coverage is 77.50%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master      #89      +/-   ##
==========================================
+ Coverage   48.70%   48.81%   +0.11%     
==========================================
  Files         179      180       +1     
  Lines        3000     3028      +28     
  Branches      269      275       +6     
==========================================
+ Hits         1461     1478      +17     
- Misses       1522     1530       +8     
- Partials       17       20       +3
Impacted Files Coverage Δ
django_mri/models/scan.py 68.22% <44.44%> (-0.56%) :arrow_down:
django_mri/utils/the_base.py 75.00% <75.00%> (ø)
django_mri/utils/bids.py 86.45% <100.00%> (-2.59%) :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 18fe9f8...46b6ac7. Read the comment docs.