Skip to content

Conversation

@cianc
Copy link
Contributor

@cianc cianc commented Jan 18, 2026

Description

  • Create EmissionsTracker.get_detected_hardware() to return basic hardware information.
  • Update internal logging to use get_detected_hardware().
  • Create examples/print_hardware.py and unit tests.
  • Add detect command to CLI to report system hardware without starting a tracker.
  • Add documentation for the detect command in README and usage guide.

Related Issue

Please link to the issue this PR resolves: #1000

Motivation and Context

Feature request to make it easier to get hardware information.

How Has This Been Tested?

New unit test as well as an example script.

Screenshots (if appropriate):

Types of changes

What types of changes does your code introduce? Put an x in all the boxes that apply:

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

Go over all the following points, and put an x in all the boxes that apply.

  • My code follows the code style of this project.
  • My change requires a change to the documentation.
  • I have updated the documentation accordingly.
  • I have read the CONTRIBUTING.md document.
  • I have added tests to cover my changes.
  • All new and existing tests passed.

… without running measurements.

   * Add `detect` command to CLI to report system hardware without starting a tracker.
   * Create `EmissionsTracker.get_detected_hardware()` to return basic hardware information.
   * Update internal logging to use `get_detected_hardware()`.
   * Create `examples/print_hardware.py` and unit tests.
   * Add documentation for the `detect` command in README and usage guide.
@cianc cianc marked this pull request as ready for review January 18, 2026 20:19
@cianc cianc requested a review from a team as a code owner January 18, 2026 20:19
Copy link
Collaborator

@inimaz inimaz left a comment

Choose a reason for hiding this comment

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

Thanks a lot @cianc for this. Looking good!
This will help decouple the setup from the measurement itself.

@inimaz inimaz added the enhancement New feature or request label Jan 19, 2026
@inimaz inimaz merged commit bfc5656 into master Jan 19, 2026
8 checks passed
@inimaz inimaz deleted the feature/hardware_reporting branch January 19, 2026 08:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Split hardware detection from measurement

3 participants