Skip to content

Commit 9f69702

Browse files
committed
some renaming and corrected a minor issue where in the gf tracker should work now for multiple objects (not tested)
1 parent 7b1e04d commit 9f69702

File tree

2 files changed

+8
-5
lines changed

2 files changed

+8
-5
lines changed

source/dbot/tracker/gaussian_tracker.cpp

Lines changed: 6 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -50,12 +50,15 @@ auto GaussianTracker::on_initialize(
5050

5151
auto GaussianTracker::on_track(const Obsrv& obsrv) -> State
5252
{
53+
// the following is approximately ok, but to be correct in a differential
54+
// geometry sense, the covariance matrix would also have to change due
55+
// to the parallel transport
56+
5357
State old_pose = belief_.mean();
54-
filter_->sensor().local_sensor().body_model().nominal_pose(
55-
old_pose);
58+
filter_->sensor().local_sensor().body_model().nominal_pose(old_pose);
5659

5760
State zero_pose = belief_.mean();
58-
zero_pose.component(0).set_zero_pose();
61+
zero_pose.set_zero_pose();
5962
belief_.mean(zero_pose);
6063

6164
filter_->predict(belief_, zero_input(), belief_);

source/dbot/tracker/particle_tracker.cpp

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -47,7 +47,7 @@ auto ParticleTracker::on_initialize(
4747

4848
for (size_t i = 0; i < filter_->belief().size(); i++)
4949
{
50-
filter_->belief().location(i).center_around_zero(delta_mean);
50+
filter_->belief().location(i).subtract(delta_mean);
5151
}
5252

5353
auto& integrated_poses = filter_->sensor()->integrated_poses();
@@ -64,7 +64,7 @@ auto ParticleTracker::on_track(const Obsrv& image) -> State
6464

6565
for (size_t i = 0; i < filter_->belief().size(); i++)
6666
{
67-
filter_->belief().location(i).center_around_zero(delta_mean);
67+
filter_->belief().location(i).subtract(delta_mean);
6868
}
6969

7070
auto& integrated_poses = filter_->sensor()->integrated_poses();

0 commit comments

Comments
 (0)