jiminy icon indicating copy to clipboard operation
jiminy copied to clipboard

[core] Remove all `...All()` methods from sensors and motors

Open duburcqa opened this issue 1 year ago • 1 comments

The data holders (SensorSharedDataHolder_t ...) should be refactored to be actual classes managing set of sensors, and as such, being friend of the corresponding abstract class (AbstractSensorBase ...). It should provide methods operating on all sensors in the holder at once (get, setOptions ...) and being renamed accordingly (SensorManager ...).

duburcqa avatar Mar 21 '23 17:03 duburcqa

Ideally, it should be possible to instantiate standalone sensors without shared storage, then update and fetch sensor measurement in isolation. The manager should only be helper that provides a bunch of optimised routines to process a ground a consistent sensors at once. It should not have to rely on private methods. It would be nice if the said manager could be completely independent from the shared storage but it actually comes together since it only makes sense if shared storage is enabled in the first place.

duburcqa avatar Feb 02 '24 07:02 duburcqa