slimgroup / JUDI.jl

Julia Devito inversion.
https://slimgroup.github.io/JUDI.jl
MIT License
98 stars 32 forks source link

Fix to judiweights so that you can do spg on it directly #27

Closed rafaelorozco closed 4 years ago

rafaelorozco commented 4 years ago

Fix to judiweights so that you can do spg on it directly with out the return array hack.

added the unary operator -judiWeights and an iterator so that isLegal(judiWeights) works.

And tests for both.

codecov[bot] commented 4 years ago

Codecov Report

Merging #27 into master will increase coverage by 1.10%. The diff coverage is 100.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master      #27      +/-   ##
==========================================
+ Coverage   57.56%   58.67%   +1.10%     
==========================================
  Files          28       28              
  Lines        2194     2156      -38     
==========================================
+ Hits         1263     1265       +2     
+ Misses        931      891      -40     
Impacted Files Coverage Δ
src/TimeModeling/judiWeights.jl 53.27% <100.00%> (+25.71%) :arrow_up:
src/TimeModeling/ModelStructure.jl 50.00% <0.00%> (-21.43%) :arrow_down:
src/TimeModeling/judiExtendedSource.jl 52.94% <0.00%> (-14.71%) :arrow_down:
...c/TimeModeling/extended_source_interface_serial.jl 68.18% <0.00%> (-9.10%) :arrow_down:
src/TimeModeling/time_modeling_parallel.jl 82.35% <0.00%> (-8.83%) :arrow_down:
src/TimeModeling/time_modeling_serial.jl 73.07% <0.00%> (-7.70%) :arrow_down:
...TimeModeling/extended_source_interface_parallel.jl 80.64% <0.00%> (-6.46%) :arrow_down:
src/TimeModeling/judiPDEextended.jl 56.52% <0.00%> (-4.35%) :arrow_down:
src/TimeModeling/judiWavefield.jl 21.42% <0.00%> (-3.58%) :arrow_down:
src/TimeModeling/fwi_objective_serial.jl 84.84% <0.00%> (-3.04%) :arrow_down:
... and 12 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 41c8e46...d63a57f. Read the comment docs.

codecov[bot] commented 4 years ago

Codecov Report

Merging #27 into master will increase coverage by 1.28%. The diff coverage is 75.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master      #27      +/-   ##
==========================================
+ Coverage   57.56%   58.85%   +1.28%     
==========================================
  Files          28       28              
  Lines        2194     2197       +3     
==========================================
+ Hits         1263     1293      +30     
+ Misses        931      904      -27     
Impacted Files Coverage Δ
src/TimeModeling/judiWeights.jl 50.00% <75.00%> (+22.44%) :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 41c8e46...d63a57f. Read the comment docs.