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

Fixes #126 - STATEINTENT_INTERNAL #128

Conversation

tclune
Copy link
Collaborator

@tclune tclune commented Feb 18, 2023

This change introduces a new constant ESMF_STATEINTENT_INTERNAL of type ESMF_StateIntent_Flag.

  • Does not attempt to extend/modify built-in tests to cover the (minimal) new functionality.

  • Note that the internal ids for the UNSPECIFIED/INVALID cases are changed from 3/4 to 4/5 respectievly. This could in theory break user code that incorrectly relies on those values.

  • Fixed minor typo (missing period) in documentation.

This change introduces a new constant `ESMF_STATEINTENT_INTERNAL` of
type `ESMF_StateIntent_Flag`.

- Does not attempt to extend/modify built-in tests to cover
  the (minimal) new functionality.

- Note that the internal ids for the UNSPECIFIED/INVALID cases are
  changed from 3/4 to 4/5 respectievly.  This could in theory break
  user code that incorrectly relies on those values.

- Fixed minor typo (missing period) in documentation.
@theurich theurich self-assigned this Feb 22, 2023
@theurich theurich self-requested a review February 22, 2023 20:18
Copy link
Member

@theurich theurich left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Besides the comment in code that needs to be addressed, there also should be an update to file /src/Superstructure/State/doc/State_options.tex to add the new state intent.

src/Superstructure/State/src/ESMF_StateAPI.cppF90 Outdated Show resolved Hide resolved
 - Also added documentation for the new option.
@tclune
Copy link
Collaborator Author

tclune commented Feb 28, 2023

ok - I've verified the latest patch builds on laptop

@theurich theurich changed the base branch from develop to feature/state-intent-internal May 15, 2023 23:17
@theurich theurich merged commit 107dc0b into esmf-org:feature/state-intent-internal May 15, 2023
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