Skip to content

Minor fixes#79

Closed
totork wants to merge 5 commits into
boutproject:masterfrom
totork:master-fix-progress-bar
Closed

Minor fixes#79
totork wants to merge 5 commits into
boutproject:masterfrom
totork:master-fix-progress-bar

Conversation

@totork

@totork totork commented Jun 16, 2026

Copy link
Copy Markdown
Contributor

Here are some changes that came up while running the code

  1. -Importing the TCV X-21 field gave me an error of different dimensions of psi, changing the way psi is handled in the GEQDSK fixed this and enables using such files for the magnetic field
  2. Using By for the area calculation can lead to negative field strengths depending on the orientation and hence negative surface areas. Also to preserve magnetic flux, the magnitude should be used I think?
  3. Add dapg to be run by default, otherwise grids miss this and the start of the simulation throws an error.
  4. Added chunking in X direction

@totork totork closed this Jun 16, 2026
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.

1 participant