-
Notifications
You must be signed in to change notification settings - Fork 200
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
fix(compiler): equality and external libraries bugs in simulator #5554
Conversation
Thanks for opening this pull request! 🎉
|
Console preview environment is available at https://wing-console-pr-5554.fly.dev 🚀 Last Updated (UTC) 2024-03-06 20:39 |
BenchmarksComparison to Baseline ⬜⬜⬜⬜⬜⬜⬜⬜⬜⬜⬜⬜⬜
⬜ Within 1.5 standard deviations Benchmarks may vary outside of normal expectations, especially when running in GitHub Actions CI. Results
Last Updated (UTC) 2024-03-06 20:45 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Very exciting! This PR supercedes #5547 right?
btw, on the note of worker vs child process: One of the big optimization options with workers is that you can share memory between them and the host, so we can use SharedArrayBuffer as a way to do fast (and still serialized) communication.
Might be an interesting future optimization, but cp makes sense now too
@MarkMcCulloh yep, I think this can supersede the other PR assuming the approach makes sense to y'all.
Interesting, yeah I see how that could give us some efficiency gains. It would be interesting to do some profiling on wing programs to see how much time is spent computing vs on IPC/HTTP etc. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looking forward for this one!
Signed-off-by: monada-bot[bot] <monabot@monada.co>
Signed-off-by: monada-bot[bot] <monabot@monada.co>
Signed-off-by: monada-bot[bot] <monabot@monada.co>
Signed-off-by: monada-bot[bot] <monabot@monada.co>
Signed-off-by: monada-bot[bot] <monabot@monada.co>
Signed-off-by: monada-bot[bot] <monabot@monada.co>
Thanks for contributing, @Chriscbr! This PR will now be added to the merge queue, or immediately merged if |
Congrats! 🚀 This was released in Wing 0.59.43. |
This pull request fixes a range of funky JavaScript issues we've had running inflight code in the Wing simulator by switching internally from running JS code in a
vm
to running JS code using Node.js child processes.The main issues with
vm
are:vm
and outside. This may be related to differences in vm behavior as described in Contexts created with vm.createContext() do not define the URL() constructor nodejs/node#28823vm
process, making it impossible to simulate cloud.Function timeoutsFixes #1980
Fixes #4131
Fixes #4118
Fixes #4792
Closes #4725
Checklist
pr/e2e-full
label if this feature requires end-to-end testingBy submitting this pull request, I confirm that my contribution is made under the terms of the Wing Cloud Contribution License.