Skip to content

refactor: Cleanup report module#609

Merged
gfieni merged 2 commits intomasterfrom
refactor/cleanup-report-module
May 14, 2025
Merged

refactor: Cleanup report module#609
gfieni merged 2 commits intomasterfrom
refactor/cleanup-report-module

Conversation

@gfieni
Copy link
Contributor

@gfieni gfieni commented May 14, 2025

PR overview:

  • Remove the unused procfs report ;
  • Remove unused methods from power and hwpc reports ;
  • Cleanup the unit tests of the report module.

@gfieni gfieni self-assigned this May 14, 2025
@gfieni gfieni requested a review from a team as a code owner May 14, 2025 13:25
@gfieni gfieni force-pushed the refactor/cleanup-report-module branch from 7f4dac4 to 48980d0 Compare May 14, 2025 13:29
@sonarqubecloud
Copy link

@gfieni gfieni merged commit 3934229 into master May 14, 2025
8 of 9 checks passed
@gfieni gfieni deleted the refactor/cleanup-report-module branch May 14, 2025 13:45
@codecov
Copy link

codecov bot commented May 14, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 68.98%. Comparing base (c5573c7) to head (48980d0).
Report is 3 commits behind head on master.

Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #609      +/-   ##
==========================================
- Coverage   69.10%   68.98%   -0.12%     
==========================================
  Files          75       73       -2     
  Lines        2874     2757     -117     
==========================================
- Hits         1986     1902      -84     
+ Misses        888      855      -33     
Files with missing lines Coverage Δ
src/powerapi/cli/generator.py 84.37% <100.00%> (-0.10%) ⬇️
src/powerapi/dispatch_rule/__init__.py 100.00% <ø> (ø)
src/powerapi/report/__init__.py 100.00% <100.00%> (ø)
src/powerapi/report/hwpc_report.py 92.20% <ø> (+0.95%) ⬆️
src/powerapi/report/power_report.py 81.33% <ø> (+4.86%) ⬆️
src/powerapi/report/report.py 95.08% <ø> (+1.33%) ⬆️
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant