search

ethz-asl / curves

A library of curves for estimation.
BSD 3-Clause "New" or "Revised" License
74 stars 28 forks source link

Updates to use the latest gtsam development branch #43

Closed sanderson77 closed 9 years ago

sanderson77 commented 9 years ago

Changes to use gtsam devel branch as of today (rev 61666f22d618010bc1a63298ef856741db62f1d3)

Changes are primarily related to include locations and the .value function interface

ethzasl-jenkins commented 9 years ago

Can one of the admins verify this patch?

rdube commented 9 years ago

Hi Sean, I checked out rev 61666f22d618010bc1a63298ef856741db62f1d3. I run into this error when making curves tests :

In file included from /home/renaud/projects/ctsm/devel/include/gtsam_unstable/nonlinear/Expression.h:22:0, from /home/renaud/projects/ctsm/devel/include/gtsam_unstable/nonlinear/ExpressionFactor.h:20, from /home/renaud/projects/ctsm/src/curves/curves/test/test_Expressions.cpp:12: /home/renaud/projects/ctsm/devel/include/gtsam_unstable/nonlinear/Expression-inl.h:58:7: error: redefinition of ‘class gtsam::JacobianMap’ class JacobianMap {

Don't you get this on your side?

sanderson77 commented 9 years ago

Something here looks wrong. Expression.h is no longer located in include/gtsam_unstable/nonlinear. Rather it is now in include/gtsam/nonlinear. Are you sure you are rebuilding all the deps?

rdube commented 9 years ago

Thanks! This solved the problem. I will merge now.