From d21f904735c802b9dc1900a81b290ff3700a05fa Mon Sep 17 00:00:00 2001 From: Emily KL <4672118+emilykl@users.noreply.github.com> Date: Mon, 30 Jun 2025 15:57:09 -0400 Subject: [PATCH 1/2] re-add ipython.get_ipython() check in _renderers.py --- plotly/io/_renderers.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/plotly/io/_renderers.py b/plotly/io/_renderers.py index 9ddd1db5dd..4c21760bb0 100644 --- a/plotly/io/_renderers.py +++ b/plotly/io/_renderers.py @@ -485,7 +485,7 @@ def show(fig, renderer=None, validate=True, **kwargs): ) default_renderer = env_renderer -elif ipython: +elif ipython and ipython.get_ipython(): # Try to detect environment so that we can enable a useful # default renderer if not default_renderer: From f141978a84d934bdef872292f2bd04224f7c7cd7 Mon Sep 17 00:00:00 2001 From: Emily KL <4672118+emilykl@users.noreply.github.com> Date: Mon, 30 Jun 2025 16:07:32 -0400 Subject: [PATCH 2/2] add test to ensure default renderer is 'browser' --- tests/test_io/test_renderers.py | 7 +++++++ 1 file changed, 7 insertions(+) diff --git a/tests/test_io/test_renderers.py b/tests/test_io/test_renderers.py index 4860c738a0..43dd250d8b 100644 --- a/tests/test_io/test_renderers.py +++ b/tests/test_io/test_renderers.py @@ -33,6 +33,13 @@ def fig1(request): ) +def test_default_renderer(): + """ + The default renderer should be 'browser'. + """ + assert pio.renderers.default == "browser" + + # JSON # ---- def test_json_renderer_mimetype(fig1):