Skip to content

Commit e836488

Browse files
inuex35claude
andcommitted
Address Copilot review: fix serialization and print formatting
Include std_optional_serialization.h for portable std::optional serialization and fix inconsistent newline in print() labels. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
1 parent 985e852 commit e836488

File tree

2 files changed

+3
-2
lines changed

2 files changed

+3
-2
lines changed

gtsam/navigation/PseudorangeFactor.cpp

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -168,7 +168,7 @@ void PseudorangeFactorArm::print(const std::string& s,
168168
gtsam::print(satClkBias_, "sat clock bias (s): ");
169169
gtsam::print(Vector(bL_), "lever arm (body frame meters): ");
170170
if (ecef_T_nav_) {
171-
ecef_T_nav_->print("ecef_T_nav:\n");
171+
ecef_T_nav_->print("ecef_T_nav: ");
172172
}
173173
}
174174

@@ -265,7 +265,7 @@ void DifferentialPseudorangeFactorArm::print(
265265
gtsam::print(satClkBias_, "sat clock bias (s): ");
266266
gtsam::print(Vector(bL_), "lever arm (body frame meters): ");
267267
if (ecef_T_nav_) {
268-
ecef_T_nav_->print("ecef_T_nav:\n");
268+
ecef_T_nav_->print("ecef_T_nav: ");
269269
}
270270
}
271271

gtsam/navigation/PseudorangeFactor.h

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -6,6 +6,7 @@
66
**/
77
#pragma once
88

9+
#include <gtsam/base/std_optional_serialization.h>
910
#include <gtsam/geometry/Point3.h>
1011
#include <gtsam/geometry/Pose3.h>
1112
#include <gtsam/nonlinear/NoiseModelFactorN.h>

0 commit comments

Comments
 (0)