Update how the various magnetic fields in pf_coil.py are handled #3633
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description
Replaced various magnetic field methods: the CS self field (bfmax), the general PF coil self field, all coil field contributions to other coils.
CS check:
Reference Bluemira plot for k, B0 and Bmax for a current of 1.0. Here B0 is the central point of the CS coil ie r->0 whilst Bmax is at the inner edge of the CS coil.
The changes to the code result in the following plot for PROCESS (same params as Bluemira with current at 1.0).
The original result for PROCESS before the changes are:
These results all seem similar to each other but there are some differences. Firstly, the old process method has curves that differ compared to the other 2, most noticeable in the k and Bm graphs. There is a slight difference from Bluemira to the PROCESS new method even though they should be near identical. In terms of improvement it is most beneficial in the regions of higher alpha or beta.
Checklist
I confirm that I have completed the following checks: