Updated PF numbering in outputs

[kj5248]

Description

closes #4502

In MFILE.DAT and OUT.DAT there were many places where PF coils were numbered from 0. This has been updated to start from 1.

Also in places such as Volt Second Consumption and Waveforms all the circuits were numbers, the final number has now shows CS as that's what it is. However in the PF Circuit Waveform Data section I left the circuit_var_name as the original as it is a var not an output name.

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.

[codecov-commenter]

Codecov Report

❌ Patch coverage is 0% with 65 lines in your changes missing coverage. Please review.
✅ Project coverage is 49.85%. Comparing base (8a6f0af) to head (7e6015a).
⚠️ Report is 4 commits behind head on main.

Files with missing lines	Patch %	Lines
process/models/pfcoil.py	0.00%	49 Missing ⚠️
process/core/io/plot/summary.py	0.00%	16 Missing ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #4519      +/-   ##
==========================================
- Coverage   49.88%   49.85%   -0.03%     
==========================================
  Files         151      151              
  Lines       29871    29886      +15     
==========================================
  Hits        14901    14901              
- Misses      14970    14985      +15     

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

@kj5248 Have noticed that the coil numbering in the plots now starts from 2. Can you quickly update that to and I will take a full look after

[chris-ashe]

All OUT.DAT looks ok, though I am getting a bug for plot_pf_dimensions() when running the ST regression were it does not do the final coil, Coil 8 in this case

[kj5248]

@chris-ashe > All OUT.DAT looks ok, though I am getting a bug for plot_pf_dimensions() when running the ST regression were it does not do the final coil, Coil 8 in this case

When I run the ST regression and use plot summary on it i get this which has 8 PF coils?

Although I did notice somewhere in the OUT.dat for ST which seems wrong but wanted to double check. Under waveforms it drops to 7 coils and a CS, feels like this is wrong?

[chris-ashe]

@chris-ashe > All OUT.DAT looks ok, though I am getting a bug for plot_pf_dimensions() when running the ST regression were it does not do the final coil, Coil 8 in this case

When I run the ST regression and use plot summary on it i get this which has 8 PF coils?

Although I did notice somewhere in the OUT.dat for ST which seems wrong but wanted to double check. Under waveforms it drops to 7 coils and a CS, feels like this is wrong?

Yeah, there must be a dodgy difference of 1 somewhere, the coil number should always be even. Also, if possible can you apply tabulate to the output to make everything line up properly, 2 birds with one stone

[kj5248]

Tabulated the outputs. I also noticed what I am pretty sure is a duplicated output (it is in table form in the waveforms section) so I removed it @chris-ashe:

[chris-ashe]

The PF/CS mutual inductance matrix colour plot in summary.py is still starting from 0. Also the PF coil dimensions plot is not adding the dimensions for the final coil but the numbering is correct