Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

glk0
Copy link
Contributor

@glk0 glk0 commented Jul 18, 2025

Related

Old signature of inialization of Viewer class:

def __init__(
self,
*,
width: int | None = None,
height: int | None = None,
url: str | None = None,
panel_states: Mapping[Panel, PanelState] | None = None,
**kwargs: Any,
) -> None:

The new signature
def __init__(
self,
*,
width: int | Literal["auto"],
height: int | Literal["auto"],
url: str | None = None,
panel_states: Mapping[Panel, PanelState] | None = None,
fallback_token: str | None = None,
**kwargs: Any,
) -> None:

What

Fixes rerun_notebook/example.ipynb breaking due to reliance on older signature.

Copy link
Member

@ntjohnson1 ntjohnson1 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for contributing for the first time and for raising the other issue!

@ntjohnson1 ntjohnson1 merged commit 6b4dc41 into rerun-io:main Jul 20, 2025
40 of 41 checks passed
@emilk emilk changed the title fix breaking example in rerun_notebook/example.ipynb Fix breaking example in rerun_notebook/example.ipynb Aug 7, 2025
@emilk emilk changed the title Fix breaking example in rerun_notebook/example.ipynb Fix breaking example in rerun_notebook/example.ipynb Aug 7, 2025
@emilk emilk mentioned this pull request Aug 7, 2025
9 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🪳 bug Something isn't working include in changelog notebook Jupyter notebooks etc
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants