Skip to content

Conversation

Joshua-Kloepfer
Copy link
Collaborator

@Joshua-Kloepfer Joshua-Kloepfer commented Aug 7, 2025

Added test for tetrahedron with multiple components and moved current tetrahedron test to the point mapping test.

Copy link
Contributor

@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 overall. Would you please add a PR description that indicates what the tests are doing?

Once the tests are fixed this should be in good shape.

@cwsmith
Copy link
Contributor

cwsmith commented Aug 8, 2025

/runtests

Copy link

github-actions bot commented Aug 8, 2025

Test Result: success (details)

@cwsmith
Copy link
Contributor

cwsmith commented Aug 8, 2025

@Joshua-Kloepfer Would you please merge main to this branch so the github hosted CI tests pass?

Copy link
Contributor

@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.

A couple minor comments are below.

@cwsmith cwsmith merged commit 164bd2b into main Aug 14, 2025
6 checks passed
@cwsmith cwsmith deleted the jk/tetTests branch August 14, 2025 12:57
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.

2 participants