Skip to content
This repository has been archived by the owner on Oct 1, 2023. It is now read-only.

Support for duty cycle encoders #433

Closed
CollectivelyCode opened this issue May 24, 2022 · 2 comments
Closed

Support for duty cycle encoders #433

CollectivelyCode opened this issue May 24, 2022 · 2 comments
Labels
component: generation Generation of robot projects

Comments

@CollectivelyCode
Copy link

I've noticed that duty cycle is not supported. Would it be possible to have support for such?
@calcmogul
Copy link
Member

calcmogul commented May 24, 2022 •
edited
Loading

SysId requires a velocity measurement, but DutyCycleEncoder doesn't provide one. I suggest making an issue to add DutyCycleEncoder::GetRate() first. It would probably have to use a backward finite difference. SysId could add code to do that itself, but it's not as clean.

@HoJo42 HoJo42 mentioned this issue Aug 30, 2023
Closed
@calcmogul calcmogul added the component: generation Generation of robot projects label Sep 20, 2023
@calcmogul
Copy link
Member

OBE by #518.

@calcmogul calcmogul closed this as not planned Won't fix, can't repro, duplicate, stale Sep 21, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
component: generation Generation of robot projects
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@calcmogul @CollectivelyCode