Merge pull request #307 from OscarVanL/zerodivisionfix

dathudeptrai · web-flow · commit 73cac7fc06aa · 2020-10-15T17:13:58.000+07:00
Fixed ZeroDivisionError when mfa_longer list is empty
diff --git a/examples/mfa_extraction/fix_mismatch.py b/examples/mfa_extraction/fix_mismatch.py
@@ -105,7 +105,7 @@ def fix(base_path: str, dur_path: str, trimmed_dur_path: str, use_norm: str):
 
         logging.info(
             f"{t} stats: number of mfa with longer duration: {len(mfa_longer)}, total diff: {sum(mfa_longer)}"
-            f", mean diff: {sum(mfa_longer)/len(mfa_longer)}"
+            f", mean diff: {sum(mfa_longer)/len(mfa_longer) if len(mfa_longer) > 0 else 0}"
         )
         logging.info(
             f"{t} stats: number of mfa with shorter duration: {len(mfa_shorter)}, total diff: {sum(mfa_shorter)}"

Original file line number	Diff line number	Diff line change
`@@ -105,7 +105,7 @@ def fix(base_path: str, dur_path: str, trimmed_dur_path: str, use_norm: str):`
`105`	`105`
`106`	`106`	`logging.info(`
`107`	`107`	`f"{t} stats: number of mfa with longer duration: {len(mfa_longer)}, total diff: {sum(mfa_longer)}"`
`108`		`- f", mean diff: {sum(mfa_longer)/len(mfa_longer)}"`
	`108`	`+ f", mean diff: {sum(mfa_longer)/len(mfa_longer) if len(mfa_longer) > 0 else 0}"`
`109`	`109`	`)`
`110`	`110`	`logging.info(`
`111`	`111`	`f"{t} stats: number of mfa with shorter duration: {len(mfa_shorter)}, total diff: {sum(mfa_shorter)}"`