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

Add an XDriveModel ctor that takes a gyro #2

Open
Octogonapus opened this issue Mar 28, 2018 · 5 comments
Open

Add an XDriveModel ctor that takes a gyro #2

Octogonapus opened this issue Mar 28, 2018 · 5 comments
Labels
enhancement New feature or request

Comments

@Octogonapus
Copy link
Member

Octogonapus commented Mar 28, 2018

This is Brad's suggestion (WPIlib code here) so you can drive around independent of heading. This has been tried in the past on the Cortex and it didn't work well enough to be used for a match. The gyro drifts too much. V5 would have to perform substantially better for this to work.

Alternatively, instead of taking a gyro object, we could do this the WPIlib way and just take the value of the heading so we aren't locked in to using a gyro. Then the heading measurement could come from a combination of quads & a gyro.

Gyro is waiting on Vex fixing the ADI port analog in sampling rate.

@Octogonapus Octogonapus changed the title Add ChassisController or ChassisModel ctor that takes a gyro Add an XDriveModel ctor that takes a gyro Mar 28, 2018
@Octogonapus
Copy link
Member Author

Taking a heading value is the way to go for this one. Going to keep this in the Freezer because we don't want to encourage bad designs like this one. The gyro VEX current sells just isn't good enough yet.

@Octogonapus Octogonapus added the enhancement New feature or request label Apr 18, 2018
@Octogonapus
Copy link
Member Author

Update: It looks like the ADI sampling rate will never be better than 5ms. Still leaving this in the freezer for now.

@HotelCalifornia
Copy link
Contributor

(oops)

@Octogonapus Octogonapus added question Further information is requested needs docs This issue needs documentation in the pros-docs repo. and removed question Further information is requested labels Jun 8, 2018
@baylessj
Copy link
Member

The gyro is actually usable on V5 as of a somewhat recent firmware update, so this could still be a possibility if anyone is interested. That being said, I doubt we'll see a lot of use of X drives this year.

@Octogonapus
Copy link
Member Author

I am going to keep this in the freezer unless people start specifically asking for it.

@Octogonapus Octogonapus removed the needs docs This issue needs documentation in the pros-docs repo. label Jan 13, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

3 participants