slimgroup / JUDI.jl

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

Remove extend_gradient #142

Closed mloubout closed 1 year ago

mloubout commented 1 year ago

move aperture combination to PhysicalParameter arithmetic to avoid large zero padding and communication

Now the arithmetic of m1 + m2 for two different PhysicalParameter process different size and origin to combine them.

codecov[bot] commented 1 year ago

Codecov Report

Base: 84.11% // Head: 83.96% // Decreases project coverage by -0.14% :warning:

Coverage data is based on head (0cc7e82) compared to base (03ea073). Patch coverage: 91.89% of modified lines in pull request are covered.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## master #142 +/- ## ========================================== - Coverage 84.11% 83.96% -0.15% ========================================== Files 24 24 Lines 1939 1940 +1 ========================================== - Hits 1631 1629 -2 - Misses 308 311 +3 ``` | [Impacted Files](https://codecov.io/gh/slimgroup/JUDI.jl/pull/142?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=slimgroup) | Coverage Δ | | |---|---|---| | [src/JUDI.jl](https://codecov.io/gh/slimgroup/JUDI.jl/pull/142/diff?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=slimgroup#diff-c3JjL0pVREkuamw=) | `83.33% <ø> (ø)` | | | [src/TimeModeling/Modeling/time\_modeling\_serial.jl](https://codecov.io/gh/slimgroup/JUDI.jl/pull/142/diff?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=slimgroup#diff-c3JjL1RpbWVNb2RlbGluZy9Nb2RlbGluZy90aW1lX21vZGVsaW5nX3NlcmlhbC5qbA==) | `95.45% <ø> (+3.78%)` | :arrow_up: | | [src/TimeModeling/Utils/auxiliaryFunctions.jl](https://codecov.io/gh/slimgroup/JUDI.jl/pull/142/diff?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=slimgroup#diff-c3JjL1RpbWVNb2RlbGluZy9VdGlscy9hdXhpbGlhcnlGdW5jdGlvbnMuamw=) | `78.46% <0.00%> (-1.01%)` | :arrow_down: | | [src/TimeModeling/Types/ModelStructure.jl](https://codecov.io/gh/slimgroup/JUDI.jl/pull/142/diff?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=slimgroup#diff-c3JjL1RpbWVNb2RlbGluZy9UeXBlcy9Nb2RlbFN0cnVjdHVyZS5qbA==) | `85.41% <94.11%> (-0.42%)` | :arrow_down: | | [src/TimeModeling/Modeling/misfit\_fg.jl](https://codecov.io/gh/slimgroup/JUDI.jl/pull/142/diff?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=slimgroup#diff-c3JjL1RpbWVNb2RlbGluZy9Nb2RlbGluZy9taXNmaXRfZmcuamw=) | `92.85% <100.00%> (-0.25%)` | :arrow_down: | | [src/TimeModeling/Modeling/twri\_objective.jl](https://codecov.io/gh/slimgroup/JUDI.jl/pull/142/diff?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=slimgroup#diff-c3JjL1RpbWVNb2RlbGluZy9Nb2RlbGluZy90d3JpX29iamVjdGl2ZS5qbA==) | `70.00% <100.00%> (-0.59%)` | :arrow_down: | Help us with your feedback. Take ten seconds to tell us [how you rate us](https://about.codecov.io/nps?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=slimgroup). Have a feature suggestion? [Share it here.](https://app.codecov.io/gh/feedback/?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=slimgroup)

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