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

Identify subdomains tool. #2227

Merged
merged 6 commits into from
Oct 2, 2018
Merged

Identify subdomains tool. #2227

merged 6 commits into from
Oct 2, 2018

Conversation

endJunction
Copy link
Member

@endJunction endJunction commented Sep 28, 2018

Creates/updates/verifies bulk_node_ids and bulk_element_ids in subdomain meshes.
Test and docu included.

return element_ids;
}

/// Find one-to-one mapping of subdomain nodes to the bulk mesh nodes and
Copy link
Member

Choose a reason for hiding this comment

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

Comment seems a copy from above, nodes -> elements?

Copy link
Member

@TomFischer TomFischer left a comment

Choose a reason for hiding this comment

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

Maybe a user documentation like here for the tool can be created ...

Copy link
Member

@wenqing wenqing left a comment

Choose a reason for hiding this comment

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

👍

Algorithms for searching the nodes and elements of a subdomain mesh
in a bulk mesh. The bulk_node_ids and bulk_element_ids
mesh properties are added to the subdomain mesh or checked for
consistency.
For checking and setting of the node and element correspondences
between subdomain and bulk meshes.
@endJunction endJunction merged commit 214afcb into ufz:master Oct 2, 2018
@endJunction endJunction deleted the IdentifySubdomains branch October 2, 2018 19:13
@ogsbot
Copy link
Member

ogsbot commented Jun 19, 2020

OpenGeoSys development has been moved to GitLab.

See this pull request on GitLab.

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.

4 participants