Update superconducting tests
@kj5248 is already working on this.
Since Jun 3, 2026.
- Dominant language
- Python
- Stars
- 71
- Forks
- 27
- Avg merge
- 2d 18h
- Merged PRs (30d)
- 33
Description
From #3823 we have found that the superconducting tests are not comprehensive. Only i_tf_superconductor = 5 is tested. There has also been a lot of duplication found, eg in #3823 we found that there was duplication in the paramterised SuperconParam instances used in the test for supercon.
Also found this error
in superconducting_tf_coil.py where an argument was typed wrong, but since it wasn't being tested (this is called only in i_tf_superconductor = 3) this wasn't picked up on the CI.
The tests need to be redone so that all instances of i_tf_superconductor are being tested and to ensure all the parameterised variables are being used.
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Assessment
This issue has not been assessed yet.