Skip to content

Conversation

@kj5248
Copy link
Collaborator

@kj5248 kj5248 commented Apr 16, 2025

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.

bluemira_cs

The changes to the code result in the following plot for PROCESS (same params as Bluemira with current at 1.0).

process_cs_new

The original result for PROCESS before the changes are:

process_cs_old

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:

  • 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.

@kj5248 kj5248 linked an issue Apr 16, 2025 that may be closed by this pull request
4 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Imprecise calculation of bfmaxoh

2 participants