Skip to content

Updated Polyscript & Coincident to their latest #2327

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

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

WebReflection
Copy link
Contributor

Description

This MR is a follow up of this one: pyscript/polyscript#132

It's meant to test improvements (or possible issues) in CI or in the wild, using the latest single rountrip and simplified coincident approach.

Changes

  • updated polyscript to its "beta" version
  • tested everything is fine

Checklist

  • I have checked make build works locally.
  • I have created / updated documentation for this change (if applicable).

@WebReflection
Copy link
Contributor Author

image

this put a big smile on my face, it's usually 3 up to 5 but on average I've seen 30 seconds less time, compared to other normal builds ... meaning there is some gain.

Locally, it used to take ~32 seconds and now is more like ~19 for the pyodide in worker case ... and like I've said, there's more to do but as a starter, and a fundamental change in the deepest part of our core, I hope this improvement is welcomed.

@WebReflection WebReflection force-pushed the coincident-rewrite branch 6 times, most recently from 0a9dd34 to 39fc191 Compare April 9, 2025 19:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants