Skip to content

New toml file for input - #10

Open
avadean wants to merge 9 commits into
sjd549:developfrom
PlasmaFAIR:input
Open

New toml file for input#10
avadean wants to merge 9 commits into
sjd549:developfrom
PlasmaFAIR:input

Conversation

@avadean

@avadean avadean commented Mar 31, 2026

Copy link
Copy Markdown
Contributor

Now uses toml to configure HELENA switchboard inputs. The default input toml file is "input.toml", but can be supplied as the first argument when running helena some_input.toml for example.

Check tests/input_files/input_tecplot2D.toml for every possible input written out, and the current defaults.

@sjd549 if you would like any changes to any toml headings or categories etc. then just let me know - super easy to change of course. Defaults also can be updated easily.

The code for all of this has been put in the big run command for now - but once I have made some modulisation changes it can all be moved to its own file.

Marked as Draft Pull Request for now until we are happy with toml categories and defaults.

avadean added 6 commits March 26, 2026 17:21
… range of toml file sections. Temporarily set current switchboard values to those provided by toml file (in future will just user toml file values directly). Add expansion capability of variable sets in (IEDF/NEDF/Phase/multi)variables. Add expert options ignore_div_zero and ignore_empty_contours.
…some standard defaults with the new toml input file capabilities.
# Conflicts:
#	src/helena/helena.py
@LiamPattinson

Copy link
Copy Markdown

So far it's looking good to me! You might be able to cut down on a lot of the data validation by making use of Pydantic. It looks like they have their own version of dataclass that does this automatically based on type hints.

@sjd549
sjd549 marked this pull request as ready for review April 29, 2026 11:17
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.

2 participants