Skip to content
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

Update notebook's kernel name and display name for default one (python3) #126

Merged
merged 1 commit into from
Dec 2, 2021

Conversation

nhuet
Copy link
Contributor

@nhuet nhuet commented Nov 29, 2021

The current chosen kernel name (cinb_env) was specific to my configuration and it will be not found on a generic computer.

It results to

  • a popup message asking for choosing the appropriate kernel when running locally (so not completely an error but still unsatisfactory)
  • an error when testing the notebook with tool like nbmake with pytest --nbmake

We may want to add a pre-commit hook to check that later.

@dbarbier
Copy link
Contributor

Why is this change needed? Please explain in the commit message what issue it solves.

@nhuet nhuet force-pushed the nh/benchmark_nb_kernelname branch from 7254f6f to 6e3c71b Compare November 29, 2021 14:51
The current chosen kernel name (cinb_env) was specific to my
configuration and it will be not found on a generic computer.
It results to
- a popup message asking for choosing the appropriate kernel
when running locally (so not completely an error but still
unsatisfactory)
- an error when testing the notebook with tool like nbmake
  with `pytest --nbmake`
@nhuet nhuet force-pushed the nh/benchmark_nb_kernelname branch from 6e3c71b to 14d09e7 Compare November 29, 2021 14:52
@nhuet
Copy link
Contributor Author

nhuet commented Nov 29, 2021

Commit message and PR updated with the appropriate info.

@galleon galleon merged commit c3f957c into airbus:master Dec 2, 2021
@nhuet nhuet deleted the nh/benchmark_nb_kernelname branch December 23, 2021 09:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants