-
Notifications
You must be signed in to change notification settings - Fork 13.6k
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
OA: interface update #12349
OA: interface update #12349
Conversation
mrivi
commented
Jun 27, 2019
•
edited
Loading
edited
- Add the waypoint type (mavlink command associated with the waypoint) to the points in the OA interface
- refactor the check on the z progress of the mission
Did you see the CI failures? http://ci.px4.io:8080/blue/organizations/jenkins/PX4_misc%2FFirmware-SITL_tests/detail/PR-12349/1/pipeline ../../src/lib/FlightTasks/tasks/Utility/ObstacleAvoidance.cpp:55:1: error: narrowing conversion of '255' from 'int' to 'bool' inside { } [-Werror=narrowing]
};
^ |
SITL MC avoidance as well. |
@dagar We cannot merge this one until there is a mavros update and consequent CI update. Problem is that Also updating the mavlink submodule on its own will break the avoidance CI. |
bd4bd1e
to
d266d99
Compare
78f17b8
to
deb1c4a
Compare
@@ -37,7 +37,7 @@ px4_add_module( | |||
MODULE modules__mc_pos_control | |||
MAIN mc_pos_control | |||
COMPILE_FLAGS | |||
STACK_MAIN 1300 | |||
STACK_MAIN 1500 |
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.
Why is this needed?