Merge pull request #408 from claritychallenge/407-pre-commit-raises-s…

…everal-errors-in-jupyter-notebooks Place imports at top of cell
claritychallenge · Sep 9, 2024 · b383605 · b383605
2 parents 9fcdf8b + ddb3b22
commit b383605
Show file tree

Hide file tree

Showing 3 changed files with 11 additions and 7 deletions.
diff --git a/notebooks/01_Installing_clarity_tools_and_using_metadata.ipynb b/notebooks/01_Installing_clarity_tools_and_using_metadata.ipynb
@@ -63,12 +63,13 @@
    },
    "outputs": [],
    "source": [
+    "import os\n",
+    "import sys\n",
+    "\n",
     "print(\"Changing directory...\")\n",
     "%cd clarity\n",
     "print(\"Installing Clarity tools\")\n",
     "%pip install -e .\n",
-    "import os\n",
-    "import sys\n",
     "\n",
     "sys.path.append(os.getcwd())\n",
     "print(\"Moving back to project root directory\")\n",

diff --git a/notebooks/02_Running_the_CEC2_baseline_from_commandline.ipynb b/notebooks/02_Running_the_CEC2_baseline_from_commandline.ipynb
@@ -84,15 +84,17 @@
    },
    "outputs": [],
    "source": [
+    "import os\n",
+    "import sys\n",
+    "\n",
+    "from IPython.display import clear_output\n",
+    "\n",
     "print(\"Cloning git repo...\")\n",
     "!git clone --quiet https://github.com/claritychallenge/clarity.git\n",
     "%cd clarity\n",
     "%pip install -e .\n",
-    "import os\n",
-    "import sys\n",
     "\n",
     "sys.path.append(f'{os.getenv(\"NBOOKROOT\")}/clarity')\n",
-    "from IPython.display import clear_output\n",
     "\n",
     "clear_output()\n",
     "print(\"Repository installed\")"

diff --git a/notebooks/03_Running_the_CEC2_baseline_from_python.ipynb b/notebooks/03_Running_the_CEC2_baseline_from_python.ipynb
@@ -48,13 +48,14 @@
    },
    "outputs": [],
    "source": [
+    "import os\n",
+    "import sys\n",
+    "\n",
     "print(\"Cloning git repo...\")\n",
     "\n",
     "!git clone --quiet https://github.com/claritychallenge/clarity.git\n",
     "%cd clarity\n",
     "%pip install -e .\n",
-    "import os\n",
-    "import sys\n",
     "\n",
     "sys.path.append(os.getcwd())\n",
     "%cd .."