Conversation
Checklist for reviewers ☑️All references to "You" in the following text refer to the code reviewer.
|
Failure optional scenariosNote This is an experimental check and does not block merging the pull-request. scenario failed: execution_time_test <failure type="SimulationFailure" message="CustomCommandAction typed "exitFailure" was triggered by the named Conditions {"update time checker", "avoid startup"}: {"update time checker": Is the /simulation/interpreter/execution_time/update (= 0.005232999999999999804489725364) is greaterThan 0.005?}, {"avoid startup": Is the simulation time (= 5.499999999999988453680543898372) is greaterThan 1.000000000000000000000000000000?}" /> |
|
HansRobo
left a comment
There was a problem hiding this comment.
LGTM. I also checked the build include this on Autoware Evaluator.
Could you explain why you've added
This way the users can keep using the modern tooling. Related: |
|
Actually, since this PR comes with a destructive change, I don't have the courage to revert this. We should instead update the build farm version with these changes and use this vendor package from the build farm with rosdep. Tracked issue: |



Description
Abstract
Use ament_vendor for zmqpp library, instead of ExternalProject.
Details
This section is same as autowarefoundation/zmqpp_vendor#9
References
Destructive Changes