Skip to content

Commit 286107b

Browse files
committed
try remove GTSAM_EXPORT from cpp
1 parent 9712acd commit 286107b

File tree

4 files changed

+25
-22
lines changed

4 files changed

+25
-22
lines changed

gtsam/geometry/SO3.cpp

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -261,7 +261,6 @@ Matrix3 SO3::Hat(const Vector3& xi) {
261261

262262
//******************************************************************************
263263
template <>
264-
GTSAM_EXPORT
265264
Vector3 SO3::Vee(const Matrix3& X) {
266265
Vector3 xi;
267266
xi(0) = -X(1, 2);

gtsam/geometry/SO4.cpp

Lines changed: 0 additions & 21 deletions
Original file line numberDiff line numberDiff line change
@@ -135,27 +135,6 @@ SO4 SO4::Expmap(const Vector6& xi, ChartJacobian H) {
135135
}
136136

137137
//******************************************************************************
138-
template <>
139-
GTSAM_EXPORT
140-
SO4::VectorN2 SO4::vec(OptionalJacobian<16, 6> H) const {
141-
const Matrix& Q = matrix_;
142-
if (H) {
143-
H->setZero();
144-
H->block<4, 1>(0, 2) = -Q.col(3);
145-
H->block<4, 1>(0, 4) = -Q.col(2);
146-
H->block<4, 1>(0, 5) = Q.col(1);
147-
H->block<4, 1>(4, 1) = Q.col(3);
148-
H->block<4, 1>(4, 3) = Q.col(2);
149-
H->block<4, 1>(4, 5) = -Q.col(0);
150-
H->block<4, 1>(8, 0) = -Q.col(3);
151-
H->block<4, 1>(8, 3) = -Q.col(1);
152-
H->block<4, 1>(8, 4) = Q.col(0);
153-
H->block<4, 1>(12, 0) = Q.col(2);
154-
H->block<4, 1>(12, 1) = -Q.col(1);
155-
H->block<4, 1>(12, 2) = Q.col(0);
156-
}
157-
return Eigen::Map<const SO4::VectorN2>(Q.data());
158-
}
159138

160139
///******************************************************************************
161140
template <>

gtsam/geometry/SO4.h

Lines changed: 22 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -74,6 +74,28 @@ GTSAM_EXPORT Matrix3 topLeft(const SO4 &Q, OptionalJacobian<9, 6> H = {});
7474
*/
7575
GTSAM_EXPORT Matrix43 stiefel(const SO4 &Q, OptionalJacobian<12, 6> H = {});
7676

77+
template <>
78+
GTSAM_EXPORT
79+
SO4::VectorN2 SO4::vec(OptionalJacobian<16, 6> H) const {
80+
const Matrix& Q = matrix_;
81+
if (H) {
82+
H->setZero();
83+
H->block<4, 1>(0, 2) = -Q.col(3);
84+
H->block<4, 1>(0, 4) = -Q.col(2);
85+
H->block<4, 1>(0, 5) = Q.col(1);
86+
H->block<4, 1>(4, 1) = Q.col(3);
87+
H->block<4, 1>(4, 3) = Q.col(2);
88+
H->block<4, 1>(4, 5) = -Q.col(0);
89+
H->block<4, 1>(8, 0) = -Q.col(3);
90+
H->block<4, 1>(8, 3) = -Q.col(1);
91+
H->block<4, 1>(8, 4) = Q.col(0);
92+
H->block<4, 1>(12, 0) = Q.col(2);
93+
H->block<4, 1>(12, 1) = -Q.col(1);
94+
H->block<4, 1>(12, 2) = Q.col(0);
95+
}
96+
return Eigen::Map<const SO4::VectorN2>(Q.data());
97+
}
98+
7799
#if GTSAM_ENABLE_BOOST_SERIALIZATION
78100
template <class Archive>
79101
/** Serialization function */

gtsam/linear/NoiseModel.h

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -575,6 +575,9 @@ namespace gtsam {
575575
* An isotropic noise model created by specifying a precision
576576
*/
577577
static shared_ptr Precision(size_t dim, double precision, bool smart = true) {
578+
if (0 == precision) {
579+
return nullptr;
580+
}
578581
return Variance(dim, 1.0/precision, smart);
579582
}
580583

0 commit comments

Comments
 (0)