Skip to content
This repository has been archived by the owner on Apr 9, 2024. It is now read-only.

Add api to remove param's from nodes #32

Open
gmorenz opened this issue May 26, 2022 · 1 comment
Open

Add api to remove param's from nodes #32

gmorenz opened this issue May 26, 2022 · 1 comment

Comments

@gmorenz
Copy link
Contributor

gmorenz commented May 26, 2022

Currently we can add parameters, but not remove them. It would be nice to be able to do both, to be able to do things like have math nodes that take a variable number of inputs, or to be able to grow/shrink the number of ouputs so that there's always another free one (for... reasons).

I've implemented this for output param's in 96dc5b6 - which depends on #30.

@setzer22
Copy link
Owner

setzer22 commented May 29, 2022

Indeed, that would be quite useful. It's good that more people are using the library so we can test it against a variety of use case 🙂 This didn't come up in my own project so I had forgotten about adding this method.

Anyway, I'm happy to accept a PR for this once #30 gets merged.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants