Skip to content

Rename to_start -> set_start for state management - #110

Merged
chemiskyy merged 2 commits into
masterfrom
wip/start_set_naming
Sep 11, 2026
Merged

Rename to_start -> set_start for state management#110
chemiskyy merged 2 commits into
masterfrom
wip/start_set_naming

Conversation

@chemiskyy

Copy link
Copy Markdown
Member

Rename internal state-management API from to_start to set_start across the UMAT modular code to unify naming and clarify semantics (store current values as start-of-increment / accept increment). Updated declarations/definitions in InternalVariable, InternalVariableCollection and ModularUMAT, adjusted callers (e.g. strain_mechanism) and corresponding tests. Also improved doc comments in phase/state variable headers to distinguish rollback (to_start) vs acceptance (set_start) behavior. No functional logic changes beyond the API rename and documentation/test updates.

Rename internal state-management API from to_start to set_start across the UMAT modular code to unify naming and clarify semantics (store current values as start-of-increment / accept increment). Updated declarations/definitions in InternalVariable, InternalVariableCollection and ModularUMAT, adjusted callers (e.g. strain_mechanism) and corresponding tests. Also improved doc comments in phase/state variable headers to distinguish rollback (to_start) vs acceptance (set_start) behavior. No functional logic changes beyond the API rename and documentation/test updates.

@chemiskyy chemiskyy left a comment

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

LGTM

@chemiskyy
chemiskyy merged commit 6056869 into master Sep 11, 2026
7 checks passed
@chemiskyy
chemiskyy deleted the wip/start_set_naming branch September 11, 2026 06:46
@github-project-automation github-project-automation Bot moved this from Backlog to Done in simcoon 2.0 Sep 11, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

1 participant