-
Notifications
You must be signed in to change notification settings - Fork 50
implemenet @block_reentrance
decorator
#744
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
added a simple example with trivially "synced" selectors: unit_circle-2025-03-03_22.50.14.mp4 |
The decorator is working as expected and exceptions are also raised correctly, @clewis7 if you can test on your end too it should be gtg! This is probably my favorite PR, small few lines decorator that makes life so much easier 🥳 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
super cool! might need to run the docs to update API?
No API changes here. |
This simple decorator applied on all the
GraphicFeature.set_value()
methods should allow us to create complex event graphs with cycles. Users don't have to think about managing bidirectional events 😄related #741
Example:
Need to test with more graphics and also test when exceptions are raised in
GraphicFeature.set_value()
@apasarkar @FlynnOConnell you guys will love this!