Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Build failing: ‘class gtsam::Values’ has no member named ‘filter’ #1400

Closed
h2jaafar opened this issue Jan 21, 2023 · 2 comments
Closed

Comments

@h2jaafar
Copy link

Description

Trying to build GTSAM with the depreciated flag below, but the build is failing. Both myself and a colleague are having similar issues.

[ 84%] Building CXX object examples/CMakeFiles/StereoVOExample_large.dir/StereoVOExample_large.cpp.o
/home/hussein/gbp/gtsam/examples/StereoVOExample_large.cpp: In function ‘int main(int, char**)’:
/home/hussein/gbp/gtsam/examples/StereoVOExample_large.cpp:116:31: error: ‘class gtsam::Values’ has no member named ‘filter’
  116 |   Values pose_values = result.filter<Pose3>();
      |                               ^~~~~~
/home/hussein/gbp/gtsam/examples/StereoVOExample_large.cpp:116:43: error: expected primary-expression before ‘>’ token
  116 |   Values pose_values = result.filter<Pose3>();
      |                                           ^
/home/hussein/gbp/gtsam/examples/StereoVOExample_large.cpp:116:45: error: expected primary-expression before ‘)’ token
  116 |   Values pose_values = result.filter<Pose3>();
      |                                             ^

Steps to reproduce

git clone https://github.com/borglab/gtsam.git
cd gtsam
mkdir build && cd build
cmake -DGTSAM_ALLOW_DEPRECATED_SINCE_V42:=OFF .. 
sudo make install

Expected behavior

Build successfuly

Environment

Xubuntu 20.04, cmake version 3.24.0-rc3

Additional information

I had previously successfully built gtsam a few days ago. This may be due recent changes.

@dellaert
Copy link
Member

Thanks! Fixed in #1399 in develop which should be merged once reviewed by @jlblancoc or @varunagrawal , but maybe you are trying to compile release/4.2 ? I should make the changes there, too, PR coming!

@h2jaafar
Copy link
Author

Thank you for the prompt responses Dr!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants