Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add Missing UDA Dimension for GCONPROD GRAT Limit #4248

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

bska
Copy link
Member

@bska bska commented Oct 3, 2024

Appears to have been simply forgotten when this function was first added in commit d8d6749 (PR #2620). Without this we fail loading a restarted simulation run with UDAs for GCONPROD's GRAT limit and issue a diagnostic of the form

Internal error: No dimension for UDA control 'GCONPROD_GRAT'

@bska
Copy link
Member Author

bska commented Oct 3, 2024

jenkins build this please

@akva2
Copy link
Member

akva2 commented Oct 3, 2024

I assume this get some test coverage in upcoming work?

@bska bska force-pushed the fix-missing-rst-gconprod-uda-grat-dim branch from 6b2f702 to 0f133c8 Compare October 3, 2024 17:18
@bska
Copy link
Member Author

bska commented Oct 3, 2024

I assume this get some test coverage in upcoming work?

Kind of. I've pushed an update to the unit tests for UnitSystem now.

@bska
Copy link
Member Author

bska commented Oct 3, 2024

jenkins build this please

Appears to have been simply forgotten when this function was first
added in commit d8d6749 (PR OPM#2620).  Without this we fail loading
a restarted simulation run with UDAs for GCONPROD's GRAT limit and
issue a diagnostic of the form

  Internal error: No dimension for UDA control 'GCONPROD_GRAT'
@bska bska force-pushed the fix-missing-rst-gconprod-uda-grat-dim branch from 0f133c8 to dcffab5 Compare October 4, 2024 15:55
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