Skip to content

Updating Syntax of Magnetics and Summary ids'#5

Open
LariDG wants to merge 4 commits intomainfrom
updating-existing-mappings
Open

Updating Syntax of Magnetics and Summary ids'#5
LariDG wants to merge 4 commits intomainfrom
updating-existing-mappings

Conversation

@LariDG
Copy link
Copy Markdown
Collaborator

@LariDG LariDG commented Apr 7, 2026

Using the IMAS_MASTU mappings to update the magnetics and summary mappings here.

@stephen-dixon
Copy link
Copy Markdown
Collaborator

stephen-dixon commented Apr 7, 2026

Probably shouldn't be a blocker now but for info for the summary ids.

One of the rules here is that summary should present reduced datasets compared to other ones. Concretely that there should be a common timebase with a small number of points (100-300 I think) and all signal data in summary needs to be interpolated onto that common timebase. What we have now is just the raw signals so the imas-python package will refuse to write to ids with strict checking as it breaks the homogeneous_time rule.

The reason this shouldn't be blocking now is that the interpolation functionality isn't available in libtokamap yet so there's practically not much you can do except maybe tag the relevant ones with a comment that interpolation transform will need to be added later. guess it's a bit like the geometry ones where you've got the skeleton in place even though the backend functionality isn't there yet.

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