Skip to content
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

Put less verbose CPU info command first #2065

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

Put less verbose CPU info command first #2065

wants to merge 1 commit into from

Conversation

opotowsky
Copy link
Member

@opotowsky opotowsky commented Jan 28, 2025

What is the change?

Changing the order of CPU info commands _getSystemInfoLinux uses.

Why is the change being made?

lscpu is much less verbose and should satisfy most logging needs.


Checklist

  • This PR has only one purpose or idea.
  • Tests have been added/updated to verify any new/changed code. No change to tests
  • The release notes have been updated if necessary. N/A: tiny update
  • The documentation is still up-to-date in the doc folder.
  • The dependencies are still up-to-date in pyproject.toml.

@opotowsky opotowsky added the low priority Style points and non-features label Jan 28, 2025
@albeanth
Copy link
Member

image
low priority, but such a big win imo

@opotowsky
Copy link
Member Author

(This can be merged whenever @john-science!)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
low priority Style points and non-features
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants