tests: Miscellaneous fixes for the test suite#2929
Conversation
Codecov Report❌ Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #2929 +/- ##
==========================================
- Coverage 83.55% 83.53% -0.03%
==========================================
Files 257 257
Lines 53526 53536 +10
Branches 4583 4584 +1
==========================================
- Hits 44726 44722 -4
- Misses 8007 8019 +12
- Partials 793 795 +2
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Harness. 🚀 New features to boost your workflow:
|
…ronment variable for GPU
…e_outer on POWER architecture
754998d to
e041b0c
Compare
|
|
||
| # Use any GPU present for parameters such as the padding by default. This can be | ||
| # disabled by setting this parameter to `0`, but should not be toggled. | ||
| configuration.add('gpu', 1, [0, 1], preprocessor=bool) |
There was a problem hiding this comment.
I don't think this is needed, and in general, it seems off to have such an option in configuration
I'm not even sure I understand the comment above...
Why can we not use a combo of skipif and platform instead?
There was a problem hiding this comment.
It was needed because it was taking out my local machine!
Relevant discussion: https://devitocodes.slack.com/archives/C7JMLMSG0/p1778763735024689
Encountered when running outside of CI