pcdshub / happi

Heuristic Access to Positions of Photon Instruments
https://pcdshub.github.io/happi/master
Other
13 stars 29 forks source link

ENH: add name = _id fix to `happi repair` #302

Closed tangkong closed 1 year ago

tangkong commented 1 year ago

Expected Behavior

happi repair should also check and align an item's name with their _id.

The rationale behind this should also be documented once we agree on it, since we keep questioning why we hold this convention. a quick trawl through issues / pr's shows the reason to be...?

Some quick bits of conversation:

There are probably other checks to add, maybe they get lumped in with this?

Current Behavior

happi repair only checks that all mandatory information have values.

Possible Solution

Additional code block in the happi repair command

Context

brought up in #300

Your Environment

pcds-5.6.0