Austin Schuh | dcb6b36 | 2022-02-25 18:06:21 -0800 | [diff] [blame] | 1 | #include "frc971/vision/calibration_accumulator.h" |
milind-u | 8c72d53 | 2021-12-11 15:02:42 -0800 | [diff] [blame] | 2 | |
Jim Ostrowski | b3cab97 | 2022-12-03 15:47:00 -0800 | [diff] [blame] | 3 | #include <algorithm> |
Stephan Pleines | 6191f1d | 2024-05-30 20:44:45 -0700 | [diff] [blame] | 4 | #include <iomanip> |
Jim Ostrowski | b3cab97 | 2022-12-03 15:47:00 -0800 | [diff] [blame] | 5 | #include <limits> |
Jim Ostrowski | b3cab97 | 2022-12-03 15:47:00 -0800 | [diff] [blame] | 6 | |
milind-u | 8c72d53 | 2021-12-11 15:02:42 -0800 | [diff] [blame] | 7 | #include "Eigen/Dense" |
Philipp Schrader | 790cb54 | 2023-07-05 21:06:52 -0700 | [diff] [blame] | 8 | #include "external/com_github_foxglove_schemas/CompressedImage_schema.h" |
| 9 | #include "external/com_github_foxglove_schemas/ImageAnnotations_schema.h" |
| 10 | #include <opencv2/highgui/highgui.hpp> |
| 11 | |
milind-u | 8c72d53 | 2021-12-11 15:02:42 -0800 | [diff] [blame] | 12 | #include "aos/events/simulated_event_loop.h" |
Jim Ostrowski | b3cab97 | 2022-12-03 15:47:00 -0800 | [diff] [blame] | 13 | #include "aos/network/team_number.h" |
milind-u | 8c72d53 | 2021-12-11 15:02:42 -0800 | [diff] [blame] | 14 | #include "aos/time/time.h" |
| 15 | #include "frc971/control_loops/quaternion_utils.h" |
Austin Schuh | dcb6b36 | 2022-02-25 18:06:21 -0800 | [diff] [blame] | 16 | #include "frc971/vision/charuco_lib.h" |
Jim Ostrowski | ba2edd1 | 2022-12-03 15:44:37 -0800 | [diff] [blame] | 17 | #include "frc971/wpilib/imu_batch_generated.h" |
| 18 | |
milind-u | 8c72d53 | 2021-12-11 15:02:42 -0800 | [diff] [blame] | 19 | DEFINE_bool(display_undistorted, false, |
| 20 | "If true, display the undistorted image."); |
Jim Ostrowski | ba2edd1 | 2022-12-03 15:44:37 -0800 | [diff] [blame] | 21 | DEFINE_string(save_path, "", "Where to store annotated images"); |
| 22 | DEFINE_bool(save_valid_only, false, |
| 23 | "If true, only save images with valid pose estimates"); |
milind-u | 8c72d53 | 2021-12-11 15:02:42 -0800 | [diff] [blame] | 24 | |
Stephan Pleines | f63bde8 | 2024-01-13 15:59:33 -0800 | [diff] [blame] | 25 | namespace frc971::vision { |
milind-u | 8c72d53 | 2021-12-11 15:02:42 -0800 | [diff] [blame] | 26 | using aos::distributed_clock; |
| 27 | using aos::monotonic_clock; |
| 28 | namespace chrono = std::chrono; |
| 29 | |
Austin Schuh | 5b37907 | 2021-12-26 16:01:04 -0800 | [diff] [blame] | 30 | constexpr double kG = 9.807; |
| 31 | |
milind-u | 8c72d53 | 2021-12-11 15:02:42 -0800 | [diff] [blame] | 32 | void CalibrationData::AddCameraPose( |
| 33 | distributed_clock::time_point distributed_now, Eigen::Vector3d rvec, |
| 34 | Eigen::Vector3d tvec) { |
Jim Ostrowski | ba2edd1 | 2022-12-03 15:44:37 -0800 | [diff] [blame] | 35 | // Always start with IMU (or turret) reading... |
| 36 | // Note, we may not have a turret, so need to handle that case |
| 37 | // If we later get a turret point, then we handle removal of camera points in |
| 38 | // AddTurret |
| 39 | if ((!imu_points_.empty() && imu_points_[0].first < distributed_now) && |
| 40 | (turret_points_.empty() || turret_points_[0].first < distributed_now)) { |
Austin Schuh | 5b37907 | 2021-12-26 16:01:04 -0800 | [diff] [blame] | 41 | rot_trans_points_.emplace_back(distributed_now, std::make_pair(rvec, tvec)); |
| 42 | } |
milind-u | 8c72d53 | 2021-12-11 15:02:42 -0800 | [diff] [blame] | 43 | } |
| 44 | |
| 45 | void CalibrationData::AddImu(distributed_clock::time_point distributed_now, |
| 46 | Eigen::Vector3d gyro, Eigen::Vector3d accel) { |
Jim Ostrowski | ba2edd1 | 2022-12-03 15:44:37 -0800 | [diff] [blame] | 47 | double zero_threshold = 1e-12; |
| 48 | // We seem to be getting 0 readings on IMU, so ignore for now |
| 49 | // TODO<Jim>: I think this has been resolved in HandleIMU, but want to leave |
| 50 | // this here just in case there are other ways this could happen |
| 51 | if ((fabs(accel(0)) < zero_threshold) && (fabs(accel(1)) < zero_threshold) && |
| 52 | (fabs(accel(2)) < zero_threshold)) { |
| 53 | LOG(FATAL) << "Ignoring zero value from IMU accelerometer: " << accel |
| 54 | << " (gyro is " << gyro << ")"; |
| 55 | } else { |
| 56 | imu_points_.emplace_back(distributed_now, std::make_pair(gyro, accel)); |
| 57 | } |
milind-u | 8c72d53 | 2021-12-11 15:02:42 -0800 | [diff] [blame] | 58 | } |
| 59 | |
Austin Schuh | 2895f4c | 2022-02-26 16:38:46 -0800 | [diff] [blame] | 60 | void CalibrationData::AddTurret( |
| 61 | aos::distributed_clock::time_point distributed_now, Eigen::Vector2d state) { |
Jim Ostrowski | ba2edd1 | 2022-12-03 15:44:37 -0800 | [diff] [blame] | 62 | // We want the turret to be known too when solving. But, we don't know if |
| 63 | // we are going to have a turret until we get the first reading. In that |
| 64 | // case, blow away any camera readings from before. |
| 65 | // NOTE: Since the IMU motion is independent of the turret position, we don't |
| 66 | // need to remove the IMU readings before the turret |
| 67 | if (turret_points_.empty()) { |
| 68 | while (!rot_trans_points_.empty() && |
| 69 | rot_trans_points_[0].first < distributed_now) { |
| 70 | LOG(INFO) << "Erasing, distributed " << distributed_now; |
| 71 | rot_trans_points_.erase(rot_trans_points_.begin()); |
| 72 | } |
Austin Schuh | 2895f4c | 2022-02-26 16:38:46 -0800 | [diff] [blame] | 73 | } |
| 74 | turret_points_.emplace_back(distributed_now, state); |
| 75 | } |
| 76 | |
Austin Schuh | dcb6b36 | 2022-02-25 18:06:21 -0800 | [diff] [blame] | 77 | void CalibrationData::ReviewData(CalibrationDataObserver *observer) const { |
milind-u | 8c72d53 | 2021-12-11 15:02:42 -0800 | [diff] [blame] | 78 | size_t next_camera_point = 0; |
Jim Ostrowski | ba2edd1 | 2022-12-03 15:44:37 -0800 | [diff] [blame] | 79 | size_t next_imu_point = 0; |
| 80 | size_t next_turret_point = 0; |
| 81 | |
| 82 | // Just go until one of the data streams runs out. We lose a few points, but |
| 83 | // it makes the logic much easier |
| 84 | while ( |
| 85 | next_camera_point != rot_trans_points_.size() && |
| 86 | next_imu_point != imu_points_.size() && |
| 87 | (turret_points_.empty() || next_turret_point != turret_points_.size())) { |
| 88 | // If camera_point is next, update it |
| 89 | if ((rot_trans_points_[next_camera_point].first <= |
| 90 | imu_points_[next_imu_point].first) && |
| 91 | (turret_points_.empty() || |
| 92 | (rot_trans_points_[next_camera_point].first <= |
| 93 | turret_points_[next_turret_point].first))) { |
| 94 | // Camera! |
| 95 | observer->UpdateCamera(rot_trans_points_[next_camera_point].first, |
| 96 | rot_trans_points_[next_camera_point].second); |
| 97 | ++next_camera_point; |
| 98 | } else { |
| 99 | // If it's not the camera, check if IMU is next |
| 100 | if (turret_points_.empty() || (imu_points_[next_imu_point].first <= |
| 101 | turret_points_[next_turret_point].first)) { |
| 102 | // IMU! |
| 103 | observer->UpdateIMU(imu_points_[next_imu_point].first, |
| 104 | imu_points_[next_imu_point].second); |
| 105 | ++next_imu_point; |
milind-u | 8c72d53 | 2021-12-11 15:02:42 -0800 | [diff] [blame] | 106 | } else { |
Jim Ostrowski | ba2edd1 | 2022-12-03 15:44:37 -0800 | [diff] [blame] | 107 | // If it's not IMU or camera, and turret_points is not empty, it must be |
| 108 | // the turret! |
| 109 | observer->UpdateTurret(turret_points_[next_turret_point].first, |
| 110 | turret_points_[next_turret_point].second); |
| 111 | ++next_turret_point; |
milind-u | 8c72d53 | 2021-12-11 15:02:42 -0800 | [diff] [blame] | 112 | } |
| 113 | } |
| 114 | } |
| 115 | } |
| 116 | |
James Kuszmaul | 969e4ab | 2023-01-28 16:09:19 -0800 | [diff] [blame] | 117 | CalibrationFoxgloveVisualizer::CalibrationFoxgloveVisualizer( |
Maxwell Henderson | ecc8a7c | 2024-02-29 20:19:45 -0800 | [diff] [blame] | 118 | aos::EventLoop *event_loop, std::string_view camera_channel) |
James Kuszmaul | 969e4ab | 2023-01-28 16:09:19 -0800 | [diff] [blame] | 119 | : event_loop_(event_loop), |
Maxwell Henderson | ecc8a7c | 2024-02-29 20:19:45 -0800 | [diff] [blame] | 120 | image_converter_(event_loop_, camera_channel, camera_channel, |
James Kuszmaul | 969e4ab | 2023-01-28 16:09:19 -0800 | [diff] [blame] | 121 | ImageCompression::kJpeg), |
| 122 | annotations_sender_( |
Maxwell Henderson | ecc8a7c | 2024-02-29 20:19:45 -0800 | [diff] [blame] | 123 | event_loop_->MakeSender<foxglove::ImageAnnotations>(camera_channel)) { |
| 124 | } |
James Kuszmaul | 969e4ab | 2023-01-28 16:09:19 -0800 | [diff] [blame] | 125 | |
| 126 | aos::FlatbufferDetachedBuffer<aos::Configuration> |
| 127 | CalibrationFoxgloveVisualizer::AddVisualizationChannels( |
| 128 | const aos::Configuration *config, const aos::Node *node) { |
| 129 | constexpr std::string_view channel_name = "/visualization"; |
| 130 | aos::ChannelT channel_overrides; |
| 131 | channel_overrides.max_size = 10000000; |
| 132 | aos::FlatbufferDetachedBuffer<aos::Configuration> result = |
| 133 | aos::configuration::AddChannelToConfiguration( |
| 134 | config, channel_name, |
| 135 | aos::FlatbufferSpan<reflection::Schema>( |
| 136 | foxglove::ImageAnnotationsSchema()), |
| 137 | node, channel_overrides); |
| 138 | return aos::configuration::AddChannelToConfiguration( |
| 139 | &result.message(), channel_name, |
| 140 | aos::FlatbufferSpan<reflection::Schema>( |
| 141 | foxglove::CompressedImageSchema()), |
| 142 | node, channel_overrides); |
| 143 | } |
| 144 | |
James Kuszmaul | 7e95881 | 2023-02-11 15:34:31 -0800 | [diff] [blame] | 145 | Calibration::Calibration( |
| 146 | aos::SimulatedEventLoopFactory *event_loop_factory, |
| 147 | aos::EventLoop *image_event_loop, aos::EventLoop *imu_event_loop, |
Jim Ostrowski | 3dc2164 | 2024-01-22 16:08:40 -0800 | [diff] [blame] | 148 | std::string_view hostname, |
James Kuszmaul | 7e95881 | 2023-02-11 15:34:31 -0800 | [diff] [blame] | 149 | const calibration::CameraCalibration *intrinsics_calibration, |
| 150 | TargetType target_type, std::string_view image_channel, |
| 151 | CalibrationData *data) |
milind-u | 8c72d53 | 2021-12-11 15:02:42 -0800 | [diff] [blame] | 152 | : image_event_loop_(image_event_loop), |
| 153 | image_factory_(event_loop_factory->GetNodeEventLoopFactory( |
| 154 | image_event_loop_->node())), |
| 155 | imu_event_loop_(imu_event_loop), |
| 156 | imu_factory_( |
| 157 | event_loop_factory->GetNodeEventLoopFactory(imu_event_loop_->node())), |
| 158 | charuco_extractor_( |
James Kuszmaul | 7e95881 | 2023-02-11 15:34:31 -0800 | [diff] [blame] | 159 | image_event_loop_, intrinsics_calibration, target_type, image_channel, |
milind-u | 8c72d53 | 2021-12-11 15:02:42 -0800 | [diff] [blame] | 160 | [this](cv::Mat rgb_image, monotonic_clock::time_point eof, |
Jim Ostrowski | b3cab97 | 2022-12-03 15:47:00 -0800 | [diff] [blame] | 161 | std::vector<cv::Vec4i> charuco_ids, |
| 162 | std::vector<std::vector<cv::Point2f>> charuco_corners, |
| 163 | bool valid, std::vector<Eigen::Vector3d> rvecs_eigen, |
| 164 | std::vector<Eigen::Vector3d> tvecs_eigen) { |
milind-u | 8c72d53 | 2021-12-11 15:02:42 -0800 | [diff] [blame] | 165 | HandleCharuco(rgb_image, eof, charuco_ids, charuco_corners, valid, |
Jim Ostrowski | b3cab97 | 2022-12-03 15:47:00 -0800 | [diff] [blame] | 166 | rvecs_eigen, tvecs_eigen); |
| 167 | }), |
Jim Ostrowski | cb8b408 | 2024-01-21 02:23:46 -0800 | [diff] [blame] | 168 | // TODO: Need to make this work for pi or orin |
Jim Ostrowski | b3cab97 | 2022-12-03 15:47:00 -0800 | [diff] [blame] | 169 | image_callback_( |
| 170 | image_event_loop_, |
Jim Ostrowski | 3dc2164 | 2024-01-22 16:08:40 -0800 | [diff] [blame] | 171 | absl::StrCat("/", aos::network::ParsePiOrOrin(hostname).value(), |
| 172 | std::to_string( |
| 173 | aos::network::ParsePiOrOrinNumber(hostname).value()), |
| 174 | image_channel), |
Jim Ostrowski | b3cab97 | 2022-12-03 15:47:00 -0800 | [diff] [blame] | 175 | [this](cv::Mat rgb_image, const monotonic_clock::time_point eof) { |
| 176 | charuco_extractor_.HandleImage(rgb_image, eof); |
milind-u | 8c72d53 | 2021-12-11 15:02:42 -0800 | [diff] [blame] | 177 | }), |
James Kuszmaul | 969e4ab | 2023-01-28 16:09:19 -0800 | [diff] [blame] | 178 | data_(data), |
| 179 | visualizer_event_loop_(image_factory_->MakeEventLoop("visualization")), |
Maxwell Henderson | ecc8a7c | 2024-02-29 20:19:45 -0800 | [diff] [blame] | 180 | visualizer_(visualizer_event_loop_.get(), image_channel) { |
milind-u | 8c72d53 | 2021-12-11 15:02:42 -0800 | [diff] [blame] | 181 | imu_factory_->OnShutdown([]() { cv::destroyAllWindows(); }); |
| 182 | |
Jim Ostrowski | ba2edd1 | 2022-12-03 15:44:37 -0800 | [diff] [blame] | 183 | // Check for IMUValuesBatch topic on both /localizer and /drivetrain channels, |
| 184 | // since both are valid/possible |
| 185 | std::string imu_channel; |
| 186 | if (imu_event_loop->HasChannel<frc971::IMUValuesBatch>("/localizer")) { |
| 187 | imu_channel = "/localizer"; |
| 188 | } else if (imu_event_loop->HasChannel<frc971::IMUValuesBatch>( |
| 189 | "/drivetrain")) { |
| 190 | imu_channel = "/drivetrain"; |
| 191 | } else { |
| 192 | LOG(FATAL) << "Couldn't find channel with IMU data for either localizer or " |
| 193 | "drivtrain"; |
| 194 | } |
| 195 | |
| 196 | VLOG(2) << "Listening for " << frc971::IMUValuesBatch::GetFullyQualifiedName() |
| 197 | << " on channel: " << imu_channel; |
| 198 | |
milind-u | 8c72d53 | 2021-12-11 15:02:42 -0800 | [diff] [blame] | 199 | imu_event_loop_->MakeWatcher( |
Jim Ostrowski | ba2edd1 | 2022-12-03 15:44:37 -0800 | [diff] [blame] | 200 | imu_channel, [this](const frc971::IMUValuesBatch &imu) { |
milind-u | 8c72d53 | 2021-12-11 15:02:42 -0800 | [diff] [blame] | 201 | if (!imu.has_readings()) { |
| 202 | return; |
| 203 | } |
| 204 | for (const frc971::IMUValues *value : *imu.readings()) { |
| 205 | HandleIMU(value); |
| 206 | } |
| 207 | }); |
| 208 | } |
| 209 | |
Jim Ostrowski | b3cab97 | 2022-12-03 15:47:00 -0800 | [diff] [blame] | 210 | void Calibration::HandleCharuco( |
| 211 | cv::Mat rgb_image, const monotonic_clock::time_point eof, |
| 212 | std::vector<cv::Vec4i> /*charuco_ids*/, |
James Kuszmaul | 969e4ab | 2023-01-28 16:09:19 -0800 | [diff] [blame] | 213 | std::vector<std::vector<cv::Point2f>> charuco_corners, bool valid, |
Jim Ostrowski | b3cab97 | 2022-12-03 15:47:00 -0800 | [diff] [blame] | 214 | std::vector<Eigen::Vector3d> rvecs_eigen, |
| 215 | std::vector<Eigen::Vector3d> tvecs_eigen) { |
James Kuszmaul | 969e4ab | 2023-01-28 16:09:19 -0800 | [diff] [blame] | 216 | visualizer_.HandleCharuco(eof, charuco_corners); |
milind-u | 8c72d53 | 2021-12-11 15:02:42 -0800 | [diff] [blame] | 217 | if (valid) { |
Jim Ostrowski | b3cab97 | 2022-12-03 15:47:00 -0800 | [diff] [blame] | 218 | CHECK(rvecs_eigen.size() > 0) << "Require at least one target detected"; |
| 219 | // We only use one (the first) target detected for calibration |
| 220 | data_->AddCameraPose(image_factory_->ToDistributedClock(eof), |
| 221 | rvecs_eigen[0], tvecs_eigen[0]); |
milind-u | 8c72d53 | 2021-12-11 15:02:42 -0800 | [diff] [blame] | 222 | |
milind-u | 8c72d53 | 2021-12-11 15:02:42 -0800 | [diff] [blame] | 223 | Eigen::IOFormat HeavyFmt(Eigen::FullPrecision, 0, ", ", ",\n", "[", "]", |
| 224 | "[", "]"); |
| 225 | |
| 226 | const double age_double = |
| 227 | std::chrono::duration_cast<std::chrono::duration<double>>( |
| 228 | image_event_loop_->monotonic_now() - eof) |
| 229 | .count(); |
Jim Ostrowski | ba2edd1 | 2022-12-03 15:44:37 -0800 | [diff] [blame] | 230 | VLOG(1) << std::fixed << std::setprecision(6) << "Age: " << age_double |
Jim Ostrowski | b3cab97 | 2022-12-03 15:47:00 -0800 | [diff] [blame] | 231 | << ", Pose is R:" << rvecs_eigen[0].transpose().format(HeavyFmt) |
| 232 | << "\nT:" << tvecs_eigen[0].transpose().format(HeavyFmt); |
milind-u | 8c72d53 | 2021-12-11 15:02:42 -0800 | [diff] [blame] | 233 | } |
| 234 | |
Jim Ostrowski | b3cab97 | 2022-12-03 15:47:00 -0800 | [diff] [blame] | 235 | if (FLAGS_visualize) { |
| 236 | if (FLAGS_display_undistorted) { |
| 237 | const cv::Size image_size(rgb_image.cols, rgb_image.rows); |
| 238 | cv::Mat undistorted_rgb_image(image_size, CV_8UC3); |
| 239 | cv::undistort(rgb_image, undistorted_rgb_image, |
| 240 | charuco_extractor_.camera_matrix(), |
| 241 | charuco_extractor_.dist_coeffs()); |
milind-u | 8c72d53 | 2021-12-11 15:02:42 -0800 | [diff] [blame] | 242 | |
Jim Ostrowski | b3cab97 | 2022-12-03 15:47:00 -0800 | [diff] [blame] | 243 | cv::imshow("Display undist", undistorted_rgb_image); |
| 244 | } |
milind-u | 8c72d53 | 2021-12-11 15:02:42 -0800 | [diff] [blame] | 245 | |
Jim Ostrowski | b3cab97 | 2022-12-03 15:47:00 -0800 | [diff] [blame] | 246 | cv::imshow("Display", rgb_image); |
| 247 | cv::waitKey(1); |
milind-u | 8c72d53 | 2021-12-11 15:02:42 -0800 | [diff] [blame] | 248 | } |
Jim Ostrowski | ba2edd1 | 2022-12-03 15:44:37 -0800 | [diff] [blame] | 249 | |
| 250 | if (FLAGS_save_path != "") { |
| 251 | if (!FLAGS_save_valid_only || valid) { |
| 252 | static int img_count = 0; |
| 253 | std::string image_name = absl::StrFormat("/img_%06d.png", img_count); |
| 254 | std::string path = FLAGS_save_path + image_name; |
| 255 | VLOG(2) << "Saving image to " << path; |
| 256 | cv::imwrite(path, rgb_image); |
| 257 | img_count++; |
| 258 | } |
| 259 | } |
milind-u | 8c72d53 | 2021-12-11 15:02:42 -0800 | [diff] [blame] | 260 | } |
| 261 | |
| 262 | void Calibration::HandleIMU(const frc971::IMUValues *imu) { |
Jim Ostrowski | ba2edd1 | 2022-12-03 15:44:37 -0800 | [diff] [blame] | 263 | // Need to check for valid values, since we sometimes don't get them |
| 264 | if (!imu->has_gyro_x() || !imu->has_gyro_y() || !imu->has_gyro_z() || |
| 265 | !imu->has_accelerometer_x() || !imu->has_accelerometer_y() || |
| 266 | !imu->has_accelerometer_z()) { |
| 267 | return; |
| 268 | } |
| 269 | |
| 270 | VLOG(2) << "IMU " << imu; |
milind-u | 8c72d53 | 2021-12-11 15:02:42 -0800 | [diff] [blame] | 271 | imu->UnPackTo(&last_value_); |
| 272 | Eigen::Vector3d gyro(last_value_.gyro_x, last_value_.gyro_y, |
| 273 | last_value_.gyro_z); |
| 274 | Eigen::Vector3d accel(last_value_.accelerometer_x, |
| 275 | last_value_.accelerometer_y, |
| 276 | last_value_.accelerometer_z); |
| 277 | |
James Kuszmaul | 969e4ab | 2023-01-28 16:09:19 -0800 | [diff] [blame] | 278 | // TODO: ToDistributedClock may be too noisy. |
milind-u | 8c72d53 | 2021-12-11 15:02:42 -0800 | [diff] [blame] | 279 | data_->AddImu(imu_factory_->ToDistributedClock(monotonic_clock::time_point( |
| 280 | chrono::nanoseconds(imu->monotonic_timestamp_ns()))), |
Austin Schuh | 5b37907 | 2021-12-26 16:01:04 -0800 | [diff] [blame] | 281 | gyro, accel * kG); |
milind-u | 8c72d53 | 2021-12-11 15:02:42 -0800 | [diff] [blame] | 282 | } |
| 283 | |
Stephan Pleines | f63bde8 | 2024-01-13 15:59:33 -0800 | [diff] [blame] | 284 | } // namespace frc971::vision |