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
fixes for R1 SDF https://github.com/PX4/sitl_gazebo/issues/533 #534
Conversation
@bperseghetti Thanks for getting this working! How are you testing this? While testing this PR, I noticed that the gps position is not valid in this model anymore, probably because #517 wasn't ported to this model probably. Are you only testing in manual mode with the joysticks? |
Yes it was https://github.com/PX4/sitl_gazebo/pull/517/files |
@TSC21 Ah, sorry. @bperseghetti I guess this means this PR needs a rebase on the latest master |
@Jaeyoung-Lim @TSC21 here: 34fe131 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks! Tested locally, and is a huge improvement in controls!
Mission mode is still not working, but should be fixed on the firmware side.
The sdf validation is failing on https://travis-ci.org/github/PX4/sitl_gazebo/jobs/701776596#L234
Fixes for R1 that also requires PR for Firmware: PX4/PX4-Autopilot#15209 to work fully.
Fixes issues brought up in #533