Skip to content

Conversation

@Leymore
Copy link
Contributor

@Leymore Leymore commented Dec 25, 2025

This PR fixes a ZeroDivisionError that occurs when the average value of the measured metric is zero.

The PR also adds a test for collecting the sm__idc_divergent_instructions.avg metric which is zero for a simple vector add, hence verifying nsight-python behaviour when metric value is zero.

@copy-pr-bot
Copy link

copy-pr-bot bot commented Dec 25, 2025

This pull request requires additional validation before any workflows can run on NVIDIA's runners.

Pull request vetters can view their responsibilities here.

Contributors can view more details about this message here.

@Alok-Joshi
Copy link
Collaborator

How did you encounter this issue? Which metric were you trying to collect? It would be useful if you extend test_parameter_metric test to include this case as well.

@Alok-Joshi
Copy link
Collaborator

/ok to test f31deab

@Alok-Joshi
Copy link
Collaborator

@Leymore can you please merge the latest main branch from upstream? We have fixed the CI issues.

@Alok-Joshi
Copy link
Collaborator

/ok to test 11a2042

Signed-off-by: Alok Joshi <[email protected]>
@Alok-Joshi
Copy link
Collaborator

/ok to test d24ccb0

@Alok-Joshi
Copy link
Collaborator

/ok to test 46f8d11

@Alok-Joshi Alok-Joshi self-assigned this Jan 14, 2026
@nvssatoor nvssatoor merged commit 6c364d7 into NVIDIA:main Jan 16, 2026
3 checks 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.

4 participants