Skip to content

Conversation

@Liameloy
Copy link
Member

Adjusted the logic which determines an intersection between tetrahedron and plane.
Added new test case for specific data which caused issues beforehand

@Liameloy
Copy link
Member Author

@pjaap Please have a look. Thanks!

@Liameloy Liameloy marked this pull request as draft May 22, 2025 09:25
@Liameloy
Copy link
Member Author

Something strange is going on with the documentation. I am trying to figure it out on my machine. Until then...

@Liameloy Liameloy marked this pull request as ready for review May 22, 2025 12:43
@pjaap
Copy link
Member

pjaap commented May 22, 2025

Thank you. I am happy with these changes.

Can you

  • sqash into one commit
  • add a CHANGES entry
  • rebase to current master

@Liameloy Liameloy force-pushed the dev/tetrahedronSliceFixing branch from 368f2d5 to 700714d Compare May 22, 2025 14:53
Liameloy added 2 commits May 22, 2025 17:11
…on and plane

Added new test case for specific data which caused issues beforehand
@Liameloy Liameloy force-pushed the dev/tetrahedronSliceFixing branch from 700714d to 1d190ac Compare May 22, 2025 15:15
@pjaap
Copy link
Member

pjaap commented May 22, 2025

@j-fu What do you think? @Liameloy has fixed an ugly corner case that appeared in a slice plot example.

I like the renameing tet_x_plane!() to calculate_plane_tetrahedron_intersection!().

Do you think this is an interface change? The function is not exported.

@pjaap pjaap merged commit ae54513 into WIAS-PDELib:main May 27, 2025
11 checks passed
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