Page MenuHomePhabricator

Refactor Claron tracking classes
Closed, WontfixPublic

Description

Get rid of -stub and -interface classes, use threading

Event Timeline

Threading is already used in the class ClaronTrackingDevice. The stub- and interface class was implemented to be hardware independent. If the Microntracker is not installed in CMake the stub class is used and you can test classes above without the driver or hardware.

This design was discussed with Marco before integrating the MicronTracker. The same design was later used for the hardware dependent ToF classes for example.

So I don't think it is nessesary to change this. If you agree you can close this bug.

The design differs from other TrackingDevice classes. Intention of this ticket was to unify the design and therefore improve maintainability. This would be nice, but is not mandatory. If no one wants to do it, we can keep the current implementation.

kislinsk added a project: Bulk Edit.
kislinsk removed a project: Bulk Edit.