quadratichq / quadratic

Quadratic | Technical Spreadsheet with Python, SQL, and AI
https://QuadraticHQ.com
Other
2.8k stars 140 forks source link

feat: make getCells synchronous #1450

Open AyushAgrawal-A2 opened 1 week ago

AyushAgrawal-A2 commented 1 week ago

related to #1447, #1409, #1114

This pr is meant to explore usage of Atomics and SharedArrayBuffer, for communication between workers. Using this we can block(sleep) a worker thread and execute synchronously, instead of using async / await

Pros:

Cons:

vercel[bot] commented 1 week ago

@AyushAgrawal-A2 is attempting to deploy a commit to the Quadratic Team on Vercel.

A member of the Team first needs to authorize it.

vercel[bot] commented 1 week ago

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Updated (UTC)
quadratic ✅ Ready (Inspect) Visit Preview Jun 24, 2024 1:02pm
davidfig commented 1 day ago

This works great. Outside of the necessary server changes, is there anything else that makes this a POC and not ready for review? We'll also need this ported to JS so we can remove the await requirement there as well.

AyushAgrawal-A2 commented 1 day ago

@davidfig ready for review

davidfig commented 19 hours ago

Awesome. Works wonderfully.