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

Improve ElementSubdomainModifier #27899

Closed
wants to merge 9 commits into from
Closed

Conversation

Wendy-Ji
Copy link
Contributor

There have been several bug reports and feature requests since the introduction of ElementSubdomainModifier. This PR fixes most, if not all, of the bugs and introduces a new feature.

Notable changes:

  • meshChanged() is now only called when at least one element changes its subdomain. The old behavior led to an adaptivity file (*-s0*.e) file at every time step.
  • A new input parameter "active_subdomain" asks for active subdomains on which the equation system exists. This enables the detection of newly activated dofs so that variable initialization and material property initialization can be properly performed.
  • A new input parameter "initialize_variables" allows users to choose which variables to initialize. This supports nonlinear and aux variables of any order and family.
  • A new input parameter "initialization_strategy" allows users to choose how to initialize each variable. The supported strategies are
    • IC: initial condition of the variable will be applied.
    • NEAREST: the old values of the nearest dofs (in the spatial sense) will be assigned on newly activated dofs.
    • CONSTANT: a constant value will be assigned on newly activated dofs.
  • A new input parameter "initialization_constant" additionally specifies the constant used with "initialization_strategy = CONSTANT".

close #18765
close #18835
ref #21579

Also references many many discussions...

This is an update of pull request #26254 by @hugary1995

hugary1995 and others added 8 commits June 6, 2024 11:06
There have been several bug reports and feature requests since the introduction of `ElementSubdomainModifier`.
This PR fixes most, if not all, of the bugs and introduces a new feature.

Notable changes:
- `meshChanged()` is now only called when at least one element changes its subdomain. The old behavior led to an adaptivity file (*-s0*.e) file at every time step.
- A new input parameter "active_subdomain" asks for _active_ subdomains on which the equation system exists. This enables the
  detection of newly activated dofs so that variable initialization and material property initialization can be properly performed.
- A new input parameter "initialize_variables" allows users to choose which variables to initialize. This supports nonlinear and aux variables of any order and family.
- A new input parameter "initialization_strategy" allows users to choose *how* to initialize each variable. The supported strategies are
  - IC: initial condition of the variable will be applied.
  - NEAREST: the old values of the nearest dofs (in the spatial sense) will be assigned on newly activated dofs.
  - CONSTANT: a constant value will be assigned on newly activated dofs.
- A new input parameter "initialization_constant" additionally specifies the constant used with "initialization_strategy = CONSTANT".

close idaholab#18765
close idaholab#18835
ref idaholab#21579

Also references many many discussions...
@GiudGiud GiudGiud self-assigned this Jun 16, 2024
Copy link

This pull request has been automatically marked as stale because it has not had recent activity in the last 100 days. It will be closed in 7 days if no further activity occurs. Thank you for your contributions.

@github-actions github-actions bot added the stale PRs that have reached or exceeded 90 days with no activity label Sep 30, 2024
@GiudGiud
Copy link
Contributor

Closing as superseded by #27965

@GiudGiud GiudGiud closed this Sep 30, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
stale PRs that have reached or exceeded 90 days with no activity
Projects
None yet
3 participants