Skip to content

Fixed calculation for intersections #25

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

Merged
merged 2 commits into from
May 27, 2025

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