search

observablehq / runtime

The reactive dataflow runtime that powers Observable Framework and Observable notebooks
https://observablehq.com/@observablehq/how-observable-runs
ISC License
1.02k stars 72 forks source link

Don’t explicitly update outputs. #239

Closed mbostock closed 4 years ago

mbostock commented 4 years ago

This shouldn’t be necessary as the transitive outputs of an updating variable are automatically computed if they are reachable. This passes the tests, but I’m curious if there’s a reason we were doing this.

jashkenas commented 4 years ago

Interesting. If you don't mind me asking — how did you notice this? Saw some extra computation occurring while writing the runtime onboarding notebook?

mbostock commented 4 years ago

I’m reviewing the code as I write documentation, and it looked superfluous based on my understanding of it.