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

GCS Bridge Plugin #1117

Closed
wants to merge 6 commits into from
Closed

Conversation

arwagoner
Copy link

Converts the GCS Bridge into a plugin.

Tested it by writing a node that publishes a test NAMED_VALUE_FLOAT to /mavlink/from and saw that it shows up on QGC. Users do have to specify a gcs_url in their mavros launch file to enable this plugin.

qgc_bridge_plugin

It does not currently support OBC reboot/shutdown commands from GCS as requested in #333 and #450.

}
else
ROS_INFO("GCS bridge disabled");
if (gcs_url != "") {
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Indentation are 8 spaces, not 4. Please use uncrustify (uncrustify -c ${ROS_WORKSPACE}/src/mavros/mavros/tools/uncrustify-cpp.cfg --replace --no-backup <path/to/file.ext>).

Copy link
Member

@vooon vooon left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Where changes in MavRos class? How that new callbacks works?
Where message filter?

For now that is early WIP.

Also i'm thinking that GCS plugin should have different base class to differentiate message sources.

@arwagoner
Copy link
Author

Where changes in MavRos class?

I added a new pointer for the GCS link. It follows the same format as the FCU link

How that new callbacks works?

It works the same way as the callback worked in the original GCS_Bridge. The only difference is that this is a plugin.

Where message filter?

What kind of a message filter are you looking for?

For now that is early WIP. Also i'm thinking that GCS plugin should have different base class to differentiate message sources.

There is not currently a good way to communicate between the companion computer and QGC. I think it makes sense to get the base functionality out there and then allow the community to continue building on it as it goes.

Copy link
Contributor

@amilcarlucas amilcarlucas left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please rebase and squash some of your commits

@@ -47,4 +47,7 @@
<class name="vision_speed_estimate" type="mavros::extra_plugins::VisionSpeedEstimatePlugin" base_class_type="mavros::plugin::PluginBase">
<description>Send vision speed estimate to FCU.</description>
</class>
<class name="gcs_bridge_plugin" type="mavros::extra_plugins::GCSBridgePlugin" base_class_type="mavros::plugin::PluginBase" output="screen">
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Indentation please

@vooon
Copy link
Member

vooon commented Apr 25, 2021

That feature woun't be implemented in mavros v1. But it should be possible with v2 because here we have filters and router.

@vooon vooon closed this Apr 25, 2021
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.

None yet

4 participants