Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

api: enforce interpolation radius to be smaller than any input space … #2234

Merged
merged 1 commit into from
Oct 13, 2023

Conversation

mloubout
Copy link
Contributor

…order

@mloubout mloubout added the API api (symbolics, types, ...) label Oct 12, 2023
@codecov
Copy link

codecov bot commented Oct 12, 2023

Codecov Report

Merging #2234 (5e60d91) into master (074df11) will increase coverage by 0.00%.
The diff coverage is 95.45%.

@@           Coverage Diff           @@
##           master    #2234   +/-   ##
=======================================
  Coverage   87.03%   87.04%           
=======================================
  Files         229      229           
  Lines       40935    40955   +20     
  Branches     7493     7498    +5     
=======================================
+ Hits        35629    35650   +21     
+ Misses       4694     4693    -1     
  Partials      612      612           
Files Coverage Δ
devito/operations/interpolators.py 98.25% <100.00%> (+0.13%) ⬆️
devito/types/sparse.py 88.70% <100.00%> (+0.02%) ⬆️
tests/test_dle.py 95.92% <100.00%> (ø)
tests/test_mpi.py 99.18% <100.00%> (ø)
tests/test_operator.py 97.60% <100.00%> (ø)
tests/test_gpu_common.py 1.42% <0.00%> (ø)
tests/test_interpolation.py 96.87% <94.73%> (-0.20%) ⬇️

... and 1 file with indirect coverage changes

@mloubout mloubout force-pushed the sradius branch 7 times, most recently from 7d36f3c to c8fd8bd Compare October 12, 2023 23:11
@FabioLuporini FabioLuporini merged commit b6f7308 into master Oct 13, 2023
32 checks passed
@FabioLuporini FabioLuporini deleted the sradius branch October 13, 2023 07:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
API api (symbolics, types, ...)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants