Merge v9.1 to main

[JWhitleyWork]

No description provided.

[Copilot]

Pull request overview

Updates lab_sim_behaviors to use the MoveIt Studio vision wrapper header for cv_bridge integration (consistent with the package’s declared dependencies), as part of merging v9.1 into main.

Changes:

• Replaced direct cv_bridge include with moveit_studio_vision’s cv_bridge wrapper header.

Comments suppressed due to low confidence (1)

src/lab_sim_behaviors/src/compute_tray_place_positions_using_apriltags.cpp:20

• This file uses sensor_msgs::image_encodings::RGB8 and std::find_if, but does not include the corresponding headers (<sensor_msgs/image_encodings.hpp> and <algorithm>). With the switch away from cv_bridge/cv_bridge.h, relying on transitive includes becomes more brittle and may break builds depending on what moveit_studio_vision/cv_bridge.hpp includes. Add the direct includes here to make compilation deterministic.

#include <moveit_pro_behavior_interface/get_required_ports.hpp>
#include <moveit_pro_behavior_interface/metadata_fields.hpp>
#include <moveit_studio_vision/cv_bridge.hpp>
#include <moveit_studio_vision/image_utils.hpp>
#include <moveit_studio_vision_msgs/msg/object_detection_array.hpp>
#include <opencv2/opencv.hpp>
#include <sensor_msgs/msg/camera_info.hpp>

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.