Skip to content

Comments

Turn some frequent DGX errors into warnings#29

Merged
metalmatze merged 2 commits intomainfrom
dgx-fixes
Jan 7, 2026
Merged

Turn some frequent DGX errors into warnings#29
metalmatze merged 2 commits intomainfrom
dgx-fixes

Conversation

@gnurizen
Copy link
Contributor

No description provided.

@gnurizen gnurizen requested a review from metalmatze November 18, 2025 20:10
Copy link
Member

@metalmatze metalmatze left a comment

Choose a reason for hiding this comment

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

I'm fine merging this on the client side.
Now this begs the question, how useful is our GPU profiling page without all of these metrics that the DGX doesn't support...

Are there newer APIs we want to support for DGX? We should investigate that.

@gnurizen
Copy link
Contributor Author

gnurizen commented Nov 19, 2025

Yeah I think we should sit on these fixes until we:

  • fix the UI to how it looks
  • look for alternatives

I think we need to look for solutions for memory utilization and power consumption which nvtop provides, must be a different way to get it for the DGX.

Copy link
Member

@metalmatze metalmatze left a comment

Choose a reason for hiding this comment

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

Let's get this in for now and then see if there's another way to get to these metrics on the DGX. There has to be a way.

@metalmatze metalmatze merged commit a47b443 into main Jan 7, 2026
3 of 4 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.

2 participants