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

Enable tests for Alpaka builds #653

Merged
merged 1 commit into from
Aug 21, 2024

Conversation

stephenswat
Copy link
Member

Now that we have some Alpaka tests, we are ready to actually run them in the CI.

@stephenswat stephenswat added cicd Changes related to the CI system alpaka Changes related to Alpaka labels Jul 25, 2024
@stephenswat
Copy link
Member Author

Interestingly, debug builds make Alpaka code so slow that it times out the CI. 🤔

@krasznaa
Copy link
Member

krasznaa commented Aug 2, 2024

Two options we have here:

  • Increase the timeout from 1500 seconds to something larger.
  • Only test Alpaka in Release mode.

I'm not even sure which of these two bad options is the better one. But probably I'd go for the timeout increase. 🤔

@stephenswat
Copy link
Member Author

It's really interesting why the Alpaka code is so much slower; we could also try adding an -O2 to the Alpaka build; also far from ideal, though.

@StewMH
Copy link
Contributor

StewMH commented Aug 12, 2024

I think disabling for debug is fine.

@stephenswat
Copy link
Member Author

Okay, I have updated this to run the Alpaka tests for release builds only.

Now that we have some Alpaka tests, we are ready to actually run them in
the CI.
@stephenswat stephenswat merged commit 3991be9 into acts-project:main Aug 21, 2024
24 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
alpaka Changes related to Alpaka cicd Changes related to the CI system
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants