Conversation
|
@frede791 I kind of liked the dramatic rotation, but as you wish :) The vehicle is a bit too small to see what is going on with 6 m (fo example control surfaces actuating), but we can keep it like that until gazebosim/gz-gui#515 is in |
7cbb84c to
81e8303
Compare
|
@Jaeyoung-Lim It doesn't latch onto the model for me. I still have to click "follow" for the model to actually be tracked from the requested camera perspective. |
fredmarkus
left a comment
There was a problem hiding this comment.
The stringmsg doesn't seem to refer to the specific model that it is supposed to attach to in the way the old version had it.
Could you elaborate? |
|
In your previous version you used PX4_SIM_MODEL: Since the call is now in GzBridge it is now set in the follow_msg, but it doesn't seem to be working correctly on my machine. |
|
@frede791 Have you tried changing the sleep just to see if it is working? The problem is that the follow service seems to always return true, even if the model does not exist in gz sim |
5cb607c to
9afce3d
Compare
This commit makes the gz bridge follow the spawned model F Update src/modules/simulation/gz_bridge/GZBridge.cpp Co-authored-by: Frederik Markus <80588263+frede791@users.noreply.github.com> Cleanup
9afce3d to
633b209
Compare
|
@frede791 @dagar I have removed the timeouts and detect whether the scene is created by checking if it is possible to query a scene message. |
3594f1f to
583106c
Compare
|
This pull request has been mentioned on Discussion Forum for PX4, Pixhawk, QGroundControl, MAVSDK, MAVLink. There might be relevant details there: |
|
This pull request has been mentioned on Discussion Forum for PX4, Pixhawk, QGroundControl, MAVSDK, MAVLink. There might be relevant details there: |
|
unused variable |
jup, I have the same issue |
|
Shows errors but still works despite the error printout |
Solved Problem
When running the simulation in gz, the model is not being followed by default.
@frede791 FYI
Solution
Start gz with a follow mode engaged to the model
Changelog Entry
For release notes:
Alternatives
We could also create a custom plugin to follow the model
Test coverage
Context