Johan Mabille comments

Results 364 comments of


                                            Johan Mabille

Use papermill library to execute notebooks and check necessary outputs ?

This should be done in a testing framework repo (like https://github.com/jupyter/jupyter_kernel_test), not in xeus itself.

Install conda_oci_mirror to use in conda-forge-ci-setup-feedstock

cc @ocefpaf

Fix: Add auto_unbox to JSON serialization for VS Code compatibility

Thanks for the fix! Can you update the tests so that they reflect the changes?

cmd.exe custom shell doesn't work with micromamba 2

The thing that changed is that the `micromamba.bat` script generated by `micromamba shell init -s cmd.exe` is now called ... `mamba.bat`. When calling micromamba with v1, it was actually calling...

update xeus 5

It looks like [cpp-tabulate](https://github.com/p-ranav/tabulate/blob/master/CMakeLists.txt#L39C5-L45) exports the CXX standards publicly (which is a bad practice), and is built with CXX 11 on conda-forge. This overwrites whatever CXX standard we can set...

`jupyter kernel` process doesn't exit after kernel is shutdown

The SUB socket of the client can take time to connect to the IOPUB channel, and the client can miss important messages (especially those with the kernel status). The current...

Avoid warnings in compiler version checks

I'll rebase to have the CI run and merge after.

Add Bazel support

Hi @Vertexwahn , and thank you for your contribution. > In the worst case the Bazel build does simply not work [...] I am not fan of providing sometihng that...

ERROR: received bad message: NameError: name 'debugpy' is not defined

Can you check if debugpy has been installed in the container? If not, how did you install `xeus-python`?

xtensor readme example doesn't work with xeus-cpp-lite

Ok, let's keep this one open for now, I'll try to do that next week. @ThisIsFineTM Thanks for trying it/ Let me know if you need a fix for xtensor...