Conversation
Signed-off-by: Mabel Zhang <mabel@openrobotics.org>
Signed-off-by: Mabel Zhang <mabel@openrobotics.org>
Signed-off-by: Mabel Zhang <mabel@openrobotics.org>
|
The PR job is failing because you'll also need to remove this from the rosbag2 Line 16 in 038bae2 |
Signed-off-by: Mabel Zhang <mabel@openrobotics.org>
|
The PR job is still failing. Is that expected? Relatedly, it looks like pybind11_vendor needs to be released into rolling. I'll suggest that we do that before removing the |
|
Are you referring to the Rpr__rosbag2__ubuntu_focal_amd64? That seems to normally fail. Some of the recently merged PRs pass everything except for that one. |
|
Looks just yellow. I think it's because some of the end to end tests are flaky. |
Yeah, looking at an earlier PR job, it looks like there were 4 failures, while this one has 8. But still, I don't see how this particular PR would cause that, so I'm still going to approve. @emersonknapp Do you have anything to add here? Otherwise we'll go ahead and merge. |
* AMENT_IGNORE rosbag2_py Signed-off-by: Mabel Zhang <mabel@openrobotics.org> * remove rosbag2_py from meta package Signed-off-by: Mabel Zhang <mabel@openrobotics.org>
* AMENT_IGNORE rosbag2_py Signed-off-by: Mabel Zhang <mabel@openrobotics.org> * remove rosbag2_py from meta package Signed-off-by: Mabel Zhang <mabel@openrobotics.org>
* AMENT_IGNORE rosbag2_py Signed-off-by: Mabel Zhang <mabel@openrobotics.org> * remove rosbag2_py from meta package Signed-off-by: Mabel Zhang <mabel@openrobotics.org> Signed-off-by: Alexander Gutenkunst <alex.gutenkunst+github.com>
* AMENT_IGNORE rosbag2_py Signed-off-by: Mabel Zhang <mabel@openrobotics.org> * remove rosbag2_py from meta package Signed-off-by: Mabel Zhang <mabel@openrobotics.org>
Adding
AMENT_IGNOREtorosbag2_py, due to Nightly failures on Debug mode in Windows #504 and Linux with Clang #505.We have a solution for Linux with Clang that passes the compiler but the tests are failing.
Still need to look into Windows.
We opted to add
AMENT_IGNOREinstead of reverting #308 because given that there were a large number of lines added in that PR, we thought this would make it easier to review the patch, which we will make to fix the errors.CI is green, with Clang and build type Debug, running on meta package
rosbag2(Ignore all the commits from my old fork. Will clean up the commit comments in the squashed commit when merging.)