Add excel optional dependency

.github/workflows/examples.yml

@@ -52,7 +52,7 @@ jobs:
         run: |
           sudo apt update
           sudo apt install -y graphviz r-base
-          pip install .[test]
+          pip install .[test,excel]
 
       - name: Run example
         run: |

examples/Eijgenraam/eijgenraam_mordm.py

@@ -5,7 +5,6 @@
 import functools
 import numpy as np
 import matplotlib.pyplot as plt
-from matplotlib.colors import LogNorm
 from rhodium import * 
 
 ##==============================================================================

pyproject.toml

@@ -18,6 +18,7 @@ dependencies = [
   "mplcursors<=0.5.3",
   "pydot<=2.0.0",
   "SALib<=1.4.6.1",
+  "pandas",
   "platypus-opt",
   "prim"
 ]
@@ -30,6 +31,7 @@ dynamic = ["version"]  # Version is read from rhodium/__init__.py
 [project.optional-dependencies]
 test = ["pytest", "mock", "pyper<=1.1.2"]
 openmdao = ["openmdao"]
+excel = ["pandas[excel]"]
 
 [tool.setuptools.dynamic]
 version = {attr = "rhodium.__version__"}

rhodium/classification.py

@@ -191,11 +191,6 @@ def __str__(self):
         return self._to_string()
 
     def print_tree(self, coi=None, all=True, **kwargs):
-        # Being unfamiliar with version control, I just added "#", or comments, in places where I made changes to the original Rhodium code.
-        # Below is the unaltered (although commented out):
-        # print(self._to_string(coi, all, **kwargs))
-
-        # But it does not return a list of the scenario nodes, which we would like. So I made alterations to the _to_string() function to achieve this:
         results, node_list = self._to_string(coi, all, **kwargs)
         print(results)
         return node_list

rhodium/model.py

@@ -440,8 +440,6 @@ def __getitem__(self, key):
             return self._data[key]
 
     def __setitem__(self, key, value):
-        print(key)
-        print(value)
         if not isinstance(value, self.type):
             raise TypeError("can only add " + self.type.__name__ + " objects")