Merge pull request #155 from mwcraig/little-fixes

A few little fixes for issues identified by the loads/images checker

Author: mwcraig

notebooks/02.Interact/02.00-Using-Interact.ipynb

@@ -297,7 +297,7 @@
    "metadata": {},
    "outputs": [],
    "source": [
-    "# %load solutions/interact-basic-list/reverse-text.py\n"
+    "# %load solutions/reverse-text.py\n"
    ]
   },
   {
@@ -435,7 +435,7 @@
    "metadata": {},
    "outputs": [],
    "source": [
-    "# %load solutions/interact-basic-list/plot-function.py"
+    "# %load solutions/plot-function.py"
    ]
   },
   {

notebooks/images/WidgetModelView.graffle renamed to notebooks/03.Basics/images/WidgetModelView.graffle

@@ -0,0 +1,43 @@
+"""
+Iterate through and execute all notebooks
+"""
+from collections import defaultdict
+from pathlib import Path
+import random
+import re
+
+from execute_notebooks import print_exceptions
+
+
+def main():
+    p = Path('.')
+    notebook_paths = p.glob('**/*.ipynb')
+    patterns = dict(
+        loads=re.compile(r'%load ([^\s]+\.py)'),
+        images=re.compile(r'\((images/[^\s]+(?:png|jpg|jpeg))\)')
+    )
+    exceptions = defaultdict(list)
+
+    for nb_path in notebook_paths:
+        if '.ipynb_checkpoints' in str(nb_path):
+            continue
+        print(nb_path)
+        with open(nb_path) as f:
+            contents = f.read()
+        for name, pattern in patterns.items():
+            print(f'Checking {name}...')
+            matches = pattern.findall(contents)
+            for match in matches:
+                print(f'\t{match}')
+                p = nb_path.parent / match
+                try:
+                    with open(p) as _:
+                        pass
+                except FileNotFoundError as e:
+                    exceptions[str(nb_path)].append(str(e))
+
+    print_exceptions(exceptions)
+
+
+if __name__ == '__main__':
+    main()

notebooks/images/WidgetModelView.png renamed to notebooks/03.Basics/images/WidgetModelView.png

@@ -1,16 +1,34 @@
 """
 Iterate through and execute all notebooks
 """
+from collections import defaultdict
 from pathlib import Path
+import random
 
 import nbformat as nbf
 from nbconvert.preprocessors import ExecutePreprocessor
 
 
+def print_exceptions(exceptions):
+    if exceptions:
+        print('⚠️' * 20)
+        print('\n\nFailures in these notebooks:')
+        print('\n'.join(exceptions.keys()), '\n\n')
+        print('⚠️' * 20)
+        for nb, error_list in exceptions.items():
+            face = random.choice(["😮", "😱", "🤬"])
+            print(face * 20)
+            errors = "\n".join(error_list)
+            print(f'{nb}: \n{errors}')
+            print(face * 20)
+    else:
+        print('🎉🎉 Everything worked! 🎉🎉')
+
+
 def main():
     p = Path('.')
     notebook_paths = p.glob('**/*.ipynb')
-    exceptions = {}
+    exceptions = defaultdict(list)
     for nb_path in notebook_paths:
         if '.ipynb_checkpoints' in str(nb_path):
             continue
@@ -22,17 +40,9 @@ def main():
             ep.preprocess(notebook)
             print('Done executing')
         except Exception as e:
-            exceptions[str(nb_path)] = e
+            exceptions[str(nb_path)].append(str(e))
+    print_exceptions(exceptions)
 
-    if exceptions:
-        print('⚠️' * 20)
-        print('\n\nFailures in these notebooks:')
-        print('\n'.join(exceptions.keys()), '\n\n')
-        print('⚠️' * 20)
-        for nb, error in exceptions.items():
-            print(f'{nb}: \n{error}')
-    else:
-        print('🎉🎉 Everything worked! 🎉🎉')
 
 
 if __name__ == '__main__':