Hotfix admit comma separated keys

[ferranAX]

No description provided.

[donato-ax]

Why do we have to hard-code inside the function that the keys have to be port pairs? This is unnecessarily constraining the use of this function, no?

[ferranAX]

Why do we have to hard-code inside the function that the keys have to be port pairs? This is unnecessarily constraining the use of this function, no?

Yes. In theory we should only admit dictionaries with tuples of strings as input, as it is the output of our sax spectrum endpoint as well as the standard output of sax models. However, in serialization, the SDK converted the tuple[str, str] to a comma separated string. For this reason we implement this feature and to standardize the labels.

[donato-ax]

Why do we have to hard-code inside the function that the keys have to be port pairs? This is unnecessarily constraining the use of this function, no?

Yes. In theory we should only admit dictionaries with tuples of strings as input, as it is the output of our sax spectrum endpoint as well as the standard output of sax models. However, in serialization, the SDK converted the tuple[str, str] to a comma separated string. For this reason we implement this feature and to standardize the labels.

@ferranAX why did the sdk do that? then there is something wrong with the sdk

[ferranAX]

Why do we have to hard-code inside the function that the keys have to be port pairs? This is unnecessarily constraining the use of this function, no?

Yes. In theory we should only admit dictionaries with tuples of strings as input, as it is the output of our sax spectrum endpoint as well as the standard output of sax models. However, in serialization, the SDK converted the tuple[str, str] to a comma separated string. For this reason we implement this feature and to standardize the labels.

@ferranAX why did the sdk do that? then there is something wrong with the sdk

Indeed, but there is a presentation today. I believe it's best to get this, even if it's just a temporary fix.