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

Replace rc override service with topic #93

Closed
vooon opened this issue Aug 8, 2014 · 7 comments
Closed

Replace rc override service with topic #93

vooon opened this issue Aug 8, 2014 · 7 comments

Comments

@vooon
Copy link
Member

vooon commented Aug 8, 2014

@tonybaltovski requested in #22.

I think topic more suitable for override stream (and FCU does not send ACK), so i decide replace it.

@vooon vooon added this to the Version 0.7.0 milestone Aug 8, 2014
@vooon vooon closed this as completed in 10befa4 Aug 8, 2014
@tonybaltovski
Copy link
Contributor

Thank you. I will test this.

@tonybaltovski
Copy link
Contributor

@vooon This does not actually get used by the FCU. Have you tested it?

@TSC21
Copy link
Member

TSC21 commented Sep 4, 2014

In #133:

Perhaps PX4 don't handle RC override messages, i can't find handler for MAVLINK_MSG_ID_RC_CHANNELS_OVERRIDE message.

In the matter of fact, it isn't supported. You have propose that in the PX4 Firmware issues.

@vooon
Copy link
Member Author

vooon commented Sep 4, 2014

@tonybaltovski I'll test it with APM, they support it, but require mavros system id == SYSID_MYGCS.

https://github.com/diydrones/ardupilot/blob/9de1ae8fbbb4cf614bd59203c7605787241e60b4/ArduPlane/GCS_Mavlink.pde#L1350

@tonybaltovski
Copy link
Contributor

@vooon Does it matter that I am not using a gcs? Also, I tried the teleop with an APM and it was not changing when data is sent. @TSC21 Have you tried this?

@TSC21
Copy link
Member

TSC21 commented Sep 4, 2014

@tonybaltovski check my above comment.

@vooon
Copy link
Member Author

vooon commented Sep 4, 2014

@tonybaltovski try to set gcs sysid:

rosrun mavros mavparam set SYSID_MYGCS 1

Note: APM silently drop override if it comes not from gcs sysid, PX4 don't support this message at all.

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

No branches or pull requests

3 participants