search

MagneticResonanceImaging / MRIReco.jl

Julia Package for MRI Reconstruction
https://magneticresonanceimaging.github.io/MRIReco.jl/latest/
Other
85 stars 22 forks source link

Fix fieldmap nfft copy #153

Closed alexjaffray closed 1 year ago

alexjaffray commented 1 year ago

Fix copying of FieldmapNFFTOp to return a copy that is the same size in memory as the original op. Prior to this PR, copying a FieldmapNFFTOp would return a copy larger than the original in memory.

codecov-commenter commented 1 year ago

Codecov Report

Patch coverage has no change and project coverage change: +0.49 :tada:

Comparison is base (4457ec4) 65.67% compared to head (58b2698) 66.17%.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## master #153 +/- ## ========================================== + Coverage 65.67% 66.17% +0.49% ========================================== Files 9 9 Lines 268 269 +1 ========================================== + Hits 176 178 +2 + Misses 92 91 -1 ``` [see 1 file with indirect coverage changes](https://app.codecov.io/gh/MagneticResonanceImaging/MRIReco.jl/pull/153/indirect-changes?src=pr&el=tree-more&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=MagneticResonanceImaging)

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.

tknopp commented 1 year ago

looks good