Closed paquiteau closed 2 years ago
Adding @zaccharieramzi for review..
@paquiteau maybe add some description?
Merging #135 (b5d8907) into master (76ddca8) will increase coverage by
0.15%
. The diff coverage is93.33%
.
@@ Coverage Diff @@
## master #135 +/- ##
==========================================
+ Coverage 81.18% 81.33% +0.15%
==========================================
Files 39 39
Lines 1982 2009 +27
==========================================
+ Hits 1609 1634 +25
- Misses 373 375 +2
Flag | Coverage Δ | |
---|---|---|
unittests | 81.33% <93.33%> (+0.15%) |
:arrow_up: |
Flags with carried forward coverage won't be shown. Click here to find out more.
Impacted Files | Coverage Δ | |
---|---|---|
...i/reconstructors/utils/extract_sensitivity_maps.py | 85.39% <91.42%> (+0.67%) |
:arrow_up: |
mri/tests/test_sensitivity_extraction.py | 98.64% <100.00%> (+0.21%) |
:arrow_up: |
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 76ddca8...b5d8907. Read the comment docs.
This enhance the Smaps estimations by proposing a windowing before performing the adjoint (NU)FFT. The window is circular invariant and uses a circular mask (thresholded in the same way than the square one).