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

Tests: Mark thermal as slow #520

Merged
merged 1 commit into from
Feb 4, 2024

Conversation

ax3l
Copy link
Member

@ax3l ax3l commented Feb 3, 2024

Mark the thermal test as slow.
This test runs 50sec in release mode, >10min in Debug mode.

Exclude in CI Debug builds.
Exclude in UBSAN/ASAN tests.

The thermal distribution implementation should be sufficiently covered by the faster running bithermal tests and continues to be run in the other tests in CI.

X-ref #422 #496

Close #501

@ax3l ax3l added the component: tests examples, tests and benchmarks label Feb 3, 2024
@ax3l ax3l requested a review from cemitch99 February 3, 2024 23:13
@ax3l ax3l force-pushed the ctest-mark-thermal-slow branch from 01b80cb to 1a7599b Compare February 3, 2024 23:43
Mark the `thermal` test as slow.
This test runs 50sec in release mode, >10min in Debug mode.

Exclude in CI Debug builds.
Exclude in UBSAN/ASAN tests.

The thermal distribution implementation should be sufficiently covered
by the faster running `bithermal` tests and continues to be run
in the other tests in CI.
@ax3l ax3l force-pushed the ctest-mark-thermal-slow branch from 1a7599b to e018e41 Compare February 3, 2024 23:46
@ax3l ax3l enabled auto-merge (squash) February 4, 2024 00:21
@ax3l ax3l merged commit dc5e2d1 into ECP-WarpX:development Feb 4, 2024
15 checks passed
@ax3l ax3l deleted the ctest-mark-thermal-slow branch February 4, 2024 02:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
component: tests examples, tests and benchmarks
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants