Jeremy Tuloup

Results 2209 comments of Jeremy Tuloup

Thanks @dev-amanydv for working on this!

This seems to be coming from this line: https://github.com/WebReflection/coincident/blob/b8f9cb41a411d66a1b665212eb10e79de53a4d8b/esm/bridge.js#L44

For reference a similar error is also happening with `jupyterlite-xeus` and the `xeus-python` kernel: ![image](https://github.com/jupyterlite/jupyterlite/assets/591645/a85e65fc-f71e-4efe-9e30-683839683b9c)

Digging a bit more in the `coincident` code base, it looks like `coincident` sets its own metadata on the patched `postMessage`. So maybe switching from `postMessage` to `coincident` on the...

Looks like we are still using `onmessage` here in the Pyodide kernel: https://github.com/jupyterlite/pyodide-kernel/blob/7e9351258ef41a9f2dfdd96753ea2ee1e719d12b/packages/pyodide-kernel/src/kernel.ts#L31 Which could likely be converted to use `coincident`?

Thinking we should try to fix this before `0.4.0`, so we can follow-up with the final release after.

> Thinking we should try to fix this before `0.4.0`, so we can follow-up with the final release after. Looks like this can be fixed in the kernel repositories directly:...

> * a) increase them > > * b) make them configurable, or > > * c) show a div in the output like "this video/audio is large and may...

> Hi, could you please add the required label to this pull request so it can proceed with review? It looks like this PR makes a lot of unrelated changes...

Wow, thanks again @fleming79 for opening this massive PR! I tried the notebooks and got to play with the new features, and they look great! It would make sense to...