Skip to content

GitLab

Unverified Commit 05e449a3 authored by guoshzhao's avatar guoshzhao Committed by GitHub
Browse files

Bug bash - Fix ambiguous type check in executor. (#104)

parent e7b6af35
Hide whitespace changes
Inline Side-by-side
Showing with 3 additions and 3 deletions
superbench/executor/executor.py
View file @ 05e449a3
...@@ -84,12 +84,12 @@ def __get_arguments(self, parameters): ...@@ -84,12 +84,12 @@ def __get_arguments(self, parameters):
for name, val in parameters.items(): for name, val in parameters.items():
if val is None: if val is None:
continue continue
if isinstance(val, (str, int, float)): if isinstance(val, bool) and val:
argv.append('--{}'.format(name))
elif isinstance(val, (str, int, float)):
argv.append('--{} {}'.format(name, val)) argv.append('--{} {}'.format(name, val))
elif isinstance(val, (list, ListConfig)): elif isinstance(val, (list, ListConfig)):
argv.append('--{} {}'.format(name, ' '.join(val))) argv.append('--{} {}'.format(name, ' '.join(val)))
elif isinstance(val, bool) and val:
argv.append('--{}'.format(name))
return ' '.join(argv) return ' '.join(argv)
def __exec_benchmark(self, context, log_suffix): def __exec_benchmark(self, context, log_suffix):
......
Markdown is supported
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment