open-digger icon indicating copy to clipboard operation
open-digger copied to clipboard

[Docs] Add docs for Python kernel

Open frank-zsy opened this issue 2 years ago • 6 comments

We need to add docs for Python kernel and also push Python kernel docker image for developers to use.

frank-zsy avatar Sep 13 '22 03:09 frank-zsy

This issue has not been replied for 24 hours, please pay attention to this issue: @gymgym1212 @xiaoya-yaya

open-digger-bot[bot] avatar Sep 14 '22 01:09 open-digger-bot[bot]

During writing the doc of Sample Data Usage for Python Kernel #998 , I find the doc a bit difficult to read through. I think the structure of the open-digger is a bit complex and unfriendly to both node.js and python developers. Is there any plan to tell the 2 versions(node.js and python) apart? Or some better idea? Or is this the current structure the best choice?

yoyo-wu98 avatar Oct 12 '22 14:10 yoyo-wu98

Yes, actually I think a better idea is to separate the python and node.js kernel file into different folders like src/node/ and src/python, so developers can understand better and focused on a certain kernel.

frank-zsy avatar Oct 13 '22 01:10 frank-zsy

Hello @frank-zsy , I think we can store our python_kernel image into our remote DockerHub so we don't have to build image again. What do you think?

xgdyp avatar Oct 14 '22 02:10 xgdyp

Would you like to test the new docs on #998?@Zi1l

xgdyp avatar Oct 14 '22 02:10 xgdyp

Hello @frank-zsy , I think we can store our python_kernel image into our remote DockerHub so we don't have to build image again. What do you think?

Yes, of course.

frank-zsy avatar Oct 14 '22 08:10 frank-zsy

Would you like to test the new docs on #998?@Zi1l

Based on the new docs with python_kernel, open-digger can run successfully. Besides, I am sorry that I didn't see the message in time because of the chage of my ID.

l1tok avatar Oct 22 '22 13:10 l1tok