Closed DrKittyKat closed 4 years ago
you want to get Caiden to? he has a mr also that needs same addressing
On Mon, Feb 3, 2020, 4:19 PM DrKittyKat notifications@github.com wrote:
@DrKittyKat commented on this pull request.
In motorHelper.py https://github.com/Raptacon/Robot-2020/pull/49#discussion_r374397083:
- if 'currentLimits' in motorDescp:
- currentLimits = motorDescp['currentLimits']
- absMax = currentLimits['absMax']
- absMaxTimeMs = currentLimits['absMaxTimeMs']
- nominalMaxCurrent = currentLimits['maxNominal']
- motor.configPeakCurrentLimit(absMax,10)
- motor.configPeakCurrentDuration(absMaxTimeMs,10)
- motor.configContinuousCurrentLimit(nominalMaxCurrent,10)
- motor.enableCurrentLimit(True)
if 'rampRate' in motorDescp:
motor.configOpenLoopRamp(motorDescp['rampRate'],10)
- return motor
+class WPI_TalonFeedback(ctre.WPI_TalonSRX):#ctre.wpi_talonsrx.WPI_TalonSRX
I didn't write motorHelper, but I'll try to document it
— You are receiving this because you commented. Reply to this email directly, view it on GitHub https://github.com/Raptacon/Robot-2020/pull/49?email_source=notifications&email_token=AA7EHEHIZMRE4FAMALFUNBTRBCRA3A5CNFSM4KOVYUHKYY3PNVWWK3TUL52HS4DFWFIHK3DMKJSXC5LFON2FEZLWNFSXPKTDN5WW2ZLOORPWSZGOCUCL3GA#discussion_r374397083, or unsubscribe https://github.com/notifications/unsubscribe-auth/AA7EHEH6X6IQR2XGY3UILL3RBCRA3ANCNFSM4KOVYUHA .
Basic drive train set up in "driveTrain.py" component. I believe it should replace "dtFxTest.py", but I left it in for now because I'm not sure.