Skip to content

Update SysId step voltage guidance - #3356

Open
TLNing260310 wants to merge 1 commit into
wpilibsuite:mainfrom
TLNing260310:fix-7794-lower-sysid-step-voltage
Open

TLNing260310 wants to merge 1 commit into
wpilibsuite:mainfrom
TLNing260310:fix-7794-lower-sysid-step-voltage

Conversation

@TLNing260310

Copy link
Copy Markdown

Summary

  • update the documented SysId dynamic step-voltage default from 7 V to 4 V
  • recommend starting with a lower mechanism-appropriate value and increasing it only when needed
  • explain that excessive voltage can cause wheel slip and invalidate characterization data
  • show explicit 4 V configuration in the Java and Python examples

Validation

  • official doc8_redown.py lint on the changed page (0 errors)
  • git diff --check

Companion documentation for wpilibsuite/allwpilib#9375 and wpilibsuite/allwpilib#7794.

@TLNing260310

Copy link
Copy Markdown
Author

The Read the Docs preview failure is unrelated to this change. Its only fatal warning is an unresolved external Python class reference in source/docs/software/advanced-controls/controllers/trapezoidal-profiles.rst:26 (wpimath.TrapezoidProfile.Constraints). The changed creating-routine.rst page emitted no warnings, and the repository's doc8_redown.py lint passes with 0 errors. The GitHub Actions workflow is still awaiting maintainer approval for this fork PR.

@sciencewhiz

Copy link
Copy Markdown
Collaborator

How did you validate that the updated code samples work?

The unrelated build failure has been fixed in main, so it will pass if you merge/rebase with main.

@TLNing260310
TLNing260310 force-pushed the fix-7794-lower-sysid-step-voltage branch from 5355922 to fcd9f4f Compare September 1, 2026 15:15
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