Skip to content
This repository has been archived by the owner on Sep 22, 2020. It is now read-only.

Subclass CTRE's motor controllers classes #37

Open
katzuv opened this issue Jan 15, 2020 · 0 comments
Open

Subclass CTRE's motor controllers classes #37

katzuv opened this issue Jan 15, 2020 · 0 comments
Labels

Comments

@katzuv
Copy link
Member

katzuv commented Jan 15, 2020

Currently we have two "wrappers" for CTRE's motor controllers classes:

I recommend sub-classing the aforementioned classes and integrate those two features. That way you have to call those everywhere in your code. As time goes by, we can expand our classes and add more features as we see fit.

Note you'll probably need to subclass the three classes we use (VictorSPX, TalonSRX, TalonFX). I don't think this is something that could be done by subclassing BaseMotorController.

@katzuv katzuv added enhancement New feature or request optional labels Jan 15, 2020
@pklito pklito added the on hold label Feb 7, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

No branches or pull requests

2 participants