-
Notifications
You must be signed in to change notification settings - Fork 1
Nested Digests #18
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
Comments
Necessary for datavis-tech/reactive-vis#14 |
I wonder if the current implementation supports multiple levels of nesting... I'm not sure it does. It wipes out the changed nodes before and after the topological sort and dataflow graph execution, but I wonder if it might be better to use a stack here, pushing and popping the context. |
Added a test for doubly nested digest. It does work in the current implementation. It appears that the computed topological sort serves as a kind of call stack, so I think digests can be nested arbitrarily deeply. |
Within a reactive function, it should be possible to invoke
digest
multiple times.The text was updated successfully, but these errors were encountered: