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

compiler: prevent radius dependent temps for sparse operations #2216

Merged
merged 1 commit into from
Sep 27, 2023

Conversation

mloubout
Copy link
Contributor

@mloubout mloubout commented Sep 26, 2023

Fix sparse loop to avoid weird access (see slack for issue with GPU)

@codecov
Copy link

codecov bot commented Sep 26, 2023

Codecov Report

Merging #2216 (e017800) into master (c36a25d) will increase coverage by 0.01%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master    #2216      +/-   ##
==========================================
+ Coverage   87.08%   87.10%   +0.01%     
==========================================
  Files         228      228              
  Lines       40715    40723       +8     
  Branches     7455     7456       +1     
==========================================
+ Hits        35458    35470      +12     
+ Misses       4653     4650       -3     
+ Partials      604      603       -1     
Files Coverage Δ
devito/ir/clusters/algorithms.py 95.33% <100.00%> (ø)
tests/test_dse.py 99.80% <100.00%> (+<0.01%) ⬆️

... and 2 files with indirect coverage changes

@mloubout mloubout merged commit a7d9c49 into master Sep 27, 2023
32 checks passed
@mloubout mloubout deleted the sparse-tmp branch September 27, 2023 11:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant