Skip to content

Fix #1051 - Fix a warning about CPU when RAPL works so did not need CPU#1060

Open
benoit-cty wants to merge 8 commits intomasterfrom
fix/#1051
Open

Fix #1051 - Fix a warning about CPU when RAPL works so did not need CPU#1060
benoit-cty wants to merge 8 commits intomasterfrom
fix/#1051

Conversation

@benoit-cty
Copy link
Contributor

@benoit-cty benoit-cty commented Feb 13, 2026

Description

  • Add missing CPU
  • Fix a warning about CPU when RAPL works so did not need CPU
  • Bump to release a new version
  • Improve coverage config

Related Issue

Please link to the issue this PR resolves: [issue #1051]

How Has This Been Tested?

test it by removing my CPU from the CSV : it works 👍

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.

@codecov
Copy link

codecov bot commented Feb 13, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 78.14%. Comparing base (cc62d32) to head (7552e88).
⚠️ Report is 5 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #1060      +/-   ##
==========================================
+ Coverage   68.73%   78.14%   +9.41%     
==========================================
  Files          41       37       -4     
  Lines        3873     3546     -327     
==========================================
+ Hits         2662     2771     +109     
+ Misses       1211      775     -436     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@benoit-cty benoit-cty marked this pull request as ready for review February 13, 2026 22:12
@benoit-cty benoit-cty requested a review from a team as a code owner February 13, 2026 22:12
@benoit-cty benoit-cty changed the title Fix #1051 Fix #1051 - Fix a warning about CPU when RAPL works so did not need CPU Feb 13, 2026
@benoit-cty
Copy link
Contributor Author

I manage to test it by removing my CPU from the CSV : it works 👍

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 @benoit-cty !

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.

You have a 13th Gen Intel(R) Core(TM) i5-1335U but we don't know it

2 participants