Skip to content

Conversation

@termi-official
Copy link
Member

No description provided.

@codecov
Copy link

codecov bot commented Dec 9, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 94.20%. Comparing base (f99e206) to head (69a74d5).
⚠️ Report is 2 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #1259      +/-   ##
==========================================
+ Coverage   94.19%   94.20%   +0.01%     
==========================================
  Files          40       40              
  Lines        6662     6676      +14     
==========================================
+ Hits         6275     6289      +14     
  Misses        387      387              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

Copy link
Member

@KnutAM KnutAM left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I haven't checked the output, but it looks like it could fail for higher order geometric interpolations?

Nvm, looks correct on 2nd thought !

Edit3: Or no, it does look wrong...

@KnutAM KnutAM linked an issue Dec 14, 2025 that may be closed by this pull request
@KnutAM
Copy link
Member

KnutAM commented Dec 14, 2025

Btw will (or can) this close #192, or is that already resolved?

@termi-official
Copy link
Member Author

@termi-official termi-official requested a review from KnutAM January 7, 2026 11:37
@termi-official termi-official added the awaiting review PR is finished from the authors POV, waiting for feedback label Jan 7, 2026
Copy link
Member

@KnutAM KnutAM left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The code LGTM, but haven't checked the resulting vtk-files.

Copy link
Member

@KnutAM KnutAM left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice

@termi-official termi-official merged commit 667a139 into master Jan 8, 2026
15 of 16 checks passed
@termi-official termi-official deleted the do/write-facetset branch January 8, 2026 20:56
@termi-official termi-official removed the awaiting review PR is finished from the authors POV, waiting for feedback label Jan 8, 2026
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.

Facetset writer

3 participants