If Dataset.add_provenance is called multiple times with the same prov properties, it raises ValueError. Since the actual key to be inserted in common_props is computed within the method, checking whether the key already exists cannot be done easily from the caller. Thus, add_provenance should - at least optionally - merge data for the same key in common_props.
If
Dataset.add_provenanceis called multiple times with the sameprovproperties, it raisesValueError. Since the actual key to be inserted incommon_propsis computed within the method, checking whether the key already exists cannot be done easily from the caller. Thus,add_provenanceshould - at least optionally - merge data for the same key incommon_props.