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

Rename oneMKL to oneMath and improve the lesson/exercise #387

Merged
merged 3 commits into from
Jan 15, 2025

Conversation

rafbiels
Copy link
Collaborator

Following the upstream renaming, rename oneMKL to oneMath in both the lesson slides and the corresponding code exercise.

Also improve the material:

  • Code: Replace the global queue::wait calls with proper synchronisation, which is automatic in case of buffers and explicitly passing sycl::events into APIs with USM.
  • Slides: fix the overflowing layout and expand the slides to explain what oneMath is.

Note that git marks the README, CMakeLists.txt and index.html as deleted files + new files added because the renaming affected many lines and the file name, but it's really just a file rename + a few edits. Unfortunately, this means git diff cannot be generated for them, so they can only be compared with a local diff (or by eye, side by side).

Change the directory name, file names, and all references to oneMKL
to the new name oneMath. Replace the global queue::wait calls with
proper synchronisation, which is automatic in case of buffers and
explicitly passing sycl::events into APIs with USM.
* Rename oneMKL to oneMath in the slides.
* Rename the relevant files and directory accordingly
* Improve the formatting of the slides
* Expand the slides to explain what oneMath is
…isation

Same thing (in this case!) but fewer characters
@DuncanMcBain DuncanMcBain merged commit bc79770 into codeplaysoftware:main Jan 15, 2025
1 check passed
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