vdk-events: improve Productionizing Jupyter Notebooks README #2896
Conversation
antoniivanov
left a comment
antoniivanov
left a comment
There was a problem hiding this comment.
remove explore from the name of hte pdf. Also what is this pdf ? We should not have pdf in source control in general.
It is the presentation for Explore i can remove it |
duyguHsnHsn
force-pushed
the
person/hduygu/vdk-events-productionizing-jupyter-notebooks
branch
from
a0b00b7 to
6e3abfa
Compare
for more information, see https://pre-commit.ci
Yeah, we do not need it. Let's embed some of the images in the notebook. |
yonitoo
left a comment
yonitoo
left a comment
There was a problem hiding this comment.
LGTM. Good job! Apart from the minor comment I left, a nit (subjective): sometimes "Notebook" is uppercase and sometimes not.
antoniivanov
left a comment
antoniivanov
left a comment
There was a problem hiding this comment.
Fix the description of the PR. It's not updated
for more information, see https://pre-commit.ci
antoniivanov
deleted the
person/hduygu/vdk-events-productionizing-jupyter-notebooks
branch
What: Added additional information from https://github.com/versatile-data-kit-demo/productionizing-jupyter-notebooks/tree/main to the events README
Why: To make it easier for the people to launch and understand the tutorial
Signed-off-by: Duygu Hasan hduygu@vmware.com