search

ptiede / Comrade.jl

MIT License
47 stars 7 forks source link

NFFT #85

Closed ptiede closed 2 years ago

ptiede commented 2 years ago

This pull request starts to add NFFT functionality to the package. This will require some minor modifications

codecov-commenter commented 2 years ago

Codecov Report

Merging #85 (b8635c7) into main (db03f5a) will increase coverage by 3.32%. The diff coverage is 81.36%.

Impacted file tree graph

@@            Coverage Diff             @@
##             main      #85      +/-   ##
==========================================
+ Coverage   76.56%   79.89%   +3.32%     
==========================================
  Files          15       26      +11     
  Lines         798     1318     +520     
==========================================
+ Hits          611     1053     +442     
- Misses        187      265      +78
Impacted Files Coverage Δ
src/observations/ehtim.jl 100.00% <ø> (ø)
src/models/modelimage/nfft_alg.jl 40.00% <40.00%> (ø)
src/models/modelimage/modelimage.jl 61.53% <47.05%> (ø)
src/models/modelimage/nuft.jl 52.30% <52.30%> (ø)
src/models/modifiers.jl 70.31% <57.14%> (-4.69%) :arrow_down:
src/inference/pathfinder.jl 65.51% <65.51%> (ø)
src/inference/galacticoptim.jl 69.23% <69.23%> (ø)
src/models/modelimage/dft_alg.jl 69.56% <69.56%> (ø)
src/models/combinators.jl 88.88% <70.00%> (-7.61%) :arrow_down:
src/models/modelimage/fft_alg.jl 63.04% <73.17%> (ø)
... and 23 more

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 db03f5a...b8635c7. Read the comment docs.