-
Notifications
You must be signed in to change notification settings - Fork 3
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 option to Bootloader.jl to write CPU and GPU tests in different output yaml files #98
add option to Bootloader.jl to write CPU and GPU tests in different output yaml files #98
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for this! Looks good to me, I just picked some nits in the docstrings and suggested better naming for two command line arguments.
Feel free to merge after addressing the comments.
…utput yaml files - add function to disable integration tests
33e3292
to
ad659f5
Compare
@szabo137 I think you can merge. The CUDA runner of the test PR had a time out and now PIConGPU is spawning a lot of jobs because of the release. The generated pipelines looks also correct. I don't believe that my last commit on this PR changed something the content of the generated jobs, therefore it should be fine. |
Ok, I trust you and will merge. However, if it breaks, you owe me a beer 😉 |
tests: