search

JovianHQ / jovian-py

Collaboration platform for data science projects & Jupyter notebooks
https://www.jovian.ai
112 stars 31 forks source link

Delete kernelspec entry from .ipynb file during Jovian clone #215

Closed prabhat625 closed 3 years ago

prabhat625 commented 3 years ago

Summary

Delete the kernelspec entry from the .ipynb file during jovian clone.

Clubhouse Ticket
prabhat625 commented 3 years ago

All changes at the library are done with testing. @PrajwalPrashanth @sonakshs

PrajwalPrashanth commented 3 years ago

image I'm getting this error

PrajwalPrashanth commented 3 years ago

image I'm getting this error

@prabhat625 I tried it with another notebook (just one cell) it does not seem to be consistent, but always consistent with this notebook. https://jovian.ml/PrajwalPrashanth/colab-improvements

I tried with clone without the changes of this PR, where this error would not appear for the same notebook. Can you try to clone this notebook and LMK if this persists ? Most probably some inconsistency while using json.

If you come across the same message for the above notebook, can you try with nbformat and see if that persists.

PrajwalPrashanth commented 3 years ago

image I'm getting this error

This is quite flaky and not happening for all the notebooks. Seems to be consistent with this notebook https://jovian.ml/PrajwalPrashanth/colab-improvements/v/13 and when your env has nbformat=5.0.4 (same notebook works fine with 5.0.7, which is the default version which gets installed at the time of this comment)