-
-
Notifications
You must be signed in to change notification settings - Fork 2.1k
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
Add support for reactive expressions #26
Comments
Wow, this is really cool. Thanks for sharing this! I'm going to keep this issue open to let the idea ferment a little bit longer and to promote discoverability. The verbosity of the current method with decorators isn't enough for me to warrant a new syntax, but I can imagine future versions of this package ending up using something along these lines. The current method also allows for other types of |
This type of syntax is similar to what is being proposed for client-side operations in #266 |
Add selection fixtures
Hi - this issue has been sitting for a while, so as part of our effort to tidy up our public repositories I'm going to close it. If it's still a concern, we'd be grateful if you could open a new issue (with a short reproducible example if appropriate) so that we can add it to our stack. Cheers - @gvwilson |
The current interface to specify a reactive function using decorator taking output and inputs as arguments is too verbose and doesn't feel very elegant.
Wouldn't it be better to say:
instead of:
The former one feels more natural way to specify the logic. Also, it makes it easy to use the same function in different scenarios. For example:
This would be too cumbersome to do with the current approach.
I've implemented a quick proof-of-concept code to demonstrate that such a thing is possible. The code and notes are available at:
https://gist.github.com/anandology/b93fd555215203977301ecb4994efe66
The text was updated successfully, but these errors were encountered: