Merge pull request The-OpenROAD-Project#3305 from The-OpenROAD-Project-staging/err-check-gen-elapsed

maliberty · web-flow · commit eb2be3aae958 · 2025-07-09T19:29:38.000Z
Check for None in computing totalElapsed and total_max_memory
diff --git a/flow/util/genElapsedTime.py b/flow/util/genElapsedTime.py
@@ -105,8 +105,10 @@ def print_log_dir_times(logdir, args):
                     odb_hash[0:20],
                 )
             )
-        totalElapsed += elapsedTime
-        total_max_memory = max(total_max_memory, int(peak_memory))
+        if elapsedTime is not None:
+            totalElapsed += elapsedTime
+        if peak_memory is not None:
+            total_max_memory = max(total_max_memory, int(peak_memory))
 
     if totalElapsed != 0 and not args.match:
         print(format_str % ("Total", totalElapsed, total_max_memory, ""))

Original file line number	Diff line number	Diff line change
`@@ -105,8 +105,10 @@ def print_log_dir_times(logdir, args):`
`105`	`105`	`odb_hash[0:20],`
`106`	`106`	`)`
`107`	`107`	`)`
`108`		`- totalElapsed += elapsedTime`
`109`		`- total_max_memory = max(total_max_memory, int(peak_memory))`
	`108`	`+ if elapsedTime is not None:`
	`109`	`+ totalElapsed += elapsedTime`
	`110`	`+ if peak_memory is not None:`
	`111`	`+ total_max_memory = max(total_max_memory, int(peak_memory))`
`110`	`112`
`111`	`113`	`if totalElapsed != 0 and not args.match:`
`112`	`114`	`print(format_str % ("Total", totalElapsed, total_max_memory, ""))`