Skip to content

Conversation

@FabioLuporini
Copy link
Contributor

Needed if one switches backend dynamically, e.g. via switchconfig(platform='cpu64')

@FabioLuporini FabioLuporini requested a review from mloubout May 26, 2025 15:44
@FabioLuporini FabioLuporini added the bug-C bug in the generated code label May 26, 2025
@codecov
Copy link

codecov bot commented May 26, 2025

Codecov Report

Attention: Patch coverage is 64.10256% with 14 lines in your changes missing coverage. Please review.

Project coverage is 91.99%. Comparing base (c05fabb) to head (37dae7f).
Report is 2 commits behind head on main.

Files with missing lines Patch % Lines
devito/arch/archinfo.py 59.25% 10 Missing and 1 partial ⚠️
tests/test_gpu_common.py 75.00% 1 Missing and 2 partials ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #2619      +/-   ##
==========================================
+ Coverage   91.92%   91.99%   +0.06%     
==========================================
  Files         245      245              
  Lines       48461    48495      +34     
  Branches     4255     4261       +6     
==========================================
+ Hits        44547    44612      +65     
+ Misses       3238     3202      -36     
- Partials      676      681       +5     
Flag Coverage Δ
pytest-gpu-aomp-amdgpuX 72.55% <53.84%> (+0.03%) ⬆️
pytest-gpu-nvc-nvidiaX 73.61% <48.71%> (+0.03%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

Copy link
Contributor

@EdCaunt EdCaunt left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Seems reasonable

@FabioLuporini FabioLuporini merged commit 6140492 into main May 27, 2025
34 checks passed
@FabioLuporini FabioLuporini deleted the tweak-autopad branch May 27, 2025 12:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug-C bug in the generated code

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants