-
Notifications
You must be signed in to change notification settings - Fork 17.6k
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
Copter: extend RCMap to cover at least channel 6 (maybe all channels up to 8) #685
Comments
@rmackay9 I don't know enough about copters, but isn't this covered with the servo changes that went in? I thought that got all vehicles arbitrary channel remapping... |
@WickedShell, I don't think so. Tridge's servo changes separated min/max for input/output but the improvements that adds are mostly on the output side of things. |
@peterbarker can this be closed now? |
On Tue, 15 May 2018, auturgy wrote:
@peterbarker can this be closed now?
Not until the channel-option patch stuff is merged.
|
Channel option patch is merged now AFAIK. Can this be closed now ? |
@amilcarlucas I'm actually a bit confused by it, actually. rcmap says "to find the throttle look at channel 3". e.g. So extending it to higher channels doesn't make sense. Now adding more options in there ( My "eliminate RCMAP" PR #8307 would definitely cover whatever this issue about :-) |
The point of the issue (I believe) is that copter has those switch functions for channel 6/7/8 such as motor interlock/autotune and the user wants to map that onto a different channel from their transmitter. |
I don't mind if this is left open until #8307 goes in or is closed now.. one way or the other we're definitely not going to extend RCMAP.. |
This is overtaken by events. The specific solution has not been implemented but the problem is solved. |
Some transmitters have limits on what mixing can be done on some channels. Request from Dragu MIhai.
This would also allow trad heli users to get by with a 6 or 7 channel transmitter instead of 8.
The text was updated successfully, but these errors were encountered: