Reduce duplication of geometry dataclasses in `tfcoil/superconducting.py`

[timothy-nunn]

Not a comment for you Jack, I think this class can inherit from CICCAveragedTurnGeometry which will reduce duplication. Will make an issue.

Originally posted by @timothy-nunn in #4591 (comment)

[ChangeinX]

I’d be happy to take this on. I propose making CICCIntegerTurnGeometry inherit from CICCAveragedTurnGeometry and removing the three duplicated fields. The four integer-specific dimensions and slots=True would remain.

This changes the order of positional constructor arguments. I found that the repository constructs these objects using keyword arguments and accesses their fields by name, so its existing callers should be unaffected.

To check the approach, I temporarily substituted the proposed class definition during a local test run without editing the repository files. The TF-coil test suite completed with 93 passed and 3 skipped under its existing platform checks.

I’ll extend the integer-geometry tests to check effective cable-space area and cooling fraction. I can wait for #4591 to merge before preparing the PR so its field descriptions are preserved.

If that scope works for you, please assign the issue to me.

[timothy-nunn]

@ChangeinX that would be much appreciated, I look forward to the PR. Thanks

[ChangeinX]

Thanks for assigning this. I’ve opened #4612 with the agreed refactor and test coverage. I’ve noted the overlap with #4591 and will rebase if it merges first.