Skip to content

Commit

Permalink
typecast to size_t to remove build warnings
Browse files Browse the repository at this point in the history
  • Loading branch information
varunagrawal committed Jun 13, 2019
1 parent e75c3ba commit 7395cdd
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 2 deletions.
2 changes: 1 addition & 1 deletion gtsam/base/cholesky.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -111,7 +111,7 @@ bool choleskyPartial(Matrix& ABC, size_t nFrontal, size_t topleft) {
return true;

assert(ABC.cols() == ABC.rows());
assert(ABC.rows() >= topleft);
assert(size_t(ABC.rows()) >= topleft);
const size_t n = static_cast<size_t>(ABC.rows() - topleft);
assert(nFrontal <= size_t(n));

Expand Down
2 changes: 1 addition & 1 deletion gtsam/linear/HessianFactor.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -505,7 +505,7 @@ VectorValues HessianFactor::solve() {

// Do Cholesky Factorization
const size_t n = size();
assert(info_.nBlocks() == n + 1);
assert(size_t(info_.nBlocks()) == n + 1);
info_.choleskyPartial(n);
auto R = info_.triangularView(0, n);
auto eta = linearTerm();
Expand Down

0 comments on commit 7395cdd

Please sign in to comment.