Merge pull request #197 from NREL/failed_sim_bug_fix

Return elementary dict even when simulations fail
NREL · Mar 12, 2021 · 1cd0349 · 1cd0349
2 parents 9459728 + 27f4bc0
commit 1cd0349
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 0 deletions.
diff --git a/buildstockbatch/eagle.py b/buildstockbatch/eagle.py
@@ -258,6 +258,8 @@ def run_building_d(i, upgrade_idx):
                     txt = "\n" + "#" * 20 + "\n" + f"Traceback for building{i}\n" + txt
                     f.write(txt)
                     del txt
+                upgrade_id = 0 if upgrade_idx is None else upgrade_idx + 1
+                return {"building_id": i, "upgrade": upgrade_id}
 
         # Run the simulations, get the data_point_out.json info from each
         tick = time.time()

diff --git a/docs/changelog/changelog_dev.rst b/docs/changelog/changelog_dev.rst
@@ -36,3 +36,10 @@ Development Changelog
         :tickets: 164
 
         Removing master/slave language from AWS EMR configuration in project schema.
+
+    .. change::
+        :tags: bugfix
+        :pullreq: 197
+        :tickets: 196
+
+        Fixing issue where the postprocessing fails when a building simulation crashes in buildstockbatch.