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

IndeterminantLinearSystemException when computing marginals #96

Closed
mbrossar opened this issue Aug 1, 2019 · 3 comments
Closed

IndeterminantLinearSystemException when computing marginals #96

mbrossar opened this issue Aug 1, 2019 · 3 comments
Milestone

Comments

@mbrossar
Copy link
Contributor

mbrossar commented Aug 1, 2019

I search to compute covariance marginals from a pose-graph that does not have any loop closure.

This happen even after slightly modifying Pose3SLAMExample_g2o as follow:
1-include marginal computation in examples/Pose3SLAMExample_g2o.cpp Pose3SLAMExample_g2o.cpp
2-remove loop-closure in examples/Data/pose3example.txt pose3example.txt

The optimization is done, but marginals computation is not done. Error message is:
terminate called after throwing an instance of
'gtsam::IndeterminantLinearSystemException' what():
Indeterminant linear system detected while working near variable 4 (Symbol: 4).

With prior of the first pose, I do not see why system could be underdetermined.

@dellaert
Copy link
Member

dellaert commented Aug 1, 2019

Hmm, this would make it similar to LocalizationExample. If you help me by creating a tiny PR that adds this example (and the file) as Pose3LocalizationExample.cpp, I'll try to repro and diagnose. If we fix it, it can remain as a new example :-)

@mbrossar
Copy link
Contributor Author

mbrossar commented Aug 2, 2019

Thank you, the PR has just be done.

@ProfFan ProfFan added this to the GTSAM 4.1 milestone Sep 23, 2019
@ProfFan
Copy link
Collaborator

ProfFan commented Sep 23, 2019

#121

@mbrossar mbrossar closed this as completed Apr 4, 2020
varunagrawal added a commit that referenced this issue Apr 20, 2021
b2144a712 Merge pull request #95 from borglab/feature/empty-str-default-arg
9f1e727d8 Merge pull request #96 from borglab/fix/cmake
97ee2ff0c fix CMake typo
64a599827 support empty strings as default args
7b14ed542 Merge pull request #94 from borglab/fix/cmake-messages
0978641fe clean up
5b9272557 Merge pull request #91 from borglab/feature/enums
56e6f48b3 Merge pull request #93 from borglab/feature/better-template
27cc7cebf better cmake messages
a6318b567 fix tests
b7f60463f remove export_values()
38304fe0a support for class nested enums
348160740 minor fixes
5b6d66a97 use cpp_class and correct module name
2f7ae0676 add newlines and formatting
6e7cecc50 remove support for enum value assignment
c1dc925a6 formatting
798732598 better pybind template
f6dad2959 pybind_wrapper fixes with formatting
7b4a06560 Merge branch 'master' into feature/enums
1982b7131 more comprehensive tests for enums
3a0eafd66 code for wrapping enums
398780982 tests for enum support

git-subtree-dir: wrap
git-subtree-split: b2144a712953dcc3e001c97c2ace791149c97278
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