aslam_optimizer icon indicating copy to clipboard operation
aslam_optimizer copied to clipboard

When using an M-Estimator, the wrong cost is used

Open furgalep opened this issue 11 years ago • 5 comments

We are not using \rho(x) as the cost. We are using w(x) x^T x.

This should be changed.

furgalep avatar Jan 10 '14 12:01 furgalep

See here for a possibility of how we could implement M-Estimators. They have to return p(x), p'(x), p''(x).

http://homes.cs.washington.edu/~sagarwal/ceres-solver/stable/modeling.html#LossFunction

furgalep avatar Jan 10 '14 12:01 furgalep

Here is where the M-Estimator is applied:

https://github.com/ethz-asl/aslam_optimizer/blob/master/aslam_backend/src/ErrorTerm.cpp#L20-L24

furgalep avatar Jan 10 '14 13:01 furgalep

These also need attention:

https://github.com/ethz-asl/aslam_optimizer/blob/master/aslam_backend/include/aslam/backend/implementation/ErrorTerm.hpp#L169-L192

furgalep avatar Jan 10 '14 13:01 furgalep

And

https://github.com/ethz-asl/aslam_optimizer/blob/master/aslam_backend/src/ErrorTermDs.cpp#L144-L167

furgalep avatar Jan 10 '14 13:01 furgalep

Maybe other places as well. Thanks!

furgalep avatar Jan 10 '14 13:01 furgalep