-
Notifications
You must be signed in to change notification settings - Fork 202
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
[CI] todo improvements #173
Comments
Fabien, I have tried install the new master after your pull request. |
Hi Fabien, is the building in CI parallelized? Xiangyu |
Some tests are two step test, that is, the first step for generating particles, the second for running the simulation. In windows, the combined test can also be done by script. |
Ninja build system does it in parallel by default.
If those tests are meant to be run together, then they shouldn't be split in the first place. The source file should be doing both, then we can remove having 2
I didn't face it, did you start from a fresh environment? Please make a new issue with the complete log if it is a blocking issue |
Fabien, the situation is, in practice these two parts are not always together. it is together only in the test.
|
The test is already the same executable with 2 |
-march=native
#176 to use-march=native
everywhere except on CI where it seems to cause problems (causing illegal instructions randomly and hard to debug). See also SIMD on CI runners: occurrence of "illegal" instructions #182The text was updated successfully, but these errors were encountered: