♻️ Standardise more blkt code

[chris-ashe]

This pull request introduces improvements to the blanket structure output and plotting, as well as refactors unit tests for blanket half-height calculation. The main enhancements include a new plotting function for the blanket structure that displays both areas and volumes, streamlining of output methods, and simplification of the unit test for blanket half-height by removing unnecessary patching and parameters.

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 46.25000% with 43 lines in your changes missing coverage. Please review.
✅ Project coverage is 47.51%. Comparing base (891e46b) to head (126754b).
⚠️ Report is 4 commits behind head on main.

Files with missing lines	Patch %	Lines
process/core/io/plot_proc.py	4.34%	22 Missing ⚠️
process/models/blankets/blanket_library.py	65.45%	19 Missing ⚠️
process/models/blankets/dcll.py	0.00%	1 Missing ⚠️
process/models/blankets/hcpb.py	0.00%	1 Missing ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #4075      +/-   ##
==========================================
+ Coverage   47.26%   47.51%   +0.24%     
==========================================
  Files         139      141       +2     
  Lines       29420    29534     +114     
==========================================
+ Hits        13906    14032     +126     
+ Misses      15514    15502      -12     

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

Just one question, then happy

[timothy-nunn]

I have verified that none of the existing variables that were being assigned have changed in this branch:

26853a26854,26866
> # Blanket Volumes and Surface Areas #
> Inboard_Blanket_Volume_with_gaps_and_holes_(m3)__________________________ (vol_blkt_inboard)_____________ 3.15923926205893508e+02 
> Outboard_Blanket_Volume_with_gaps_and_holes_(m3)_________________________ (vol_blkt_outboard)____________ 8.68976186101138296e+02 
> Total_Blanket_Volume_with_gaps_and_holes_(m3)____________________________ (vol_blkt_total)_______________ 1.18490011230703180e+03 
> Inboard_Blanket_Volume_if_toridally_continuous_(m3)______________________ (vol_blkt_inboard_full_coverage)_ 3.15923926205893508e+02 
> Outboard_Blanket_Volume_if_toridally_continuous_(m3)_____________________ (vol_blkt_outboard_full_coverage)_ 1.02294625719188252e+03 
> Total_Blanket_Volume_if_toridally_continuous_(m3)________________________ (vol_blkt_total_full_coverage)_ 1.33887018339777615e+03 
> Inboard_Blanket_Surface_Area_with_gaps_and_holes_(m2)____________________ (a_blkt_inboard_surface)_______ 6.63622172160947002e+02 
> Outboard_Blanket_Surface_Area_with_gaps_and_holes_(m2)___________________ (a_blkt_outboard_surface)______ 8.99584666520948076e+02 
> Total_Blanket_Surface_Area_with_gaps_and_holes_(m2)______________________ (a_blkt_total_surface)_________ 1.56320683868189508e+03 
> Inboard_blanket_surface_area_if_toridally_continuous_(m2)________________ (a_blkt_inboard_surface_full_coverage)_ 6.63622172160947002e+02 
> Outboard_blanket_surface_area_if_toridally_continuous_(m2)_______________ (a_blkt_outboard_surface_full_coverage)_ 1.10271323877904751e+03 
> Total_blanket_surface_area_if_toridally_continuous_(m2)__________________ (a_blkt_total_surface_full_coverage)_ 1.76633541093999452e+03 
26897,26909d26909
< # Blanket Volumes and Surface Areas #
< Inboard_Blanket_Volume_with_gaps_and_holes_(m3)__________________________ (vol_blkt_inboard)_____________ 3.15923926205893508e+02 
< Outboard_Blanket_Volume_with_gaps_and_holes_(m3)_________________________ (vol_blkt_outboard)____________ 8.68976186101138296e+02 
< Total_Blanket_Volume_with_gaps_and_holes_(m3)____________________________ (vol_blkt_total)_______________ 1.18490011230703180e+03 
< Inboard_Blanket_Volume_if_toridally_continuous_(m3)______________________ (vol_blkt_inboard_full_coverage)_ 0.00000000000000000e+00 
< Outboard_Blanket_Volume_if_toridally_continuous_(m3)_____________________ (vol_blkt_outboard_full_coverage)_ 0.00000000000000000e+00 
< Total_Blanket_Volume_if_toridally_continuous_(m3)________________________ (vol_blkt_total_full_coverage)_ 0.00000000000000000e+00 
< Inboard_Blanket_Surface_Area_with_gaps_and_holes_(m2)____________________ (a_blkt_inboard_surface)_______ 6.63622172160947002e+02 
< Outboard_Blanket_Surface_Area_with_gaps_and_holes_(m2)___________________ (a_blkt_outboard_surface)______ 8.99584666520948076e+02 
< Total_Blanket_Surface_Area_with_gaps_and_holes_(m2)______________________ (a_blkt_total_surface)_________ 1.56320683868189508e+03 
< Inboard_blanket_surface_area_if_toridally_continuous_(m2)________________ (a_blkt_inboard_surface_full_coverage)_ 0.00000000000000000e+00 
< Outboard_blanket_surface_area_if_toridally_continuous_(m2)_______________ (a_blkt_outboard_surface_full_coverage)_ 0.00000000000000000e+00 
< Total_blanket_surface_area_if_toridally_continuous_(m2)__________________ (a_blkt_total_surface_full_coverage)_ 0.00000000000000000e+00 

[timothy-nunn]

Just one question about why the volumes are different for the D-shaped component between the old unit test and the new unit test.

[timothy-nunn]

The old tests had slightly different values, any reason why?
Before:

expected_vol_blkt_outboard=691.06561956756764,
expected_volblkt=691.06561956756764,

[chris-ashe]

This was down to the bad tests before as the inboard blanket was not accounted for in the output. Also because the reduction of the blanket area and volume is now in a separate function, this test is only testing for a full coverage blanket so the values are higher

[timothy-nunn]

These volumes are the same as before