Fwbs dataclass

[clmould]

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.

[codecov-commenter]

Codecov Report

❌ Patch coverage is 69.58525% with 264 lines in your changes missing coverage. Please review.
✅ Project coverage is 51.49%. Comparing base (49eb4d1) to head (1691df1).
⚠️ Report is 1 commits behind head on main.

Files with missing lines	Patch %	Lines
process/models/stellarator/stellarator.py	0.00%	97 Missing ⚠️
process/models/blankets/blanket_library.py	33.06%	83 Missing ⚠️
process/models/blankets/hcpb.py	64.15%	19 Missing ⚠️
process/models/power.py	7.69%	12 Missing ⚠️
process/models/blankets/dcll.py	76.31%	9 Missing ⚠️
process/core/scan.py	0.00%	8 Missing ⚠️
process/models/availability.py	83.33%	5 Missing ⚠️
process/models/costs/costs.py	28.57%	5 Missing ⚠️
process/models/costs/costs_2015.py	54.54%	5 Missing ⚠️
process/core/caller.py	0.00%	3 Missing ⚠️
... and 14 more

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #4230      +/-   ##
==========================================
- Coverage   51.81%   51.49%   -0.32%     
==========================================
  Files         150      150              
  Lines       30296    30101     -195     
==========================================
- Hits        15697    15501     -196     
- Misses      14599    14600       +1     

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

I haven't quite checked everything but here are my comments on the main logic changes

[timothy-nunn]

I'm a little confused why this file has been created?

[clmould]

There was a circular import between the first wall and blanket, and this removed that. Also these functions aren't specific to the blanket and are used by other modules so made sense to remove them from blanket_library

[clmould]

I'll move this to another PR

[clmould]

Since this is the first PR to required iteration_variables to be changes, I tested out using some fwbs iteration variables in large_tokamak_IN.DAT:

I then ran process on this file on this branch and also on main:

The only difference was in runtime.