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

Speed up ci compilation with wrapping on follow up #706

Conversation

fbudin69500
Copy link
Contributor

No description provided.

Francois Budin added 2 commits April 9, 2019 13:20
Python tests are moved from `test` subfolder to `wrapping/test/python`.
This allows to disable C++ tests and Python (or other wrapping) tests
independently. This is very convenient to speed up compilation and
testing on CI platforms.

A new advanced CMake variable `DISABLE_MODULE_TESTS` has been added to
disable all the tests that are not part of the wrapping. All the ITK
module tests that are in the standard `test` subfolder of each module are
disabled when this option is turned ON. The only tests that are left to
compiled and run are tests in third party modules and wrapping tests.
This new option allows to significantly speed up compilation time
(about 20 to 25%).
…te folder

CMake variable was incorrectly named and therefore not used. The name is now
correct.
Baseline files were store in test folder instead of being propertly organized
in a 'Baseline' subfolder.
@fbudin69500
Copy link
Contributor Author

fbudin69500 commented Apr 9, 2019

Note: Close #688 after merging this PR.

Filtering tests that are run by label on Linux and Windows to save a little
bit of time. Do not worry about MacOS where there is no timing out issue
on CI infrastructure.
@fbudin69500 fbudin69500 force-pushed the speed_up_CI_compilation_with_wrapping_ON_follow_up branch from 0f6503c to 5498f78 Compare April 9, 2019 19:52
@fbudin69500 fbudin69500 merged commit 6afa542 into InsightSoftwareConsortium:master Apr 9, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants