ClearML/clearml
1
0
Fork 0
mirror of https://github.com/clearml/clearml synced 2025-02-07 13:23:40 +00:00
Code Issues Actions Packages Projects Releases Wiki Activity

Fix Task.connect_configuration() removes trailing + sign from configuration key

Browse Source
This commit is contained in:
allegroai 2023-07-10 11:14:09 +03:00
parent 13f11a68be
commit c394046a2d
1 changed files with 17 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

17
clearml/utilities/config.py
View File

@ -61,6 +61,23 @@ def config_dict_to_text(config):
try:
# noinspection PyBroadException
try:
def raise_on_special_key(config_):
if not isinstance(config_, dict):
return
special_chars = "$}[]:=+#`^?!@*&."
for key in config_.keys():
if not isinstance(key, str):
continue
if any(key_char in special_chars for key_char in key):
raise ValueError(
"Configuration dictionary keys cannot contain any of the following characters: {}".format(
special_chars
)
)
for val in config_.values():
raise_on_special_key(val)
# will fall back to json+pyhocon
raise_on_special_key(config)
text = HOCONConverter.to_hocon(ConfigFactory.from_dict(hocon_quote_key(config)))
except Exception:
# fallback json+pyhocon