Reduce number of plasma profile elements from 501 to 101

[timothy-nunn]

No description provided.

[codecov-commenter]

Codecov Report

❌ Patch coverage is 50.00000% with 1 line in your changes missing coverage. Please review.
✅ Project coverage is 48.64%. Comparing base (84679f5) to head (e703700).
⚠️ Report is 1 commits behind head on main.

Files with missing lines	Patch %	Lines
process/core/io/plot/summary.py	0.00%	1 Missing ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #4416      +/-   ##
==========================================
- Coverage   48.64%   48.64%   -0.01%     
==========================================
  Files         151      151              
  Lines       29703    29704       +1     
==========================================
  Hits        14449    14449              
- Misses      15254    15255       +1     

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

@timothy-nunn May as well change the value of n_plasma_profile_elements down to 100 here also

[timothy-nunn]

@timothy-nunn May as well change the value of n_plasma_profile_elements down to 100 here also

I would probably opt to do that in another PR since it will cause massive regression failures

[je-cook]

I dont see the need for this option being exposed on the main process cli, especially as the MFILE is explicitly the machine readable version. I prefer the option of reducing the number of points recorded. This is surely only talking about 121kB (31entries*500points*8bytes) / 1024 to 12kB assuming 7 char per entry?

[mkovari]

Hi Tim, Can you confirm that the plots haven't changed significantly?

[chris-ashe]

Having inspected the changes on unit tests we are getting a difference on the line averaged values only at the 7th significant figure which is completely fine. The other profile values are all analytically calculated.

[je-cook]

Ok, just the unit test failure needs fixing then

[timothy-nunn]

We opted to go with 201 elements not 101. We observed that the fusion power changed 1.3% with 101 elements for the large tokamak eval test. With 201 elements this different halved to 0.52%. Me and @chris-ashe were happier with this smaller change and so opted to have slightly more profile elements.

[je-cook]

I'm happy