New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Pen Servo Tool fixes #733

Merged

Paciente8159 merged 2 commits into Paciente8159:master from fooker:pr/servo-pen

Aug 16, 2024

Contributor

fooker commented Aug 14, 2024

This fixes the range function for the pen servo tool:

Remove the unused PEN_SERVO_MID option
Allow to have PEN_SERVO_LOW be greater than PEN_SERVO_HIGH
Fix an type conversion overflow in the range converter
Add safety clamping before sending values to PWN

fooker added 2 commits

August 14, 2024 23:26


          Remove unused PEN_SERVO_MID

874e387


          Fix pen servo range function

0623e91

Paciente8159 approved these changes

View reviewed changes

Owner

Paciente8159 left a comment

Code tested and OK

Paciente8159 merged commit a8cc420 into Paciente8159:master

19 checks passed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet