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

Add T8_ENABLE_MINIMAL_TESTS #1371

Open
lukasdreyer opened this issue Feb 3, 2025 · 1 comment · May be fixed by #1418
Open

Add T8_ENABLE_MINIMAL_TESTS #1371

lukasdreyer opened this issue Feb 3, 2025 · 1 comment · May be fixed by #1418
Assignees
Labels
priority: medium Should be solved within half a year

Comments

@lukasdreyer
Copy link
Collaborator

lukasdreyer commented Feb 3, 2025

Feature request

Is your feature request related to a problem? Please describe.
I'm always frustrated when I implement new features in the core and the testsuite runs too long.

Describe the solution or feature you'd like
I'd like to have the option to run a reduced testsuite covering only extreme cmeshes for example, where all tests combined run in under a minute using
CMAKE_CUSTOM_PARALLEL_TEST_COMMAND=mpirun -n 4, CTEST_PARALLEL_LEVEL=8, and the tests are executed using ninja test -j32.

Describe alternatives you've considered
Using GTEST_FILTER to only run specific tests, but often the failing tests are the long ones, and they would also fail, if we reduced the tests to a smaller subset of the generated tests.

@lukasdreyer lukasdreyer added the priority: high Should be solved as soon as possible label Feb 3, 2025
@lukasdreyer
Copy link
Collaborator Author

These are the tests currently preventing fast execution of all tests:

11/68 Test #33: t8_gtest_forest_commit_parallel ....................... Passed 16.04 sec
13/68 Test #31: t8_gtest_ghost_and_owner_parallel ..................... Passed 16.23 sec
19/68 Test #29: t8_gtest_ghost_exchange_parallel ...................... Passed 18.44 sec
31/68 Test #32: t8_gtest_balance_parallel ............................. Passed 10.62 sec
66/68 Test #50: t8_gtest_nca_serial ................................... Passed 42.39 sec
67/68 Test #35: t8_gtest_element_is_leaf_serial ....................... Passed 55.40 sec
68/68 Test #39: t8_gtest_iterate_replace_serial ....................... Passed 56.26 sec

@Davknapp Davknapp added priority: medium Should be solved within half a year and removed priority: high Should be solved as soon as possible labels Feb 3, 2025
@lenaploetzke lenaploetzke self-assigned this Feb 17, 2025
@lenaploetzke lenaploetzke linked a pull request Feb 18, 2025 that will close this issue
15 tasks
@lenaploetzke lenaploetzke removed a link to a pull request Feb 18, 2025
15 tasks
@lenaploetzke lenaploetzke linked a pull request Feb 20, 2025 that will close this issue
15 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
priority: medium Should be solved within half a year
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants