-
Notifications
You must be signed in to change notification settings - Fork 13.7k
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
posix: add argument to change the CWD #12482
Conversation
9750bd0
to
4c79e54
Compare
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, looks good, but can you improve the failure outputs?
95699db
to
3b6fc8f
Compare
3b6fc8f
to
2789987
Compare
@bkueng I made the changes you requested. |
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.
Looks good, let's see if Appveyor also passes.
Hey @lamping7 so I've tested your changes and I don't know if it's because something I'm doing but when I push 3 different missions to the uav's via mavros they say that they received the waypoint. Now the problem is that they don't use these waypoints they only use the height I've set for the rest they just fly a square and all land on top of each other. |
Can you share more info? Was one of the missions a square? I can't tell what you did there. |
Hey so the problem was that my coordinates where being rounded of so they ended up at almost the same location. I have tested it again and it's working correctly. |
#9046
Because the paths/files generated by px4 depend on the location of the current working directory, simply changing it allows us to avoid conflicts with launching multiple instances.
launching one vehicle with mavros_posix_sitl.launch
launching three vehicles with multi_uav_mavros_sitl.launch