Matthias Roels
01/08/2025, 1:22 PMdate
object was not a valid primitive type when creating OmegaConf objects. The issue seems to be when soft-merging params, you convert resolved dict objects back into OmegaConf objects which caused the error. Is this a bug?Hall
01/08/2025, 1:22 PMAnkita Katiyar
01/08/2025, 1:38 PMMatthias Roels
01/08/2025, 2:03 PMNok Lam Chan
01/08/2025, 3:00 PMDictConfig
to solve other problems as well. Though I think for this particularly problem we should remove the use of OmegaConf and keep the recursive merge. I haven't checked if it will break anything, I believe this was not added intentionally.Matthias Roels
01/08/2025, 3:48 PMMatthias Roels
01/08/2025, 3:52 PMDictConfig
obj internally in the getitem
function of OmegaConfLoaderMatthias Roels
01/08/2025, 3:53 PMNok Lam Chan
01/08/2025, 4:03 PMparameters
override works in the old day.Nok Lam Chan
01/08/2025, 4:14 PMNok Lam Chan
01/08/2025, 4:15 PM