epics-containers / pvi

EPICS PV Interface described in YAML
https://epics-containers.github.io/pvi
Apache License 2.0
4 stars 2 forks source link

Improve Device validation error messages #99

Closed GDYendell closed 5 months ago

GDYendell commented 7 months ago

Currently when validation fails for a Device with parents it is not clear which yaml file the failure is in.

We could catch the error in Device.deserialize and add some context, but this would be too late to tell which yaml file the problem is in because it is all loaded into one dictionary at this point. Perhaps Device.deserialize_parents should validate each as it is loaded and fail at that point.

gilesknap commented 7 months ago

I approve this message.