allwpilib icon indicating copy to clipboard operation
allwpilib copied to clipboard

Split DutyCycleEncoder/AnalogEncoder into 2 classes

Open ThadHouse opened this issue 1 year ago • 1 comments

DutyCycleEncoder as it currently exists is generally the wrong choice for users. It adds rollover support in addition to offset and conversion functionalities. But the rollover support can sometimes be problematic, and can create issues when the user doesn't need it. And generally the user doesn't need rollover support. We should move the rollover support into its own class.

ThadHouse avatar Jan 16 '24 16:01 ThadHouse

Not sure about DutyCycle, but isn't AnalogPotentiometer the class the user usually wants?

Starlight220 avatar Jan 17 '24 05:01 Starlight220

Closed by #6398

ThadHouse avatar Aug 20 '24 06:08 ThadHouse