Add more structure to controllers

[jsouter]

Setting as a draft as I have not fixed the tests yet, and I may refactor to make it less convoluted.

• Fixes EigerHandler.put so that the responses it expects line up with what is provided by the real detector, not the tickit-devices sim (I will push a PR for my sim changes soon)
• EigerController now has a subcontroller for each of the subsystems (detector, stream, monitor)
• The detector subsystem controller has another subcontroller for threshold parameters
• The threshold subcontroller has extra logic to shorten the record names
• _create_attributes is now static, and allows the user to pass in methods to name and group the attributes to facilitate this
• Each subcontroller has reference to a new Subsystem class which now owns all the queue_update logic, and an attribute_mapping which maps the parameter key to attributes if they have a custom name or belong to a subcontroller of the subsystem controller.
• The top level subsystem controllers handle the update() logic and find attributes from their subcontrollers, which don't do much more than own their Attributes
• Each subsystem has its own stale parameter Attribute, and the top level controller also has one that is True when any of the subcontrollers is stale.

Fixes #49

[jsouter]

The detector subsystem screen and threshold subscreen pictured here

[codecov]

Codecov Report

Attention: Patch coverage is 67.74194% with 30 lines in your changes missing coverage. Please review.

Project coverage is 72.20%. Comparing base (42ee5ae) to head (1ab7a23).
Report is 10 commits behind head on main.

Files with missing lines	Patch %	Lines
src/fastcs_eiger/eiger_controller.py	67.74%	30 Missing ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##             main      #51      +/-   ##
==========================================
+ Coverage   68.82%   72.20%   +3.38%     
==========================================
  Files           4        4              
  Lines         263      295      +32     
==========================================
+ Hits          181      213      +32     
  Misses         82       82              

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

[jsouter]

Opening as tickit-devices changes have been merged and I've rewritten the tests to pass with the new structure

[GDYendell]

I have had a first look over and it looks like a good start, but I don't think the overall structure is quite right.

[GDYendell]

Looks much better now! Second pass review.

[GDYendell]

I am fine to ignore the codecov patch failure here as the project coverage is higher with this.

Thanks @jsouter! Are you happy to merge?

[jsouter]

I am fine to ignore the codecov patch failure here as the project coverage is higher with this.

Thanks @jsouter! Are you happy to merge?

Yep, think it's in a good state!