Bug Fix - Fix bug when convert bool config to store_true argument. (#120)

08d25bd2 · guoshzhao · GitHub · 5d6bc58d · 08d25bd2
Unverified Commit 08d25bd2 authored Jul 09, 2021 by guoshzhao Committed by GitHub Jul 09, 2021
Hide whitespace changes
Inline Side-by-side

Showing with 3 additions and 2 deletions

superbench/executor/executor.py superbench/executor/executor.py +3 -2

No files found.
--- a/superbench/executor/executor.py
+++ b/superbench/executor/executor.py
@@ -85,8 +85,9 @@ def __get_arguments(self, parameters):
        for name, val in parameters.items():
            if val is None:
                continue
-            if isinstance(val, bool) and val:
-                argv.append('--{}'.format(name))
+            if isinstance(val, bool):
+                if val:
+                    argv.append('--{}'.format(name))
            elif isinstance(val, (str, int, float)):
                argv.append('--{} {}'.format(name, val))
            elif isinstance(val, (list, ListConfig)):