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

Comments

Projects
None yet
3 participants
@vooon
Copy link
Member

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 in 10befa4 Aug 8, 2014

@tonybaltovski

This comment has been minimized.

Copy link
Contributor

commented Aug 8, 2014

Thank you. I will test this.

@tonybaltovski

This comment has been minimized.

Copy link
Contributor

commented Sep 4, 2014

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

@TSC21

This comment has been minimized.

Copy link
Member

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

This comment has been minimized.

Copy link
Member Author

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

This comment has been minimized.

Copy link
Contributor

commented Sep 4, 2014

@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

This comment has been minimized.

Copy link
Member

commented Sep 4, 2014

@tonybaltovski check my above comment.

@vooon

This comment has been minimized.

Copy link
Member Author

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
You can’t perform that action at this time.