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

adding APM and Flysky FS-I6 compatibility #4

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

tednv
Copy link

@tednv tednv commented Jan 11, 2017

ok, i think i have it sorted out now, i spent a long time trying to get this to work a few weeks ago but now my frame is in pieces so i cannot test but this is what i had working in missionplanner. readRX() should not be updated because the last channel (10 which is actually 11) does not get used for ppm output, it is only there to shift the pwm mapped gimbal channels up

this version of the code allows for the original configuration with overrides for stuff i tested with apm mission planner

i will retest it again this weekend if i have some time to put my 450 frame back together

ok, i think i have it sorted out now, i spent a long time trying to get this to work a few weeks ago but now my frame is in pieces so i cannot test but this is what i had working in missionplanner. readRX() should not be updated because the last channel (10 which is actually 11) does not get used for ppm output, it is only there to shift the pwm mapped gimbal channels up

this version of the code allows for the original configuration with overrides for stuff i tested with apm mission planner

i will retest it again this weekend if i have some time to put my 450 frame back together
@tednv
Copy link
Author

tednv commented Jan 12, 2017

just retested this exact code with APM + Mission planner

to use, change APMTarget to true and uncomment the three lines below

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.

1 participant