Skip to content

Spenbert/manifold extraction#876

Merged
daniel-zint merged 17 commits into
mainfrom
spenbert/manifold_extraction
Feb 18, 2026
Merged

Spenbert/manifold extraction#876
daniel-zint merged 17 commits into
mainfrom
spenbert/manifold_extraction

Conversation

@Spenbert02

Copy link
Copy Markdown
Collaborator

No description provided.

@daniel-zint daniel-zint left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Not fully done, but I have to board my flight now. This should be plenty already, though.

@daniel-zint daniel-zint left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

A few more comments. Please respond to the comments or mark them as resolved if you did actually resolve them. That makes my review process much easier.
I think if you go through my comments again, the PR will be ready for merging.

But also make sure that the CI passes on every machine. Currently, it only passes on Mac.

Comment thread components/manifold_extraction/wmtk/components/manifold_extraction/CMakeLists.txt Outdated
Comment thread components/manifold_extraction/wmtk/components/manifold_extraction/graveyard.txt Outdated

@daniel-zint daniel-zint left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Looks good! I'll merge as soon as the tests pass.

@daniel-zint daniel-zint merged commit 4e60076 into main Feb 18, 2026
8 checks passed
@daniel-zint daniel-zint deleted the spenbert/manifold_extraction branch February 18, 2026 10:49
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