Skip to content

Add gpu metrics to system_metrics.csv#226

Merged
rcvalerio merged 1 commit into
devfrom
rv-add-gpu-metrics
Nov 13, 2025
Merged

Add gpu metrics to system_metrics.csv#226
rcvalerio merged 1 commit into
devfrom
rv-add-gpu-metrics

Conversation

@rcvalerio

@rcvalerio rcvalerio commented Nov 13, 2025

Copy link
Copy Markdown
Contributor

Comment thread task-runner/task_runner/system_monitor.py
Comment thread task-runner/task_runner/system_monitor.py
from typing import List, Literal, Optional, Tuple
from uuid import UUID

import GPUtil

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

This project is not maintained: https://pypi.org/project/GPUtil

Do you know if nvidia-smi is stable?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

From my testing GPUtil works great and we were already using it in the task-runner. Should we switch for another library?

I'm not sure, but wouldn't nvidia-smi only work on nvidia GPUs? We might need to support other types of GPU's (e.g. local machines)

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Please check this comment: https://github.com/inductiva/tasks/issues/1141#issuecomment-2969830897

Unfortunately, the current library and the others I mentioned in that comment are just wrappers of nvidia-smi. It might be enough for now, but if we want to run on different GPUs we may need to research an alternative.

@rcvalerio
rcvalerio merged commit 187414d into dev Nov 13, 2025
1 check passed
@rcvalerio
rcvalerio deleted the rv-add-gpu-metrics branch November 13, 2025 15:26
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.

3 participants