Skip to content

Add active_set function - #299

Merged
alecjacobson merged 1 commit into
libigl:mainfrom
SE-starshippilot:add-active-set-binding
Aug 11, 2026
Merged

Add active_set function#299
alecjacobson merged 1 commit into
libigl:mainfrom
SE-starshippilot:add-active-set-binding

Conversation

@SE-starshippilot

Copy link
Copy Markdown
Contributor

Added binding for active_set, as well as a test.

This is my first time creating PR for an open source project. Please let me know if i'm missing anything ;)

@alecjacobson

Copy link
Copy Markdown
Contributor

Looks reasonable to me. I'll see if it will compile and then merge into the next minor/dev release

@alecjacobson
alecjacobson force-pushed the add-active-set-binding branch from f3ebf1e to 4c96849 Compare August 10, 2026 21:00
@alecjacobson
alecjacobson force-pushed the add-active-set-binding branch from 4c96849 to 48f80f1 Compare August 10, 2026 21:09
@alecjacobson

Copy link
Copy Markdown
Contributor

Rebased onto current main. The binding builds cleanly and test_active_set passes; full suite is green (92 passed) locally. No code changes were needed — thanks for the contribution!

(Written by Claude Code on behalf of @alecjacobson.)

@alecjacobson
alecjacobson merged commit b309da6 into libigl:main Aug 11, 2026
39 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