IJulia.jl icon indicating copy to clipboard operation
IJulia.jl copied to clipboard

IJulia not generating standardised metadata for cells within jupyter lab to show cell execution duration time

Open octavd opened this issue 5 years ago • 4 comments

version() output

image

As can be seen from the above image, the jupyterlab-execute-time extension only shows the datetime stamp when using IJulia kernel but the execution duration is missing. Works fine for python / spark / sparkr / pyspark kernels. Having the same issue for IRkernel ticket was opened there also.

Have already validated the issue with at least one other user using a different version of OS/kernel than mine and have also reached out to the jupyterlab-execute-time devs who have confirmed that this likely appears to be an issue with the kernel and not with jupyterlab or the jupyterlab-execute-time plugin.

This issue seems to be related to the metadata that gets generated for each cell by the kernel. image

Below you can find the cell metadata generated using Python kernel for example: image

Please see the comments here - https://github.com/deshaw/jupyterlab-execute-time/issues/14

octavd avatar Oct 12 '20 08:10 octavd

Heads up: I assume this has to do with IJulia supporting the 5.0 protocol, which didn’t specify a date field in the response yet:

https://github.com/JuliaLang/IJulia.jl/blob/ad459e8d46a908c6b923cafa79572a610e2869bb/src/handlers.jl#L153

Changed in version 5.1: date in the header was accidentally omitted from the spec prior to 5.1, but it has always been in the canonical implementation, so implementers are strongly encouraged to include it. It will be mandatory in 5.1. – General Message Format

flying-sheep avatar Oct 17 '20 13:10 flying-sheep

First of all, thank you for your reply. I think so. I see that the problem for IRKernel was solved. Could you please tell me if for this this will be solved also? :D

octavd avatar Oct 19 '20 13:10 octavd

Problem is solved. Just upgrade to jupyterlab 2.2.9 and use the latest jupyterlab-execute-time extension.

octavd avatar Jan 11 '21 17:01 octavd

We should still update to the 5.1 protocol spec at some point.

stevengj avatar Jan 11 '21 21:01 stevengj