ad12 / meddlr

A flexible ML framework built to simplify medical image reconstruction and analysis experimentation.
Apache License 2.0
62 stars 14 forks source link

bugfix: add support for multi-channel inputs for MR reconstruction models #19

Closed ad12 closed 2 years ago

ad12 commented 2 years ago

This PR adds minimal support for multi-channel inputs for MR reconstruction and patches runtime errors seen in the skm-tea repository. Additional support will be provided in future PRs.

Note, in the short term, the SenseModel will support multi-channel inputs when only one set of maps for each coil is estimated. This will be fixed in #18

Changelist

codecov-commenter commented 2 years ago

Codecov Report

Merging #19 (2a00ed8) into main (296e6c3) will increase coverage by 0.02%. The diff coverage is 100.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##             main      #19      +/-   ##
==========================================
+ Coverage   58.10%   58.13%   +0.02%     
==========================================
  Files          76       76              
  Lines        5607     5611       +4     
  Branches     1194     1196       +2     
==========================================
+ Hits         3258     3262       +4     
  Misses       2040     2040              
  Partials      309      309              
Flag Coverage Δ
unittests 58.13% <100.00%> (+0.02%) :arrow_up:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
meddlr/ops/fft.py 45.00% <ø> (ø)
meddlr/forward/mri.py 77.14% <100.00%> (+2.94%) :arrow_up:
meddlr/ops/complex.py 34.22% <100.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 296e6c3...2a00ed8. Read the comment docs.