search

BCDA-APS / bluesky_training

Bluesky training, including instrument package
https://bcda-aps.github.io/bluesky_training/
Other
11 stars 0 forks source link

Trim down the default attributes that came with area detector ... #182

Open prjemian opened 1 year ago

prjemian commented 1 year ago

@QZhang234 writes by email:

... Cam and CamBase classes so that they cover only the ABSOLUTE MINIMUM of attributes, such as acquire, file name, exposure time, etc. Adding attributes are far easier than blocking attributes and allows end-users to take responsibility/ownership for their decisions. In addition, unnecessary attributes lead to unexpected crashes; it is much easier to debug a class that is missing one necessary attribute that a user needs, than a class that has too many unnecessary attributes that is irrelevant to the use case of interest (e.g. Grasshopper rgb correction value for a Pilatus).

prjemian commented 1 year ago

See https://github.com/BCDA-APS/bdp_controls/issues/38 for a set of goals.