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

Adding PigeonViaTalonSRX #292

Open
wants to merge 12 commits into
base: dev
Choose a base branch
from

Conversation

konnorreynolds
Copy link

encoder type is : pigeon_via_talonsrx
The can id should be that of the talon that it is attached to.
My local has a function to get the talon so it can be used without being created again.
Trying to work on a thrown error that tells the user the can id should be that of the talon's when the object is not created correctly.

Remade, it was using my main. Thx WispySparks for letting me know.

No TalonSRX detection to attempt to see if the user has made a fault.

I had an idea like below but don't like it.

enableVoltageComp
if (!voltageComp) {
  DriverStation.Error
}   else {
   disableVoltageComp
}
Forgot to pull the can id into the class call lol.
@thenetworkgrinch
Copy link
Contributor

Waiting on testing for this

Added some basic elevator and shooter code and for now just a spark max controller.
This reverts commit 5e9fb6e.
Added roughly by copying the existing SparkMaxEncoderSwerve and making very minor tweaks under the assumption sparkflex and max are similar enough as I have not used them.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants