Skip to content

Use constraint diagnostic COCOS consistent with EFIT - #435

Open
torrinba wants to merge 1 commit into
masterfrom
fix_equilibrium_diagnostic_cocos
Open

Use constraint diagnostic COCOS consistent with EFIT#435
torrinba wants to merge 1 commit into
masterfrom
fix_equilibrium_diagnostic_cocos

Conversation

@torrinba

@torrinba torrinba commented Aug 13, 2026

Copy link
Copy Markdown
Collaborator

Description

EFIT outputs the inferred flux variables (psi, psi_axis, etc) with a different convention than the diagnostic constraints (e.g. flux_loops) so the input COCOS for these need to be different. The constraints are now consistent with the input COCOS of the magnetics IDS.

Type of Change

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • Documentation update
  • Code refactoring
  • Other (please describe):

Testing

imas_composer

Pre-Merge Checklist

  • OMFIT has been tested with this OMAS version and you will create a PR on OMFIT that updates the OMAS submodule once this has been merged.
  • Version number has been incremented if this is a major modification or important bug fix
    • To increment the version: Edit the version number in omas/version file
    • Follow semantic versioning: MAJOR.MINOR.PATCH (e.g., 0.94.20.94.3 for bug fixes, 0.94.20.95.0 for new features)
    • A GitHub release will be automatically created when this PR is merged if the version number has changed

EFIT outputs the inferred flux variables (psi, psi_axis, etc) with
a different convention than the diagnostic constraints (e.g. flux_loops)
so the input COCOS for these need to be different. The constraints
are now consistent with the input COCOS of the magnetics IDS.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant