Skip to content

♻️ Refactor Bruner div heat split - #4511

Draft
chris-ashe wants to merge 25 commits into
mainfrom
add_bruner_div_heat_split
Draft

♻️ Refactor Bruner div heat split#4511
chris-ashe wants to merge 25 commits into
mainfrom
add_bruner_div_heat_split

Conversation

@chris-ashe

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.

@chris-ashe chris-ashe added Physics Relating to the physics models Divertor Scrape off Layer Relating to the scrape off layer outside the plasma separatrix labels Aug 10, 2026
@codecov-commenter

codecov-commenter commented Aug 10, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 33.69565% with 122 lines in your changes missing coverage. Please review.
✅ Project coverage is 49.26%. Comparing base (feb7ef3) to head (95eb74f).

Files with missing lines Patch % Lines
process/core/io/plot/summary.py 4.81% 79 Missing ⚠️
process/models/physics/exhaust.py 55.17% 26 Missing ⚠️
process/models/physics/physics.py 0.00% 16 Missing ⚠️
process/models/build.py 0.00% 1 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #4511      +/-   ##
==========================================
- Coverage   49.32%   49.26%   -0.06%     
==========================================
  Files         150      150              
  Lines       29802    29931     +129     
==========================================
+ Hits        14700    14746      +46     
- Misses      15102    15185      +83     

☔ 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.

@chris-ashe chris-ashe changed the title Add bruner div heat split ♻️ Refactor Bruner div heat split Aug 14, 2026
…date references in plasma calculations; add Brunner's method for divertor power splits calculation
…rSeparatrixPowerSplits dataclass and update related methods
… and distances for double null configuration
… inner divertor power splits and update references
@chris-ashe
chris-ashe force-pushed the add_bruner_div_heat_split branch from e5d6816 to 95eb74f Compare August 14, 2026 12:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Divertor Physics Relating to the physics models Scrape off Layer Relating to the scrape off layer outside the plasma separatrix

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants