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

#800 causes Levenberg-Marquardt optimisation to crash #809

Closed
hmellor opened this issue Jun 30, 2021 · 4 comments · Fixed by #823
Closed

#800 causes Levenberg-Marquardt optimisation to crash #809

hmellor opened this issue Jun 30, 2021 · 4 comments · Fixed by #823
Assignees
Labels
bug Bug report
Milestone

Comments

@hmellor
Copy link
Contributor

hmellor commented Jun 30, 2021

Description

When updating to the latest commit of the develop branch my existing codebase that uses GTSAM started crashing when running LM.

There is no stack-trace, only an error saying the process has died:

[ERROR] [fgo-1]: process has died [pid 8821, exit code -11,
    cmd '/home/user/ros_ws/install/fgo/lib/fgo/fgo--ros-args -r __node:=fgo
    --params-file /tmp/launch_params__dkrh3ic --params-file /tmp/launch_params_8v56gi4f
    --params-file /tmp/launch_params_btvjwucu'].

I have narrowed it down to 6ee17c2 which comes from #800. Checking out the previous commit causes the program to no longer crash.

It may help to list some classes I'm using:

  • NonlinearFactorGraph
  • CombinedImuFactor
  • GPSFactor

Expected behavior

Should not crash

Environment

  • OS
    • Distributor ID: Ubuntu
    • Description: Ubuntu 20.04.2 LTS
    • Release: 20.04
    • Codename: focal
  • ROS
    • Foxy
  • TBB Version (relevant to replace deprecated tbb functionality #800)
    • libtbb-dev/focal,now 2020.1-2 amd64 [installed]
    • libtbb2/focal,now 2020.1-2 amd64 [installed,automatic]
  • Language
    • C++
@ProfFan ProfFan self-assigned this Jul 6, 2021
@ProfFan ProfFan added the bug Bug report label Jul 6, 2021
@ProfFan
Copy link
Collaborator

ProfFan commented Jul 6, 2021

@acxz Could you take a look?

@ProfFan
Copy link
Collaborator

ProfFan commented Jul 6, 2021

Also, @hmellor could you provide a minimal reproducer (one cpp file, no other dependencies)?

@dellaert
Copy link
Member

dellaert commented Jul 7, 2021

@acxz this was your PR - could you weigh in? @hmellor thanks for letting us know the TBB versions you used.

@acxz
Copy link
Contributor

acxz commented Jul 7, 2021

This might be an issue similar to why the testGncOptimize test failed once out of 100 runs. I could take a deeper look into this issue with a minimal reproducing example.

In the meanwhile, it may be better to just revert my PR.

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

Successfully merging a pull request may close this issue.

4 participants