ukaea / ukaea/PROCESS

parabolic parametrisation warning into info?

Open
#3,868 1 comment 0 reactions 1 assignee View on GitHub

@grmtrkngtn is already working on this.

Since Sep 23, 2025.

Dominant language
Python
Stars
71
Forks
27
Avg merge
2d 18h
Merged PRs (30d)
33

Description

I am working on stellarator optimization in PROCESS and I have a small suggestion to change the warning in parabolic_paramterisation of the plasma_profiles.py into info.
How it works at the moment: when you use plasma profile with no pedestal (ipedestal=0), there is a warning unless you set up the pedestal specification to a specific set of values. Usually I don't set up these values in the input, because I do not have a pedestal. In fact there is no way to use any other values, as they are overwritten if you set them to anything else (they are compared against hardcoded set of parameters). These parameters are not even used in the profile calculations if you don't have a pedestal, so there is absolutely no reason to set them up.
Still, PROCESS will give you a warning if you don't do it, despite the fact that this is fully expected, as you are just using the default set of parameters and everything is going as it is supposed to be. For user it is confusing and I wasted my time looking into the source code, to make sure that everything is really correct.
In fact the way parabolic_paramterisation function is written can be changed, to skip the if statement and just overwrite the parameters every time and give logger info.

Contributor guide

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.