Skip to content

Fall back to 1mm outboard SOL power decay in plotting - #4621

Open
timothy-nunn wants to merge 1 commit into
mainfrom
4620-bug---full-output-errors-and-damaged-pdf
Open

timothy-nunn wants to merge 1 commit into
mainfrom
4620-bug---full-output-errors-and-damaged-pdf

Conversation

@timothy-nunn

Copy link
Copy Markdown
Collaborator

Checks for invalid values of len_sol_outboard_power_decay when plotting to avoid 0/non-finite plot axis lengths. There was a pre-existing comment to suggest this should have been happening but had been removed.

Looking at the code which produces this variable, non-finite outputs can occur if the power or magnetic field is negative.

return (
1.35e-3
* p_plasma_separatrix_mw**-0.02
* rmajor**0.04
* b_plasma_surface_poloidal_average**-0.92
* aspect**-0.42
)

@timothy-nunn timothy-nunn linked an issue Sep 29, 2026 that may be closed by this pull request
@codecov-commenter

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 0% with 2 lines in your changes missing coverage. Please review.
✅ Project coverage is 49.96%. Comparing base (2ae0ef2) to head (21483cb).

Files with missing lines Patch % Lines
process/core/io/plot/summary.py 0.00% 2 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #4621      +/-   ##
==========================================
- Coverage   49.97%   49.96%   -0.01%     
==========================================
  Files         151      151              
  Lines       29929    29931       +2     
==========================================
  Hits        14956    14956              
- Misses      14973    14975       +2     

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

@timothy-nunn
timothy-nunn marked this pull request as ready for review September 29, 2026 09:12
@timothy-nunn
timothy-nunn requested a review from a team as a code owner September 29, 2026 09:12
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.

🐛 --full-output errors and damaged PDF

2 participants