add i10-1 and i06-1magnet temperature controller

[Relm-Arrowny]

Fixes #2113, #2124

Requires: ophyd-async(#1381)

added a new generic temperature_controller:

• TemperatureController base class
• BaseTemperatureSensor and BaseHeater
• Shared PID component
• Temperature movable logic with configurable tolerance and stop behaviour
• Support for multiple sensor readbacks
• Support switchable readback control sensor.

Added support for the I06-1 SCM Cryocon M32 temperature controller:

• Heater configuration, ramp settings and PID controls

Added support for the I10-1 High Field Magnet temperature controllers:

• New HighFieldMagnetTemperatureController device

Fixed historical typo in module naming:

• temperture_controller → temperature_controller

Instructions to reviewer on how to test:

1. Do thing x
2. Confirm thing y happens

Checks for reviewer

• Would the PR title make sense to a scientist on a set of release notes
• If a new device has been added does it follow the standards
• If changing the API for a pre-existing device, ensure that any beamlines using this device have updated their Bluesky plans accordingly
• Have the connection tests for the relevant beamline(s) been run via dodal connect ${BEAMLINE}

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 99.17%. Comparing base (36e3b03) to head (089184b).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files

@@           Coverage Diff            @@
##             main    #2121    +/-   ##
========================================
  Coverage   99.16%   99.17%            
========================================
  Files         353      356     +3     
  Lines       13832    13988   +156     
========================================
+ Hits        13716    13872   +156     
  Misses        116      116            

☔ View full report in Codecov by Harness.
📢 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.

[oliwenmandiamond]

Thanks, looks good but have feedback to improve. As @Villtord mentioned in standup, could we please also make sure to add doc strings to the devices. Thanks!

[oliwenmandiamond]

Looks good, thanks. Some comments on class naming which I think needs to be updated to make support easier long term and movable_logic needs to be addressed.

[oliwenmandiamond]

Looks good, couple comments. Can we test to see if DeviceVector would work with it? If it doesn't work, happy to approve. It probably won't work but is a very natural fit for this and if find any issues with it we can raise with ophyd-async and then revisit this later

[oliwenmandiamond]

(We also need to up test coverage)

[oliwenmandiamond]

Thanks, this looks much better now. I have couple more comments and then once ophyd-async blocker added, looks great. Thank you!

[oliwenmandiamond]

I personally dislike this sensor_map. It looks quite awkward when you could just construct TemperatureSensor class outside this class and pass it directly in with the DeviceMap already configured.

[Relm-Arrowny]

This is the i10 sub class, we are efficiently doing what you are saying, think of it as a helper function so we do not have to do this x times in i10 config:

sensor = TemperatureSensor[HighFieldMagnetSensor](
    DeviceMap({
        "sensor1": BaseTemperatureSensor("BL01I-EA-HFM-01:STEMP"),
        "sensor2": BaseTemperatureSensor("BL01I-EA-HFM-01:STEMP2"),
        "sensor3": BaseTemperatureSensor("BL01I-EA-HFM-01:STEMP3"),
    })
)
hfm_temp = HighFieldMagnetTemperatureController(prefix="BL01I-EA-HFM-01:", sensor=sensor)