-
Notifications
You must be signed in to change notification settings - Fork 0
Description
Because
In clamsproject/app-swt-detection#62 we discussed using a simple dictionary with classification results. This is allowed by the schema and we felt it would be a decent thing to do.
However, the add_property() method of the Mmif Annotation object does not allow this:
mmif-python/mmif/serialize/annotation.py
Lines 119 to 124 in 887c303
| if self.check_prop_value_is_simple_enough(value): | |
| self.properties[name] = value | |
| else: | |
| raise ValueError("Property values cannot be a complex object. It must be " | |
| "either string, number, boolean, None, or a list of them." | |
| f"(\"{name}\": \"{str(value)}\"") |
We should decide whether this was a good idea, my vote would be to update check_prop_value_is_simple_enough and allow dictionary with atomic values.
Done when
No response
Additional context
No response
Metadata
Metadata
Assignees
Labels
No labels
Type
Projects
Status
Done