DiamondLightSource / DiamondLightSource/dodal

Consider MixIns for common device behaviours

Open
#1,353 1 comment 0 reactions 0 assignees View on GitHub
question
Dominant language
Python
Stars
5
Forks
13
Avg merge
3d 2h
Merged PRs (30d)
10

Description

#1277 unified some stages with common physical axes, meaning plans could potentially be re-used between them.
#1339 adds a new TemperatureController - the existing Linkam device is also a TemperatureController, it could be useful to mark these devices with a MixIn or marker interface to allow plans to be written in a generic way to use either but not so generic as to accept a `Movable[float]`?

This may provide a way forward also for other common scan axes (e.g. wavelength, energy)

## Acceptance Criteria
* A discussion about design of plans has taken place and the conclusion is added to this ticket

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.