Document robot controls and add regression tests - #70
Merged
Conversation
There was a problem hiding this comment.
Pull request overview
This PR documents how the dashboard converts joystick/keyboard input into differential-drive motor commands, and adds regression tests to keep the documented PWM mappings stable.
Changes:
- Added regression tests for joystick dead zones, proportional mixing, rotation-in-place, and arrow-key mappings.
- Documented the control algorithm and safety/throttling behavior across README(s) and technical docs.
- Added
CLAUDE.mdto forward agent guidance toAGENTS.md.
Reviewed changes
Copilot reviewed 7 out of 7 changed files in this pull request and generated 2 comments.
Show a summary per file
| File | Description |
|---|---|
| src/server/SmartBotBlazorApp.Tests/RobotMovementInputTests.cs | Adds regression tests for joystick mixing/dead zones and arrow-key motor mappings. |
| README.md | Documents one-joystick differential-drive behavior and keyboard alternative. |
| README.pl.md | Polish version of the new one-joystick differential-drive documentation. |
| docs/server.md | Adds a server-side control-input overview pointing to the relevant handlers/pages. |
| docs/getting-started.md | Adds a quick “control reference” section for joystick + arrow keys. |
| docs/architecture.md | Adds detailed motion-control equations, dead zones, and timing/safety behavior. |
| CLAUDE.md | Forwards agent instructions to AGENTS.md. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Kamilr616
force-pushed
the
agent/document-robot-controls
branch
from
July 14, 2026 17:09
6ee01a5 to
856c402
Compare
Kamilr616
force-pushed
the
agent/document-robot-controls
branch
from
July 14, 2026 17:20
856c402 to
b38c16c
Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
CLAUDE.mdforwarding agent guidance toAGENTS.mdImpact
The repository now explains how a single joystick combines throttle and steering for smooth differential-drive movement, including rotation around the robot's own axis, while also documenting keyboard control as an alternative. Tests protect the documented PWM mappings from regressions.
Validation
dotnet test src/server/SmartBotBlazorApp.Tests/SmartBotBlazorApp.Tests.csproj --no-restore --nologo— 28 passedgit diff --check— passed