FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

addressed review comments · Tensor-Robotics/navigation_stack@ccc7137 · GitHub

Repository navigation

Commit ccc7137

Browse files
addressed review comments
1 parent f4b73f7 commit ccc7137

2 files changed

Lines changed: 6 additions & 9 deletions

File tree

‎src/modules/localization/domain/dynamics/constant_acceleration_model.cpp‎

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -42,8 +42,8 @@ namespace localization::dynamics {
4242
0, dt_3*noise_ay/2, 0, 0, dt_2*noise_ay, 0, 0, dt*noise_ay, 0,
4343
0, 0, dt_3*noise_atheta/2, 0, 0, dt_2*noise_atheta, 0, 0, dt*noise_atheta,
4444
dt_2*noise_ax/2, 0, 0, dt*noise_ax, 0, 0, noise_ax, 0, 0,
45-
0, dt_2*noise_ay, 0, 0, dt*noise_ay, 0, 0, noise_ay, 0,
46-
0, 0, dt_2*noise_atheta, 0, 0, dt*noise_atheta, 0, 0, noise_atheta;
45+
0, dt_2*noise_ay/2, 0, 0, dt*noise_ay, 0, 0, noise_ay, 0,
46+
0, 0, dt_2*noise_atheta/2, 0, 0, dt*noise_atheta, 0, 0, noise_atheta;
4747

4848
}
4949

‎src/modules/localization/domain/dynamics/constant_acceleration_model.hpp‎

Lines changed: 4 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -13,14 +13,11 @@ namespace localization::dynamics {
1313
/**
1414
* @brief Construct a new Constant Acceleration Model object
1515
*
16-
* @param[in] state_covariance_x
17-
* @param[in] state_covariance_y
18-
* @param[in] state_covariance_theta
19-
* @param[in] process_noise_x
20-
* @param[in] process_noise_y
21-
* @param[in] process_noise_theta
16+
* @param[in] process_noise_ax Acceleration process noise in x
17+
* @param[in] process_noise_ay Acceleration process noise in y
18+
* @param[in] process_noise_atheta Acceleration process noise in theta
2219
*/
23-
ConstantAccelerationModel(double process_noise_x=5, double process_noise_y=5, double process_noise_theta=5);
20+
ConstantAccelerationModel(double process_noise_ax=5, double process_noise_ay=5, double process_noise_atheta=5);
2421

2522
/** @brief sets the matrix values depending on the dynamic model
2623
*

0 commit comments

Comments
 (0)

Back | FazBrowse Home | New Git URL