Skip to content

Python: keep default annotation messages local to each function - #186

Open
darkdi wants to merge 1 commit into
NVIDIA:devfrom
darkdi:darkdi/reusable-annotate-messages
Open

darkdi wants to merge 1 commit into
NVIDIA:devfrom
darkdi:darkdi/reusable-annotate-messages

Conversation

@darkdi

@darkdi darkdi commented Oct 23, 2026

Copy link
Copy Markdown

Reusing one nvtx.annotate() labels both functions with the first name and also changes later context-manager labels. Native injection tests reproduce both cases. Full Python tests pass with and without NumPy/injection on macOS arm64, Python 3.12; the sdist build and source validation pass.

Signed-off-by: Dmitry Rantovov <rantovov5@gmail.com>
@darkdi

darkdi commented Oct 23, 2026

Copy link
Copy Markdown
Author

Could you approve the Python and Source Validation workflow runs? Both currently show action_required, with no jobs started. Local results are included in the PR description.

This branch has not been deployed

No deployments
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.

1 participant