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

Allow the dashboard builder to render code cells #7546

Open
maximlt opened this issue Dec 10, 2024 · 5 comments
Open

Allow the dashboard builder to render code cells #7546

maximlt opened this issue Dec 10, 2024 · 5 comments
Labels
type: enhancement Minor feature or improvement to an existing feature

Comments

@maximlt
Copy link
Member

maximlt commented Dec 10, 2024

The new Volumetric Imaging example (holoviz-topics/examples#454) added to examples.holoviz.org leverages the feature added in Panel 1.4 that allows serving a notebook that doesn't contain any .servable() call. The notebook is written in a way that implies the code is meant to be read too, however, Panel doesn't render the code cells at all. It'd be nice if there was a way to declare the code cells must also be rendered.

@maximlt maximlt added the type: enhancement Minor feature or improvement to an existing feature label Dec 10, 2024
@droumis
Copy link
Member

droumis commented Dec 10, 2024

What do you think about a cell tag? I imagine that the more common use case might be wanting to selectively include certain code cells. In the cases that someone wants to include ALL the code cells, it might be annoying but still tolerable..?

@maximlt
Copy link
Member Author

maximlt commented Dec 10, 2024

I suggested a toggle that enables displaying all the code cells as the workflow with the builder is roughly (IIUC): Serve notebook (e.g. via the preview) -> Rearrange/delete the Editable template -> Changes are saved as metadata in the notebook. I thought having to set a cell tag didn't fit much in that workflow. Users can manually delete the code cells they don't want to display.

@philippjfr
Copy link
Member

While they can delete the code cells, I'm not convinced it's a good experience to include them by default. Don't have a good suggestion for toggling the behavior though.

@maximlt
Copy link
Member Author

maximlt commented Dec 10, 2024

Yep I agree they shouldn't be included by default.

@philippjfr
Copy link
Member

What would be possible is to toggle the behavior using a URL query parameter (e.g. ?include_code=1 or whatever) and then add a checkbox to the JupyterLab extension that toggles the query parameter on and off.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type: enhancement Minor feature or improvement to an existing feature
Projects
None yet
Development

No branches or pull requests

3 participants