mirror of
https://github.com/clearml/clearml
synced 2025-04-30 11:04:23 +00:00
Fix hyperparams with None value converted to empty string causes inferred type to change to str in consecutive connect() calls
This commit is contained in:
parent
6ed6c3ff70
commit
4f9aaa69ed
@ -1054,9 +1054,9 @@ class Task(IdObjectBase, AccessMixin, SetupUploadMixin):
|
|||||||
k = '{}/{}'.format(self._default_configuration_section_name, k)
|
k = '{}/{}'.format(self._default_configuration_section_name, k)
|
||||||
section_name, key = k.split('/', 1)
|
section_name, key = k.split('/', 1)
|
||||||
section = hyperparams.get(section_name, dict())
|
section = hyperparams.get(section_name, dict())
|
||||||
org_param = org_hyperparams.get(section_name, dict()).get(key, tasks.ParamsItem())
|
org_param = org_hyperparams.get(section_name, dict()).get(key, None)
|
||||||
param_type = params_types[org_k] if org_k in params_types else (
|
param_type = params_types[org_k] if org_k in params_types else (
|
||||||
org_param.type or (type(v) if v is not None else None)
|
org_param.type if org_param is not None else type(v) if v is not None else None
|
||||||
)
|
)
|
||||||
if param_type and not isinstance(param_type, str):
|
if param_type and not isinstance(param_type, str):
|
||||||
param_type = param_type.__name__ if hasattr(param_type, '__name__') else str(param_type)
|
param_type = param_type.__name__ if hasattr(param_type, '__name__') else str(param_type)
|
||||||
@ -1064,7 +1064,9 @@ class Task(IdObjectBase, AccessMixin, SetupUploadMixin):
|
|||||||
section[key] = tasks.ParamsItem(
|
section[key] = tasks.ParamsItem(
|
||||||
section=section_name, name=key,
|
section=section_name, name=key,
|
||||||
value=stringify(v),
|
value=stringify(v),
|
||||||
description=descriptions[org_k] if org_k in descriptions else org_param.description,
|
description=descriptions[org_k] if org_k in descriptions else (
|
||||||
|
org_param.description if org_param is not None else None
|
||||||
|
),
|
||||||
type=param_type,
|
type=param_type,
|
||||||
)
|
)
|
||||||
hyperparams[section_name] = section
|
hyperparams[section_name] = section
|
||||||
|
Loading…
Reference in New Issue
Block a user