Add top level attributes and commands to simplify API

[GDYendell]

• Requires Add hinted attributes to use in detector drivers fastcs-odin#89

[codecov]

Codecov Report

❌ Patch coverage is 61.81818% with 21 lines in your changes missing coverage. Please review.
✅ Project coverage is 82.00%. Comparing base (1620928) to head (1979b0c).

Files with missing lines	Patch %	Lines
...cs_eiger/controllers/odin/eiger_odin_controller.py	31.25%	11 Missing ⚠️
...c/fastcs_eiger/controllers/odin/odin_controller.py	70.58%	5 Missing ⚠️
src/fastcs_eiger/__main__.py	0.00%	3 Missing ⚠️
src/fastcs_eiger/controllers/eiger_controller.py	71.42%	2 Missing ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##             main      #82      +/-   ##
==========================================
- Coverage   84.86%   82.00%   -2.87%     
==========================================
  Files          13       14       +1     
  Lines         403      450      +47     
==========================================
+ Hits          342      369      +27     
- Misses         61       81      +20     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

[shihab-dls]

Looks good! Left a couple of comments that are just enquiries; Will request changes just so that you can rerequest a review when codecov is satisfied.

[shihab-dls]

I'm assuming the acquisition_id must always be the same as the FP and MW file_prefixes. With these changes, one could set self.file_prefix after self.acquisition_id with a different name. At the top level, should we not just expose:

Suggested change

	self.file_prefix = AttrRW(
	String(),
	io_ref=ConfigFanAttributeIORef([self.FP.file_prefix, self.MW.file_prefix]),
	)
	self.acquisition_id = AttrRW(
	String(),
	io_ref=ConfigFanAttributeIORef(
	[
	self.file_prefix,
	self.FP.acquisition_id,
	self.MW.acquisition_id,
	self.EF.acqid,
	]
	),
	)
	self.acquisition_id = AttrRW(
	String(),
	io_ref=ConfigFanAttributeIORef(
	[
	self.FP.file_prefix,
	self.MW.file_prefix,
	self.FP.acquisition_id,
	self.MW.acquisition_id,
	self.EF.acqid,
	]
	),
	)

such that one should/could only poke one PV consistently?

[GDYendell]

There is not any real reason to set them differently, but I think some do. We could check that with some DAQ people.

[GDYendell]

@DominicOram do you have any thoughts on this?

[shihab-dls]

Could: Now that we have wait_for_value with a timeout should we expose a DEFAULT_TIMEOUT in FastCS, such that we can ensure observation timeouts are consistent unless they explicitly shouldn't be?

[GDYendell]

I am not sure there is a widely useful default - what else would you use that default for?

I am actually not sure if 1 second here is sufficient. Perhaps it should be an Attribute so it can be changed at runtime?