ClearML/clearml
1
0
Fork 0
mirror of https://github.com/clearml/clearml synced 2025-01-31 17:17:00 +00:00
Code Issues Actions Packages Projects Releases Wiki Activity

Fixed argparser SUPPRESS as default should be resolved at remote execution the same way (i.e. empty string equals SUPPRESS)

Browse Source
This commit is contained in:
allegroai 2022-02-03 15:30:31 +02:00
parent e98e6acae1
commit 172c3e44f1
1 changed files with 4 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

5
clearml/backend_interface/task/args.py
View File

@ -248,7 +248,10 @@ class _Arguments(object):
# with the rest we have to make sure the type is correct
matched_actions = self._find_parser_action(parser, k)
for parent_parser, current_action in matched_actions:
if current_action and isinstance(current_action, _StoreConstAction):
if current_action and current_action.default == SUPPRESS and not v:
# this value should be kept suppressed, do nothing
v = SUPPRESS
elif current_action and isinstance(current_action, _StoreConstAction):
# make the default value boolean
# first check if False value
const_value = current_action.const if current_action.const is not None else (