Conversation
We write the boilerplate in TS and transpile it to JS into the public folder. From there we can later load/serve it.
Return format is compatible to JS code generator.
|
I was able to find out already that the UI hanging was not your fault at all, everything looks good in that regard. Its actually the creation of a new Context in quickjs that hangs the UI for about 5 seconds every time. If we dont use the context, then there is no hang. I will look into it more |
|
The research concludes:
Solution: Use the normal QuickJS WASM runtime in a Web Worker (which runs in a separate thread) Luckily, I have plenty of experience with web workers from NovaCrate, so I'll try to quickly take care of it |
Remove export keyword and replace console.
a589517 to
4b772b3
Compare
This speeds up container use and avoids the weird interference with python venvs.
|
fixed. On my way to finalize things. |
…using node_modules / package.json
|
Notes:
|
craquet
left a comment
There was a problem hiding this comment.
Did not do a thorough review yet, but I noticed that the trace seems to break at least if-else blocks and functions with returns. We have to check if we can find a stable ignore list to ignore these edge cases or if we should take a different approach to tracing.
Maybe we could also leave the tracing out alltogether for this PR, as it is not central to the purpose of this PR and we have to get done until the conference :D
|
|
I think I improved the blacklisting in tracing properly now. But I also disabled it (see commit b16bb90) for now. |
|
I only noticed that the otherwise blocks conditionals were flipped (returning otherwise even if either is truthy). Fixed that already. Everything else looks good |
Uh oh!
There was an error while loading. Please reload this page.