Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

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

Open
prjemian opened this issue May 22, 2023 · 1 comment
Open

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

prjemian opened this issue May 22, 2023 · 1 comment
Labels
suggestion Something to consider

Comments

@prjemian
Copy link
Contributor

@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 prjemian added the suggestion Something to consider label May 22, 2023
@prjemian
Copy link
Contributor Author

See BCDA-APS/bdp_controls#38 for a set of goals.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
suggestion Something to consider
Projects
None yet
Development

No branches or pull requests

1 participant