Skip to content

Refinement capability for Prism21, fixes for Tri7, test coverage#3445

Merged
roystgnr merged 9 commits intolibMesh:develfrom
roystgnr:embedding_updates
Nov 23, 2022
Merged

Refinement capability for Prism21, fixes for Tri7, test coverage#3445
roystgnr merged 9 commits intolibMesh:develfrom
roystgnr:embedding_updates

Conversation

@roystgnr
Copy link
Copy Markdown
Member

No description provided.

@moosebuild
Copy link
Copy Markdown

moosebuild commented Nov 22, 2022

Job Coverage on 2500494 wanted to post the following:

Coverage

16f6de #3445 250049
Total Total +/- New
Rate 59.74% 59.84% +0.10% 66.67%
Hits 48360 48448 +88 8
Misses 32588 32508 -80 4

Diff coverage report

Full coverage report

Warnings

  • New new line coverage rate 66.67% is less than the suggested 90.0%

This comment will be updated on new commits.

At some point between Tet14 and Pyramid21 we exceeded my capacity to
correct these things one entry at a time.
I need to come up with some hack to make those internal bracketing nodes
work.
Yes, I tripped this one.  Turns out that you create an empty
vector<pair> with {}, not with {{}}.
This lets us run with prism elements and Tri7-based elements too
@roystgnr
Copy link
Copy Markdown
Member Author

Cleaned up history; this isn't too invasive on non-Tri7-based elements, so if CI is still happy I'll merge.

@roystgnr roystgnr merged commit e4d8556 into libMesh:devel Nov 23, 2022
@roystgnr roystgnr deleted the embedding_updates branch November 23, 2022 14:34
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