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: Replace terms.md page with page that provides example of API syntax mapping #3726

Merged
merged 1 commit into from
Feb 5, 2025

Conversation

adeljo-amd
Copy link

The goal of this PR is to overhaul the terms.md file with a page that demonstrates some examples of how the CUDA and HIP API syntaxes map.

@adeljo-amd adeljo-amd self-assigned this Jan 22, 2025
@adeljo-amd adeljo-amd force-pushed the api_syntax branch 2 times, most recently from fef831e to 04769fd Compare January 28, 2025 12:54
@adeljo-amd adeljo-amd requested a review from neon60 January 28, 2025 12:56
@adeljo-amd adeljo-amd changed the title Draft: Docs: Replace terms.md page with page that provides example of API syntax mapping Docs: Replace terms.md page with page that provides example of API syntax mapping Jan 29, 2025
@adeljo-amd adeljo-amd added ci:docs-only Only run Read the Docs CI on this PR documentation labels Feb 3, 2025
@bbrzak-amd
Copy link

LGTM

Copy link

@randyh62 randyh62 left a comment

Choose a reason for hiding this comment

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

Left a few comments.

docs/faq.rst Outdated Show resolved Hide resolved
docs/reference/api_syntax.rst Outdated Show resolved Hide resolved
docs/index.md Outdated Show resolved Hide resolved
docs/reference/api_syntax.rst Show resolved Hide resolved
docs/reference/api_syntax.rst Outdated Show resolved Hide resolved
docs/reference/api_syntax.rst Outdated Show resolved Hide resolved
docs/reference/api_syntax.rst Outdated Show resolved Hide resolved
@adeljo-amd adeljo-amd merged commit c36483f into docs/develop Feb 5, 2025
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ci:docs-only Only run Read the Docs CI on this PR documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants