Skip to content

Fix ncu_parser for different csv column count #1

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

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

TKONIY
Copy link

@TKONIY TKONIY commented Mar 3, 2024

My Environment

  • ncu (Version 2023.1.0.0)
  • nvcc (release 12.1, V12.1.66)
  • driver version: 530.30.02
  • hardware: NVIDIA A10

Bug report

When the original ncu_parser.py parses the output CSV file generated by ncu, it asserts the number of metrics of each row is 12. While in my setting it is 15, it skips all the lines.

if len(line) != 12:
continue

Fix

While the number of metrics may differs in different ncu versions, a flexible way may be filtering the invalid row according to the actual metric number achieved from the first row.

@TKONIY TKONIY changed the title Fix ncu_parser for different csv column width Fix ncu_parser for different csv column count Mar 3, 2024
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