-
Notifications
You must be signed in to change notification settings - Fork 1
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
async/await support #54
Comments
It's a great idea! If you can get it working, I would welcome a pull request that adds this feature. |
You may also like https://github.com/observablehq/runtime which supports promises. |
Apologies but I went with another library already, and wouldn't have the bandwidth to go back and add this to topologica. I like what you built, however. |
Oh nice! I'm curious, what library did you end up going with? |
Up! This would be an amazing feature |
Thanks! I wonder what that would look like... The next step would be to come up with some failing test cases that outline the API we want to see. |
@curran I went with this one. It handles async code just fine, and schedules as many async work units as possible to be run concurrently in the current "wave" of change propagation. |
Oh wow very nice! |
This library is exactly what I was looking for, but almost all the properties I need to calculate and set in my dataflow graph are calculated using async functions. It's a shame you have to break these functions into two pieces, "cutting the dependency graph", as shown in the docs. Why not accept both async functions / promises and synchronous functions? The DAG can be scheduled in the same way, topologically.
The text was updated successfully, but these errors were encountered: