Skip to content

Rework summary.py into a module - #4632

Draft
je-cook wants to merge 18 commits into
ukaea:mainfrom
je-cook:make_summary_plot_module
Draft

je-cook wants to merge 18 commits into
ukaea:mainfrom
je-cook:make_summary_plot_module

Conversation

@je-cook

@je-cook je-cook commented Sep 30, 2026

Copy link
Copy Markdown
Collaborator

Description

Checklist

I confirm that I have completed the following checks:

  • My changes follow the PROCESS style guide
  • I have justified any large differences in the regression tests caused by this pull request in the comments.
  • I have added new tests where appropriate for the changes I have made.
  • If I have had to change any existing unit or integration tests, I have justified this change in the pull request comments.
  • If I have made documentation changes, I have checked they render correctly.
  • I have added documentation for my change, if appropriate.

@je-cook
je-cook force-pushed the make_summary_plot_module branch from bd02a17 to 6885d66 Compare September 30, 2026 15:30
@je-cook
je-cook force-pushed the make_summary_plot_module branch from 6885d66 to c6c10eb Compare September 30, 2026 15:36
@codecov-commenter

codecov-commenter commented Sep 30, 2026 •

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 12.96639% with 3289 lines in your changes missing coverage. Please review.
✅ Project coverage is 50.67%. Comparing base (4885172) to head (04f3ff0).
⚠️ Report is 1 commits behind head on main.

Files with missing lines Patch % Lines
process/core/io/plot/summary/profiles/plasma.py 6.15% 366 Missing ⚠️
process/core/io/plot/summary/api.py 13.11% 265 Missing ⚠️
process/core/io/plot/summary/plasma/physics.py 7.72% 251 Missing ⚠️
process/core/io/plot/summary/geometry/poloidal.py 11.71% 211 Missing ⚠️
process/core/io/plot/summary/profiles/misc.py 7.92% 186 Missing ⚠️
process/core/io/plot/summary/power_flow.py 4.73% 181 Missing ⚠️
process/core/io/plot/summary/reporting/misc.py 13.29% 163 Missing ⚠️
process/core/io/plot/summary/reporting/panels.py 10.05% 161 Missing ⚠️
process/core/io/plot/summary/profiles/stress.py 8.13% 158 Missing ⚠️
process/core/io/plot/summary/magnets/cs.py 10.90% 147 Missing ⚠️
... and 18 more
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #4632      +/-   ##
==========================================
+ Coverage   49.92%   50.67%   +0.75%     
==========================================
  Files         151      180      +29     
  Lines       30236    30346     +110     
==========================================
+ Hits        15094    15377     +283     
+ Misses      15142    14969     -173     

☔ View full report in Codecov by Harness.
📢 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.

@je-cook
je-cook force-pushed the make_summary_plot_module branch 2 times, most recently from d2aae4b to b5799d4 Compare October 1, 2026 14:38
@je-cook
je-cook force-pushed the make_summary_plot_module branch from 3c334a3 to d5e735c Compare October 2, 2026 08:06
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.

2 participants