Skip to content

Conversation

@leonxis
Copy link

@leonxis leonxis commented Oct 24, 2025

Features:

  • Added English comments for the modifications and verified that all defines are either in use or have default values.
  • Add independent manual calibration for analog sticks 1 and 2
  • Implement 4-step calibration process (top-left, top-right, bottom-left, bottom-right)
  • Add complete i18n support (English/Chinese) for calibration messages
  • Support parameterized messages for dynamic content

Technical changes:

  • Add joystick_center_x/y/x2/y2 fields to config proto
  • Add manual calibration API endpoints (/api/getJoystickCenter, /api/getJoystickCenter2)
  • Update analog input logic to support manual calibration values
  • Add comprehensive frontend calibration interface with step-by-step guidance
  • All code comments written in English as per project standards
  • Clean code without duplicate comments or code blocks

Features:
- Add independent manual calibration for analog sticks 1 and 2
- Implement 4-step calibration process (top-left, top-right, bottom-left, bottom-right)
- Add complete i18n support (English/Chinese) for calibration messages
- Support parameterized messages for dynamic content

Technical changes:
- Add joystick_center_x/y/x2/y2 fields to config proto
- Add manual calibration API endpoints (/api/getJoystickCenter, /api/getJoystickCenter2)
- Update analog input logic to support manual calibration values
- Add comprehensive frontend calibration interface with step-by-step guidance
- All code comments written in English as per project standards
- Clean code without duplicate comments or code blocks
@arntsonl
Copy link
Contributor

Thank you for submitting this!! I'm going to wait on reviewing/merging until after our release.

@arntsonl
Copy link
Contributor

It looks like its working but I want to touch up the UI a bit. I'll see if I can mock up a modal so you can just rotate the stick like Microsoft or any of the calibration software does and real-time feed it using an adc_read.

@leonxis
Copy link
Author

leonxis commented Nov 14, 2025

Thank you for making it even better.

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