Skip to content

refactor: Cleanup unused attributes in actor message types#615

Merged
gfieni merged 2 commits intomasterfrom
refactor/cleanup-unused-report-attrs
May 21, 2025
Merged

refactor: Cleanup unused attributes in actor message types#615
gfieni merged 2 commits intomasterfrom
refactor/cleanup-unused-report-attrs

Conversation

@gfieni
Copy link
Copy Markdown
Contributor

@gfieni gfieni commented May 21, 2025

No description provided.

@gfieni gfieni self-assigned this May 21, 2025
@gfieni gfieni requested a review from a team as a code owner May 21, 2025 14:46
@gfieni gfieni force-pushed the refactor/cleanup-unused-report-attrs branch 2 times, most recently from 362a3c6 to 8769de8 Compare May 21, 2025 14:57
@gfieni gfieni force-pushed the refactor/cleanup-unused-report-attrs branch from 8769de8 to adec0e8 Compare May 21, 2025 15:01
@sonarqubecloud
Copy link
Copy Markdown

@codecov
Copy link
Copy Markdown

codecov bot commented May 21, 2025

Codecov Report

Attention: Patch coverage is 57.14286% with 15 lines in your changes missing coverage. Please review.

Project coverage is 68.64%. Comparing base (3ae78f7) to head (adec0e8).
Report is 3 commits behind head on master.

Files with missing lines Patch % Lines
src/powerapi/puller/handlers.py 0.00% 5 Missing ⚠️
src/powerapi/report/control_report.py 25.00% 3 Missing ⚠️
src/powerapi/actor/actor.py 0.00% 2 Missing ⚠️
src/powerapi/handler/start_handler.py 0.00% 2 Missing ⚠️
src/powerapi/pusher/handlers.py 0.00% 1 Missing ⚠️
src/powerapi/report/hwpc_report.py 75.00% 1 Missing ⚠️
src/powerapi/report/power_report.py 66.66% 1 Missing ⚠️
Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #615      +/-   ##
==========================================
- Coverage   68.75%   68.64%   -0.11%     
==========================================
  Files          72       72              
  Lines        2720     2682      -38     
==========================================
- Hits         1870     1841      -29     
+ Misses        850      841       -9     
Files with missing lines Coverage Δ
src/powerapi/actor/supervisor.py 65.95% <100.00%> (ø)
src/powerapi/message.py 100.00% <100.00%> (+16.21%) ⬆️
src/powerapi/report/formula_report.py 100.00% <100.00%> (+4.54%) ⬆️
src/powerapi/report/report.py 96.00% <100.00%> (+0.91%) ⬆️
src/powerapi/pusher/handlers.py 36.36% <0.00%> (ø)
src/powerapi/report/hwpc_report.py 93.50% <75.00%> (+1.29%) ⬆️
src/powerapi/report/power_report.py 81.33% <66.66%> (ø)
src/powerapi/actor/actor.py 53.12% <0.00%> (ø)
src/powerapi/handler/start_handler.py 35.71% <0.00%> (ø)
src/powerapi/report/control_report.py 56.00% <25.00%> (ø)
... and 1 more
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@gfieni gfieni merged commit 7ebfa2e into master May 21, 2025
11 checks passed
@gfieni gfieni deleted the refactor/cleanup-unused-report-attrs branch May 21, 2025 15:05
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