fix bug where the presence of skip_financial caused the skip, rather …

…than the value
NREL · Oct 3, 2024 · ac73efb · ac73efb
1 parent 5852b2d
commit ac73efb
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 2 deletions.
diff --git a/hopp/simulation/hybrid_simulation.py b/hopp/simulation/hybrid_simulation.py
@@ -903,7 +903,7 @@ def _aggregate_financial_output(self, name, start_index=None, end_index=None) ->
         out = self.outputs_factory.create()
         for k, v in self.technologies.items():
             if k in self.sim_options.keys():
-                if 'skip_financial' in self.sim_options[k].keys():
+                if 'skip_financial' in self.sim_options[k].keys() and self.sim_options[k]['skip_financial']:
                     continue
             val = getattr(v, name)
             if start_index and end_index:

diff --git a/tests/hopp/test_custom_financial.py b/tests/hopp/test_custom_financial.py
@@ -303,9 +303,17 @@ def test_hybrid_simple_pv_with_wind_storage_dispatch(site, subtests):
             'ppa_price': 0.03
         }
     }
+    config = {
+        "simulation_options": {
+            "wind": {
+                "skip_financial": False # test that setting this to false allows financial calculations to run
+            }
+        }
+    }
     hopp_config = {
         "site": site,
-        "technologies": power_sources
+        "technologies": power_sources,
+        "config": config
     }
     hi = HoppInterface(hopp_config)
     hybrid_plant = hi.system