Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

AdiGyro and AdiServo Device Support #48

Open
Tropix126 opened this issue Apr 17, 2024 · 0 comments
Open

AdiGyro and AdiServo Device Support #48

Tropix126 opened this issue Apr 17, 2024 · 0 comments
Labels
adi-devices Relates to three-wire ADI devices. enhancement New feature or request

Comments

@Tropix126
Copy link
Member

What's the motivation for this feature?

Both of these devices are ones i've been avoiding for a while for various reasons, but we need to support them at some point.

Describe the solution you'd like

  • Add an AdiGyro support for abstracting over the old 3-wire yaw-rate gyro.
  • Add an AdiServo struct for controlling the legacy cortex-era servos.

Describe the drawbacks, if any

Both of these devices are very poorly documented in the SDK and may utilize private API functionality on VEXCode's side. Not sure.

  • In AdiGyro's case, VEXCode has an is_calibrating function while PROS does not. There's no provided function in the SDK for this functionality, meaning it's probably some undocumented status bit. Fun.
  • PROS also is forced into immediately calibrating the sensor as soon as vexDeviceAdiPortConfigSet is called.
  • In AdiServo's case, PROS doesn't even support it at all. The units/range of the ADI setter is comlpetely unknown.

I also don't have access to any of this hardware to test on.

@Tropix126 Tropix126 added enhancement New feature or request smart-devices Relates to a smartport device. adi-devices Relates to three-wire ADI devices. and removed smart-devices Relates to a smartport device. labels Apr 17, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
adi-devices Relates to three-wire ADI devices. enhancement New feature or request
Projects
Status: Ready
Development

No branches or pull requests

1 participant