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

Add Mesh2D #87

Merged
merged 8 commits into from
Feb 16, 2024
Merged

Add Mesh2D #87

merged 8 commits into from
Feb 16, 2024

Conversation

matthew-mccall
Copy link

This pull request adds Mesh2D a child class of Mesh that only allows 2D geometry. It also adds a test case for attempting to load higher dimension geometry into a Mesh2D which is expected to fail. To facilitate this, test_func_impl was updated to run tests in a modified environment such that system-level errors can be caught. (i.e a failed assert).

Added osh2vtk2d that fails if mesh is not 2D
- Renamed osh2vtk2d to load_2d
Test originally failed because it attempted to load an invalid mesh due to a change in how arguments are handled for tests that are expected to fail. Now the test fails because we are attempting to load a 3d mesh into a 2d mesh
@cwsmith cwsmith changed the base branch from master to develop February 16, 2024 14:15
@cwsmith cwsmith changed the base branch from develop to master February 16, 2024 14:15
Copy link

@cwsmith cwsmith left a comment

Choose a reason for hiding this comment

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

Looks good. Thank you. A few comments are below.

src/CMakeLists.txt Outdated Show resolved Hide resolved
src/Omega_h_mesh2d.hpp Show resolved Hide resolved
src/CMakeLists.txt Outdated Show resolved Hide resolved
matthew-mccall and others added 2 commits February 16, 2024 14:27
Co-authored-by: Cameron Smith <cwsmith@users.noreply.github.com>
@cwsmith cwsmith merged commit c101076 into SCOREC:master Feb 16, 2024
14 checks passed
@matthew-mccall matthew-mccall deleted the mmccall/2dmesh branch February 16, 2024 20:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants