observablehq / stdlib

The Observable standard library.
https://observablehq.com/@observablehq/standard-library
ISC License
957 stars 83 forks source link

Update caching logic for tables #329

Closed libbey-observable closed 1 year ago

libbey-observable commented 1 year ago

Resolves https://github.com/observablehq/observablehq/issues/9381

I did see the other possible solution suggestion, to bypass the cache entirely for array sources that are not done. If that seems like a better solution, happy to revise.

libbey-observable commented 1 year ago

Thanks! Seems a bit like what I tried, with the (welcome) additional check on the array type.

Yes, entirely your approach, with Mike B's comments incorporated. I experimented around a bit, but a better solution didn't immediately jump out at me. I'll go ahead and merge, especially since it's not automatically deployed, so there will still be a chance to revise before it goes out.