-
Notifications
You must be signed in to change notification settings - Fork 1.5k
Updated Polyscript to its latest #2355
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
Conversation
@ntoll this is a before / after in our CI ... I have 12 secs before and 6 secs after so it's aligned with almost twice as fast Pyodide bootstrap and usage which is closer to native (on main) performance. |
for more information, see https://pre-commit.ci
8dbcca1
to
7c02324
Compare
FYI this has been published on npm as both |
@ntoll I've moved the ugly way to use the experimental worker boost from global script to the config which feels like more appropriate. Description if Ideally, we should make it |
OK, in case we want to release this (we should) here I am writing down what this latest update would bring in: Release Notes
|
95bf7af
to
0a3cd41
Compare
* Updated Polyscript to its latest * added tests for `experimental_ffi_timeout`
Description
This MR updates the whole Polyscript stack and libraries behind and it uses, as opt-in, a new experimental feature that provides smart caching behind the scene and it reduces Pyodide bootstrap and execution time in workers 2X when enabled.
This MR also brings in some extra utility that might be flagged as experimental but are meant to boost a lot some ad-hoc use case once needed ... we might need time and more real-world cases to document properly these features but because there are already somehow exposed in polyscript, I've thought it'd be great to be able to test this while in Boston.
Changes
Checklist
make build
works locally.