-
Notifications
You must be signed in to change notification settings - Fork 14
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Updated title #21
Updated title #21
Conversation
"language": "python", | ||
"name": "openeo" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Note that changing the kernel name might trigger a "Select kernel" popup to users that open this notebook on CDSE jupyterlab when the name
does not exist there.
In this case kernel python3
does exist on CDSE jupyterlab, but it's the base python and doesn't have any of the importabt packages, like openeo
installed I think
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ahh sorry for confusion @soxofaan. Changes were made locally and didn't realise that change in markdown could also trigger it
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
apparently this is documented here: https://github.com/eu-cdse/notebook-samples#kernel-spec-metadata
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
yes updated
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
FYI: under #22 I added a kernel check script that now runs as pre-commit and github actions workflow
@megha-devaraju @Kampen1988 a simple reminder :) |
@megha-devaraju @Kampen1988 updated title based on Andras suggestions. Please let us know if addition changes in text are needed