_isnotebook() does not recognise JupyterLite

Abierto
#166 0 comentarios 0 reacciones 0 asignados Ver en GitHub

Los mantenedores suelen responder en 1 día

Nadie ha tomado este issue todavía.

  • #167 de @petercorke — cerrado sin fusionar

Evaluación

Dificultad
3/5
Tiempo estimado
Medio día
Aptitud para principiantes
45/100
Tipo de issue
Error
Claridad
Bastante claro
Estado de actividad
Activo
Stack tecnológico
jupyter, python
Área
tooling

Línea de trabajo

Start with _isnotebook() in base/imageio.py and the Camera image-plane reuse branch that depends on it. Check what get_ipython().__class__.__name__ returns inside a JupyterLite (Pyodide) session, then add that detection alongside the existing Colab and ZMQ checks. Done when the JupyterLite session takes the notebook branch. Confirm first whether the linked PR #167 (closed, unmerged) already covers this.

Escrito por el modelo de indexación a partir del texto del issue.

Descripción

Likely smell. Worth checking out.

base/imageio.py _isnotebook() only matches a shell class named Shell (Colab) or ZMQInteractiveShell. It has no JupyterLite check; RTB added sys.platform == "emscripten". If the Pyodide shell has another name, Camera takes the desktop branch of its image-plane reuse in the browser. I have not checked what get_ipython().__class__.__name__ is in the MVTB Lite site.

RTB, MVTB and bdsim each have their own copy of this check. rai-opensource/spatialmath-python#271 adds environment() and isnotebook() to spatialmath.base, covering terminal, IPython, Jupyter, Colab and JupyterLite. Once it is released, this could use it.

Lenguaje dominante
Python
Estrellas
225
Forks
35
Merge medio
1 d 12 h
PR fusionados (30 d)
33

Preparar el entorno

Primeros pasos

  1. Lee el issue completo y luego la guía de contribución del proyecto.
  2. Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
  3. Haz un fork del repositorio y trabaja en una rama.
  4. Abre un pull request que haga referencia al número del issue.

Más de petercorke/machinevision-toolbox-python

Todos los issues de petercorke/machinevision-toolbox-python

Issues similares

Más issues de Python

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.