Skip to content

[WIP] Enable split meshes for Newton-based FV solvers #30379

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

Draft
wants to merge 7 commits into
base: next
Choose a base branch
from

Conversation

grmnptr
Copy link
Contributor

@grmnptr grmnptr commented Apr 23, 2025

Refs #25621

* for advection-related kernels.
* @param obj_params The input parameters of the object
* @param rm_params The input parameters of the relationship manager
* @param conditional_extended_layers Number of layers the will get assigned if the conditions in the function are met
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
* @param conditional_extended_layers Number of layers the will get assigned if the conditions in the function are met
* @param conditional_extended_layers Number of layers that will get assigned if the conditions in the function are met

* for diffusion-related kernels.
* @param obj_params The input parameters of the object
* @param rm_params The input parameters of the relationship manager
* @param conditional_extended_layers Number of layers the will get assigned if the conditions in the function are met
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
* @param conditional_extended_layers Number of layers the will get assigned if the conditions in the function are met
* @param conditional_extended_layers Number of layers that will get assigned if the conditions in the function are met

@lindsayad
Copy link
Member

@grmnptr just mention me when you bring this out of draft

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Development

Successfully merging this pull request may close these issues.

2 participants