diff --git a/jupyter-parameterized-kernel-spec/jupyter-parameterized-kernel-specs.md b/jupyter-parameterized-kernel-spec/jupyter-parameterized-kernel-specs.md new file mode 100644 index 00000000..f8615b54 --- /dev/null +++ b/jupyter-parameterized-kernel-spec/jupyter-parameterized-kernel-specs.md @@ -0,0 +1,125 @@ +--- +title: Jupyter Parameterized Kernel Specs +authors: Carlos Herrero, Anastasiia Sliusar +pr-number: 87 +date-started: 2022-02-10 +shepherd: Johan Mabille +--- + +# Jupyter Parameterized Kernel Specs + +## Problem + +When creating a new kernel, we need to define a different "specs" file for each possible parameter combination used by the executable. For example, the xeus-cling kernel supports three different versions of the `C++` language, `C++11`, `C++14`, and `C++17`, which is specified by the `--std` command-line option. + +When installing xeus-cling, we install three almost identical kernel specs. The only difference is the last parameter of the execution command `--std=c++X`. + +When more than one parameter is available, the number of possible combinations grows in a combinatorics fashion. + +Besides, some kernels would benefit from being configurable when launching them. For example, the connection information to the database could be specified through kernel parameters rather than kernel magics. + +Sometimes when installing multiple kernels, the JupyterLab launcher panel is crowded with too many options to create a notebook or console with a specific kernel. The kernels that users see in the launcher are just multiple kernel specs for one kernel. We could avoid having that many kernels displayed, adding parameters to the kernel specs, showing only one option per kernel, and offering a modal dialog with the different options the user can choose from when selecting a specific kernel. + +## Proposed Enhancement + +The solution we are proposing consists of adding parameters to the kernel specs file in the form of a JSON Schema that would be added to the specs metadata. These parameters are then used to populate the `argv` list and `env` dict (respectively the command-line arguments and environment variables). + +Upon starting a new kernel instance, a front-end form generated from the JSON schema is prompted to the user to fill the parameter values. Many tools are available to generate such forms, such as react-jsonschema-form. + +These kernel parameters will be not saved into the notebook due to security reason. The app will have "allow_insecure_kernelspec_params" witch will detect whether we have to accept all kernel spec files as safe or not + +## Detailed Explanation + +As described in previous sections, we propose to parameterize the kernel specs file. In the example shown below, we can see the kernel specs file from the kernel xeus-cling. We suggest changing the last parameter of the execution command `-std=c++11` to have a variable `-std=${cpp_version}` and adding a new object `parameters` to the metadata of the kernel specs. + +```=json +{ + "display_name": "C++11", + "argv": [ + "/home/user/micromamba/envs/kernel_spec/bin/xcpp", + "-f", + "{connection_file}", + "-std=c++11" + ], + env: { + "XEUS_LOGLEVEL": "ERROR" + }, + "language": "C++11" +} +``` +```=json +{ + "display_name": "C++", + "argv": [ + "/home/user/micromamba/envs/kernel_spec/bin/xcpp", + "-f", + "{connection_file}", + "-std={cpp_version}" + ], + env: [ + "XEUS_LOGLEVEL={xeus_log_level}" + ], + "language": "C++" + "metadata": { + "parameters": { + "properties": { + "cpp_version": { + "type": "string", + "default": "C++14", + "enum": ["C++11", "C++14", "C++17"] + }, + "xeus_log_level": { + "type": "string", + "default": "ERROR", + "enum": ["TRACE", "DEBUG", "INFO", "WARN", "ERROR", "FATAL"] + } + } + } + }, +} +``` + +Note: Using the JSON Schema, we can automate how front-end forms are created directly from the parameters allowing kernels' authors to decide which parameters are necessary and how to validate them. (Note that JupyterLab already makes use of react-jsonschema-form in other parts of its UI). + +In the following screenshots, you can see a small demo of how we envision the UI changes in JupyterLab. + + Jupyterlab Launcher | Select c++ version +:-------------------------:|:-------------------------: + ![](./launcher.png) | ![](./launcher-select-c-version.png) + + +![](./notebook-select-kernel.gif) + + + + +## Pros and Cons + +Pros: + + - A less crowded list of kernels specs + +Cons: + + - Changes are required in multiple components of the stack, from the protocol specification to the front-end. + - Unless we require default values for all parameters, this would be a backward-incompatible change. + +## Decisions + +- Kernel custom parameters won't be saved into a notebook metadata due to security reason +- The application can be run with `allowed_insecure_kernelspec_params` parameter which allows a user to see a dialog where they can setup custom kernel parameters + + +## Checking secure kernelspecs + +Upon starting, a kernel client checks whether available kernel spec files are secure: + + - if a kernel spec does not have a `metadata.parameters` field, it is considered as secure. The kernel can be started directly. + + - if all parameters in a kernel spec `metadata.parameters` have constraining types (i.e. the user cannot enter abritrary input), the kernel spec is secure. Starting a kernel will show a form where the user can choose the parameters. + + If a kernel spec file is not secure and the flag `allowed_insecure_kernelspec_params` is passed when starting the app then a user will be able to fill a form. + + If a kernel parameter has a non constraining type but provides a default value, this latter will be used and the user won't be able to fill this parameter in the form. + + If a kernel parameter has a non constraining type, and does not provide a default value, then the kernel spec is considered as unsecure. In that case, unless the `allowed_insecure_kernelspec_params` has been passed when starting, the kernelspec is discarded and the user won't be able to start the kernel. \ No newline at end of file diff --git a/jupyter-parameterized-kernel-spec/launcher-select-c-version.png b/jupyter-parameterized-kernel-spec/launcher-select-c-version.png new file mode 100644 index 00000000..be0b8949 Binary files /dev/null and b/jupyter-parameterized-kernel-spec/launcher-select-c-version.png differ diff --git a/jupyter-parameterized-kernel-spec/launcher.png b/jupyter-parameterized-kernel-spec/launcher.png new file mode 100644 index 00000000..30c8e754 Binary files /dev/null and b/jupyter-parameterized-kernel-spec/launcher.png differ diff --git a/jupyter-parameterized-kernel-spec/notebook-select-kernel.gif b/jupyter-parameterized-kernel-spec/notebook-select-kernel.gif new file mode 100644 index 00000000..0c73ab55 Binary files /dev/null and b/jupyter-parameterized-kernel-spec/notebook-select-kernel.gif differ