fix plotly rendering

fneum · Feb 20, 2023 · 84461be · 84461be
1 parent 671d57d
commit 84461be
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 12 deletions.
diff --git a/data-science-for-esm/13-workshop-interactive-visualisation.ipynb b/data-science-for-esm/13-workshop-interactive-visualisation.ipynb
@@ -496,28 +496,20 @@
    "metadata": {},
    "outputs": [],
    "source": [
-    "import plotly.express as px"
+    "import plotly.io as pio\n",
+    "import plotly.express as px\n",
+    "import plotly.offline as py"
    ]
   },
   {
    "cell_type": "markdown",
    "metadata": {},
    "source": [
     ":::{note}\n",
-    "We need to set a non-default renderer, so that the interactive plots are also visible on the course's static website.\n",
+    "We need to import `plotly.io` and `plotly.offline`, so that the interactive plots are also visible on the course's static website.\n",
     ":::"
    ]
   },
-  {
-   "cell_type": "code",
-   "execution_count": null,
-   "metadata": {},
-   "outputs": [],
-   "source": [
-    "import plotly.io as pio\n",
-    "pio.renderers.default = \"notebook\""
-   ]
-  },
   {
    "cell_type": "markdown",
    "metadata": {},

diff --git a/data-science-for-esm/_config.yml b/data-science-for-esm/_config.yml
@@ -47,3 +47,8 @@ parse:
     - linkify
     - colon_fence
     - substitution
+
+sphinx:
+  config:
+    html_js_files:
+    - https://cdnjs.cloudflare.com/ajax/libs/require.js/2.3.4/require.min.js