-
Notifications
You must be signed in to change notification settings - Fork 2
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
Added oneAPI extension and buildkite CI #43
Conversation
…d tests into simpler files. Added oneAPI backend. Added tests for extension finding.
…be installed on the GPU backends. Added oneAPI CI runner. TODO: put version back to 1.0.0 once they run, then update rest of ecosystem compat
…added Aqua to test/runtests.jl
Co-authored-by: Valentin Churavy <[email protected]>
Co-authored-by: Valentin Churavy <[email protected]>
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #43 +/- ##
===========================================
- Coverage 92.95% 60.16% -32.79%
===========================================
Files 6 9 +3
Lines 142 236 +94
===========================================
+ Hits 132 142 +10
- Misses 10 94 +84
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. 🚨 Try these New Features:
|
… will then need to be updated around the ecosystem
Okay, all backend extensions are found and their tests work when temporarily setting Atomix version=0.1.0 so that all packages can be downloaded (otherwise dependency conflicts appear). @maleadt I updated to version=1.0.0 like you did the first time in the initial pull request. Any other feedback is very welcome. |
Remove oneAPI from deps, as it was in weakdeps already Co-authored-by: Christian Guinard <[email protected]>
This is almost verbatim from the extensions refactor.
With this, we should now have atomics on all AcceleratedKernels.jl backends, yay!