jupyter-collaboration icon indicating copy to clipboard operation
jupyter-collaboration copied to clipboard

Images don't display

Open davidbrochart opened this issue 1 year ago • 6 comments

See https://github.com/jupyterlab/jupyter-collaboration/issues/162#issuecomment-1821349112.

Description

In RTC images won't display.

Reproduce

  1. Install JupyterLab and jupyter-collaboration.
  2. Open an image from the file browser.
  3. See image not shown.

Expected behavior

Images should be displayed. Jpterm is able to show images using a YBlob.

Context

  • Operating System and version: Linux Ubuntu 22.04
  • JupyterLab version: 4.0.9
  • jupyter-collaboration version: 2.0.0

davidbrochart avatar Dec 27 '23 08:12 davidbrochart

With jupyter-collaboration installed, even if I'm not in an active collab session, the image gets uploaded with RTC.

What do you mean "the image gets uploaded"?

davidbrochart avatar Jan 04 '24 18:01 davidbrochart

Apologies for the confusion. Should have said - "With jupyter-collaboration installed, even if I'm not in an active collab session, and if i upload an image to jupyterlabs, the image gets uploaded with RTC."

did that clarify?

BwL1289 avatar Jan 04 '24 18:01 BwL1289

So your issue is not that double-clicking on an image in the file browser doesn't show the image? I don't really get what it means that the image gets uploaded with RTC.

davidbrochart avatar Jan 04 '24 18:01 davidbrochart

Let me clarify.

My issue, like yours, is that images don't display after I upload them and try to view them from the file browser.

I was also commenting that even when I'm not in an active collaboration session, and I upload an image, the image is marked as being uploaded with RTC. See screenshot. Perhaps this is expected behavior. I'm not familiar enough with the framework to know. Screenshot 2024-01-04 at 12 44 03 PM

BwL1289 avatar Jan 04 '24 18:01 BwL1289

Oh I see now, thanks! Yes the RTC: prefix in the path identifies the drive, but this is not the issue here. The problem is that the frontend asks for a document of type file, which corresponds to a YUnicode shared document, but this doesn't handle binary data. Instead, it should ask for a blob which correspond to a YBlob.

davidbrochart avatar Jan 04 '24 19:01 davidbrochart

Ack! Appreciate the clarification!

BwL1289 avatar Jan 04 '24 19:01 BwL1289

This is still an issue with JupyterLab 4.1.8 and Jupyter Collaboration 2.1.2. Images are still not displaying with RTC enabled.

JasonWeill avatar Oct 08 '24 18:10 JasonWeill

It should be fixed in jupyter-collaboration v2.1.3.

davidbrochart avatar Oct 11 '24 15:10 davidbrochart