Skip to content

Add test to demonstrate simplex integral fails with overintegration#364

Open
MTCam wants to merge 5 commits intoinducer:mainfrom
illinois-ceesd:simplex-overintegration-issue
Open

Add test to demonstrate simplex integral fails with overintegration#364
MTCam wants to merge 5 commits intoinducer:mainfrom
illinois-ceesd:simplex-overintegration-issue

Conversation

@MTCam
Copy link
Collaborator

@MTCam MTCam commented Sep 25, 2024

This fixes a hang-up with previous implementation of the integral and elementewise_integral utils when using simplices with overintegration, and adds TPEs to the suite of tested mesh cases in the surface area test.

  • Adds TPE mesh to tested cases in test_mass_surface_area
  • Switches integral and elementwise_integral to use quadrature weights directly instead of _apply_mass_matrix to get the weights

Fixes: #365

@MTCam MTCam marked this pull request as ready for review September 27, 2024 17:51
@MTCam MTCam requested a review from inducer September 27, 2024 17:51
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.

integrals do not work for simplices when overintegrating

1 participant