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

Make airframe outputs consistent #7474

Closed
hamishwillee opened this Issue Jun 26, 2017 · 4 comments

Comments

Projects
None yet
4 participants
@hamishwillee
Contributor

hamishwillee commented Jun 26, 2017

It would be good if the airframe outputs could use consistent outputs for the same function.

So for example, the standard plane frames use MAIN3, 4, or 5 for throttle.

(FYI I have updated the user guide so that the airframe reference is now the canonical place to get the output information, but I still think consistency in setup would make things easier for users).

@dagar

This comment has been minimized.

Member

dagar commented Jun 26, 2017

There will be exceptions of course, but we can definitely do a better job being consistent.

@LorenzMeier

This comment has been minimized.

Member

LorenzMeier commented Jul 9, 2017

Done.

@LorenzMeier LorenzMeier closed this Jul 9, 2017

@hamishwillee

This comment has been minimized.

Contributor

hamishwillee commented Jul 9, 2017

@LorenzMeier Please reopen this. The change I made better organises the content, but (despite my comment in #7482 does not "fix" the problem).

To fix this requires an update to the configuration files for the specific vehicles (mixers?) to make sure that they map the same output for throttle (and where possible other outputs). As I understand it, the change is technically easy to make, but would be a considerable "compatibility break."

@PX4BuildBot

This comment has been minimized.

Contributor

PX4BuildBot commented Feb 21, 2018

Hey, this issue has been closed because the label status/STALE is set and there were no updates for 30 days. Feel free to reopen this issue if you deem it appropriate.

(This is an automated comment from GitMate.io.)

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