clearml icon indicating copy to clipboard operation
clearml copied to clipboard

fix config parsing for the case of empty string keys

Open radugrosu opened this issue 1 year ago • 1 comments

Patch Description

Solves this bug:

from clearml.utilities.pyhocon.config_parser import ConfigFactory
ConfigFactory.from_dict({"": 1})

fails with IndexError: list index out of range.

radugrosu avatar May 21 '23 22:05 radugrosu

Hi @radugrosu,

As however this is part of the tokenizer, I'm worried there might be side-effects - since this is part of pyhocon, and the code was not changed there, I suggest contributing this there, and than, once they approve it, we can do it here as well.

WDYT?

jkhenning avatar May 29 '23 08:05 jkhenning