Skip to content

Parametrizing kernels #1

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.

Sign up for GitHub

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

Jump to bottom
Open
wants to merge 40 commits into
base: main
Choose a base branch
from
Open

Parametrizing kernels #1

wants to merge 40 commits into from

Conversation

AnastasiaSliusar
Copy link
Owner

@AnastasiaSliusar AnastasiaSliusar commented May 10, 2024 •
edited
Loading

References

This work is in progress as JEP jupyter/enhancement-proposals#87

Code changes

  • Replacing kernel spec variables for arg and env that are in kernel.json into values from custom_kernel_specs parameter which is configured by a user
  • Clearing kernel spec variables that are in kernel.json if they are present into this file but custom_kernel_specs parameter is missed
  • Adding basic validation to check whether custom_kernel_specs does not include anything wrong
  • Adding support if a kernel.json file includes customization for a kernel but a user has not selected anything then a parameterized kernel will be run by using default values from its kernel.json file

AnastasiaSliusar and others added 30 commits May 6, 2024 22:02
@AnastasiaSliusar
Add checking custom kernel specs
151528d
@AnastasiaSliusar
Add replacing custom kernel specs
f5fb178
@AnastasiaSliusar
Clear kernel spec variables from kernel.json
ce0864c
@AnastasiaSliusar
Fix cleanup
d796294
@AnastasiaSliusar
Add small fixes for validation and cleanup
4aecf42
@AnastasiaSliusar
Getting default values and fix updating env
49e3b9a
@AnastasiaSliusar
Fix defining custom env parameters
6518491
@AnastasiaSliusar
add logging
7c21d47
@AnastasiaSliusar
Fix custom parameters of a kernel
2e62e53
@AnastasiaSliusar
Fix restarting a kernel with custom env variable and clean printing v…
f6440a9 
…ariables
@AnastasiaSliusar
Merge branch 'main' into parametrizing_kernels
d25af7d
@AnastasiaSliusar
Clean printing
8cdac5c
@pre-commit-ci
[pre-commit.ci] auto fixes from pre-commit.com hooks
1b0c92e 
for more information, see https://pre-commit.ci
@AnastasiaSliusar
Fix updating launch parameters when a new kernel is selected
610f018
@pre-commit-ci
[pre-commit.ci] auto fixes from pre-commit.com hooks
db2b88e 
for more information, see https://pre-commit.ci
@AnastasiaSliusar
Fix updating launch parameters when a new kernel is selected
2d3c923
@AnastasiaSliusar
Add switching parameter
4e56a30
@AnastasiaSliusar
Merge branch 'parametrizing_kernels' of https://github.com/AnastasiaS…
c3ac039 
…liusar/jupyter_client into parametrizing_kernels
@pre-commit-ci
[pre-commit.ci] auto fixes from pre-commit.com hooks
6c6d9af 
for more information, see https://pre-commit.ci
@AnastasiaSliusar
Support replacing env and argv by default parameters
e6f68e2
@AnastasiaSliusar
Resolve and merge conflicts
47400a4
@pre-commit-ci
[pre-commit.ci] auto fixes from pre-commit.com hooks
b7fac4a 
for more information, see https://pre-commit.ci
@AnastasiaSliusar
Update a parameter
85d0829
@AnastasiaSliusar
Resolving conflicts and merging
2267438
@pre-commit-ci
[pre-commit.ci] auto fixes from pre-commit.com hooks
b044207 
for more information, see https://pre-commit.ci
@AnastasiaSliusar
Count kernel variables
072da9a
@AnastasiaSliusar
resolving conflicts
562a3ea
@pre-commit-ci
[pre-commit.ci] auto fixes from pre-commit.com hooks
b646dc1 
for more information, see https://pre-commit.ci
@AnastasiaSliusar
Update filtering kernels spec
7620d38
@AnastasiaSliusar
resolving conflicts
a747ee5
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@AnastasiaSliusar