Skip to content
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.

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

Add joystick button limit #5502

Merged
merged 1 commit into from
Aug 3, 2017

Conversation

dheideman
Copy link
Contributor

@dheideman dheideman commented Jul 26, 2017

Limits the maximum number of joystick buttons for which functions can be added on vehicle settings, joystick setup page, to 16.

Prevents issues of unconfigurable buttons outside the range 0~15 from appearing to be configurable, such as: https://cdn-business2.discourse.org/uploads/bluerobotics/original/2X/8/886612ffc44d2d46447e44976e857bec158ec7f3.png

Closes bluerobotics#21

@dheideman dheideman closed this Jul 26, 2017
@dheideman dheideman reopened this Jul 26, 2017
@hamishwillee
Copy link
Contributor

@dheideman @DonLakeFlyer Is this just sub, and in what version will this appear? Probably a useful note to add to the joystick docs

@dheideman
Copy link
Contributor Author

dheideman commented Jul 27, 2017

This was written for sub but should also apply to any other systems using joysticks in a similar fashion.

@jaxxzer
Copy link
Collaborator

jaxxzer commented Jul 27, 2017

@hamishwillee The constraint is the 16 bit field in the MANUAL_CONTROL message where the button values are sent (to all types of vehicles). So this should affect all vehicles.

@hamishwillee
Copy link
Contributor

Thanks guys. So would it be reasonable to add the following comment to the docs (for the current version/now)

"Up to 16 joystick buttons can be assigned to vehicle functions (If more than 16 button checkboxes are displayed only the first 16 will be used)"

@dheideman dheideman closed this Jul 28, 2017
@dheideman dheideman reopened this Jul 28, 2017
@DonLakeFlyer DonLakeFlyer merged commit c6ac209 into mavlink:master Aug 3, 2017
@jaxxzer
Copy link
Collaborator

jaxxzer commented Aug 3, 2017

@hamishwillee That would be correct for 3.2. After this merge, only 16 max will ever appear for assignment.

@jaxxzer jaxxzer deleted the pr-limit-joystick-btns branch August 3, 2017 18:39
@hamishwillee
Copy link
Contributor

Thanks @jaxxzer . I ended up adding the following note "Select the flight modes/vehicle functions activated by each joystick button. A maximum of 16 joystick button actions can be set.".

This works for all versions, because even if more are displayed at least it is clear why button 17+ does not work.

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.

4 participants