work on output clearing

notebooks/clear_all_notebooks.py

@@ -1,5 +1,7 @@
 import os
 import pathlib as pl
+import json
+
 notebook_count = 0
 # add the basenames of any notebooks we want to keep the results of 
 # all solutions are still not being cleared
@@ -14,12 +16,27 @@
     '10_modpath-demo.ipynb'
     ]
 
+def check_nb_for_output(notebook):
+    has_output = False
+    # run autotest on each notebook
+    with open(notebook) as src:
+        data = src.read()
+        data = json.loads(data)
+        for item in data['cells']:
+            for k, v in item.items():
+                if item['cell_type'] == 'code':
+                    if any (item['outputs']):
+                        has_output = True
+    return has_output        
+
 for [path, subdirs, files] in os.walk('.'):
     for cf in files:
         if cf.lower().endswith('.ipynb') and '.ipynb_checkpoint' not in path:
             nb = pl.Path(path) / cf
-            if 'solutions' not in str(nb) and nb.name not in skip_notebooks:
-                print(f"clearing {nb}")
-                os.system(f"jupyter nbconvert --clear-output --inplace {nb._str}")
-                notebook_count += 1
+            if ('solutions' not in str(nb) and 
+                nb.name not in skip_notebooks and 
+                check_nb_for_output(nb) is True):
+                    print(f"clearing {nb}")
+                    os.system(f"jupyter nbconvert --clear-output --inplace {nb._str}")
+                    notebook_count += 1
 print(notebook_count," notebooks cleared")

tests/test_notebook_output.py

@@ -49,6 +49,6 @@ def test_notebook_output(notebook):
         for item in data['cells']:
             for k, v in item.items():
                 if item['cell_type'] == 'code':
-                    assert not any (item['outputs']), (f"{notebook} as output "
+                    assert not any (item['outputs']), (f"{notebook} has output "
                                 "but is not listed in notebooks/clean_all_notebooks.py "
                                 "as excepted!")

tests/test_notebooks.py

@@ -13,6 +13,7 @@
     '06_matplotlib.ipynb': 'notebook intentionally incomplete',
     '00_python_basics_review__solutions.ipynb': 'intentional error',
     '01_functions_script__solution.ipynb': 'intentional error',
+    '06-Project-structured.ipynb': 'intentionally incomplete information'
 
 
 }