port tunableTalonFX #2

Open
me-it-is wants to merge 5 commits from tunable-talonfx into main
Member

Port the tunableTalonFX class from 2026_Season to use WPILibs new Tunable

Port the `tunableTalonFX` class from 2026_Season to use WPILibs new `Tunable`
@ -0,0 +37,4 @@
private final double previousSupplyCurrentLimit;
private final double previousStatorCurrentLimit;
private final TunableDouble motionMagicCruiseVelocity;
Owner

TunableDouble and your old TunableNumber are not equivalent. There's no way to set a default or use a tuning mode. TunableDouble is inherently transient and lives entirely on the NetworkTables database. There's no way to write or save values consistently. You may need to subclass or do something else to get equivalent behavior. I recommend not porting this until a later date.

`TunableDouble` and your old `TunableNumber` are not equivalent. There's no way to set a default or use a tuning mode. `TunableDouble` is inherently transient and lives entirely on the NetworkTables database. There's no way to write or save values consistently. You may need to subclass or do something else to get equivalent behavior. I recommend not porting this until a later date.
This pull request doesn't have enough approvals yet. 0 of 1 approvals granted.
You are not authorized to merge this pull request.
View command line instructions

Checkout

From your project repository, check out a new branch and test the changes.
git fetch -u origin tunable-talonfx:tunable-talonfx
git switch tunable-talonfx
Sign in to join this conversation.
No reviewers
No labels
No milestone
No project
No assignees
2 participants
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set

Reference
chainlynx/2026_SystemCore!2
No description provided.