Conversation
2e130b4 to
948c1c9
Compare
948c1c9 to
38dd12d
Compare
Jaeyoung-Lim
left a comment
There was a problem hiding this comment.
Thanks! Looks great!
Only nitpick: why is the name "system" and not plugin? If we name it as plugin or Gzplugin it would be more clear what the implementation is.
For the next step, we need an implementation of the mavlink camera manager such that QGC can discover it automatically and take photos.
|
System seems to be the convention in gz so I followed the same pattern. We could call it SystemPlugin but all plugins are system plugins so 🤷 What would be the requirements for the mavlink camera manager? Is there a way to integrate with CI or some standard tests to run against QGC? |
Fair enough 🤷
It just needs to comply with the mavlink camera protocol: https://mavlink.io/en/services/camera.html I was wondering if it would be better to make it a px4 module, or we keep it inside gazebo but I guess it is better than no plugin either way. |
|
@DonLakeFlyer from the QGC perspective what else would be nice to have for testing? |
|
Ooh, nice thanks. I'll have to think about that a bit to see if there are other things that would help. Remote ID testing comes to mind, but not sure if that's possible. |
|
The other thing is that I can't seem to get MAV_1_BROADCAST to work. I always need to futz the startup scripts manually to get UDP to be in broadcast mode. |
Streams the first camera found in the world to 127.0.0.1 at port 5600.