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

Fix LED Indicators mode for Fixed Wing and Rover #8826

Merged

Conversation

MrD-RC
Copy link
Collaborator

@MrD-RC MrD-RC commented Feb 23, 2023

Currently, the indicators mode is only really works for quads. It doesn't make any sense for fixed wing. This PR changes that. Roll movements to the left triggers indicators on the left half of the grid. Roll movements to the right triggers indicators on the right half of the grid.

This change may also be suitable for rovers too. If so, it's an easy modification. So feedback from rover people would be great.

Currently, the indicator mode is only really works for quads. It doesn't make any sense for fixed wing. This PR changes that. Roll movements to the left triggers indicators on the left half of the grid. Roll movements to the right triggers indicators on the right half of the grid.

This change may also be suitable for rovers too. If so, it's an easy modification. So feedback from rover people would be great.
@MrD-RC MrD-RC added this to the 6.1 milestone Feb 23, 2023
@MrD-RC MrD-RC changed the title Fix LED Indicator mode for fixed wing Fix LED Indicators mode for fixed wing Feb 26, 2023
@WizzX-FPV
Copy link
Contributor

I'm INAV rover guy and using stick controller (no pistol) TX16s I'm using left stick (yaw) for steering and roll axis for camera pan. But I asked another guy on RC groups and he using left stick for throttle only a right stick (roll) for steering. So seems like there must be selectable channel for using indicators :)

@sercona
Copy link

sercona commented Mar 2, 2023

so far, what seems to work for me with my rover (single bidirectional brushed or brushless motor, single steering servo, modified wltoys 144001 buggy) is to use air stick controller (I also avoid pistol remotes and use sticks for both air and surface) and have the self-centering mode2 stick on the right do throttle forward/backward as pitch and steering as roll, but its not roll, its really yaw, just done on that single stick. I left the left-hand mode2 stick alone, for now, since having speed and direction on that single stick really works out well in racing and bashing (as they call it).

@sercona
Copy link

sercona commented Mar 2, 2023

also, braking - one of the new ESCs out there (for surface) supports a drag brake (level) channel. give it usual servo 1k-2k value and it will set the drag brake to that. it kind of also works as a sudden brake. I have this mapped to the spring return 'beeper' switch on my air remote (often the far right switch at the top) and if I need more brake than I set for 1500 (neutral) throttle value, then I can add in or override the brake value with that switch, sending max value on the brake control channel. (need to use pwm wire on your FC to control this as its a real wire just like the throttle wire, normal pwm like servo)

@DzikuVx DzikuVx changed the base branch from master to release_6.1.0 March 14, 2023 20:28
After asking. Around 80% of rover drivers said they turn using the roll channel. And that they would like indicators that worked as the direction input.
@MrD-RC MrD-RC changed the title Fix LED Indicators mode for fixed wing Fix LED Indicators mode for Fixed Wing and Rover Mar 27, 2023
@MrD-RC
Copy link
Collaborator Author

MrD-RC commented Mar 27, 2023

Around 80% of rover drivers who responded said they use the roll channel for turning. So added the indicators for rovers too.

@MrD-RC MrD-RC merged commit 4249019 into release_6.1.0 Apr 11, 2023
@MrD-RC MrD-RC deleted the MrD_Correct-Indicator-LED-function-for-Fixed-Wing branch April 11, 2023 17:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants