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

[Docs/PTX] Add device tensor map init example #1983

Open
wants to merge 8 commits into
base: main
Choose a base branch
from

Conversation

ahendriksen
Copy link
Contributor

Description

closes #1982

Adds documentation of tensormap modification/initialization on device. Also improves on navigation and table of contents of the cuda::ptx docs.

Checklist

  • New or existing tests cover these changes.
  • The documentation is up to date with these changes.

@ahendriksen ahendriksen requested review from a team as code owners July 12, 2024 11:57
-----------------

The full code example is included below. The `cuda::ptx` instructions in this
tutorial have become available in CUDA Toolkit 12.5.
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Godbolt does not yet have CUDA 12.5, so there is no link here. I didn't want to link to a broken example.

@ahendriksen
Copy link
Contributor Author

Please don't merge yet. I still have to incorporate some internal feedback.

@ahendriksen ahendriksen added the blocked This PR cannot be merged due to various reasons label Jul 25, 2024
@ahendriksen ahendriksen force-pushed the docs-add-device-tensor-map-init-example branch from 68653eb to 90b6aa8 Compare October 7, 2024 13:01
@ahendriksen
Copy link
Contributor Author

In the mean time, the code example for on-device tensor map modification has made it into the CUDA programming guide. Instead of duplicating the documentation and code sample, I have linked to the relevant section in the programming guide. The improvements to table of contents and layout are still very much worth it in this PR.

I have copied over the fixed links from Bryan van de Ven's PR.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
blocked This PR cannot be merged due to various reasons
Projects
Status: In Review
Development

Successfully merging this pull request may close these issues.

[DOC]: PTX: Document how to initialize/modify a tensor map in device memory
5 participants