Plasma geometry option `igeom`

[jonmaddock]

In GitLab by @jmorris-uk on Apr 8, 2020, 16:54

@skahn @stuartmuldrew @mkovari

There is a plasma geometry option switch igeom. In the documentation it only mentions that one is an older calculation and the other newer. One of the comments in global variables seems to suggest that igeom = 0 is for STs. The default is igeom = 1 and it appears none of the tracked input files use it.

Would anyone like to volunteer to investigate the two choices and determine if one is indeed ST relevant. If not, do we need both?

The code above is in plasma_geometry.f90

[jonmaddock]

In GitLab by @stuartmuldrew on Apr 8, 2020, 16:58

I have been using igeom=0 for FNSF and STEP based on the the ST comment. I'm happy to take a look.

[jonmaddock]

In GitLab by @wg7580 on Jul 12, 2022, 11:20

@stuartmuldrew Is this still something that needs looking at Stewart?

[jonmaddock]

In GitLab by @stuartmuldrew on Jul 12, 2022, 11:49

This has been on my to-do list for two years apparently. I forgot about it so will reassign it. It should remain open.

[jonmaddock]

In GitLab by @j-a-foster on Jul 29, 2022, 11:16

Ran igeom = 0 and igeom = 1 for both baseline_2019 and an SPR and found almost no difference between them, for either.

comp_baseline_2019.txt
comp_SPR.txt

Will remove STAR code.

[jonmaddock]

In GitLab by @j-a-foster on Jul 29, 2022, 11:16

created merge request !788 to address this issue

[jonmaddock]

In GitLab by @stuartmuldrew on Mar 22, 2023, 11:34

closed via merge request !788

[jonmaddock]

In GitLab by @stuartmuldrew on Mar 22, 2023, 11:34

mentioned in commit 25074c652f60d8d6a7f7a558beb6c79125201b17

[jonmaddock]

In GitLab by @mkovari on Apr 5, 2023, 16:19

igeom no longer appears in the code (although it does appear in a number of json files for some reason).

[jonmaddock]

In GitLab by @ajpearcey on Apr 11, 2023, 16:02

The igeom calculations where looked at in the past see here