-
Notifications
You must be signed in to change notification settings - Fork 113
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
[RST-2186] Added a FixedLagSmootherParams struct #68
Conversation
&FixedLagSmoother::optimizerTimerCallback, | ||
this); | ||
|
||
// Advertise a service that resets the optimizer to its initial state | ||
reset_service_server_ = private_node_handle_.advertiseService("reset", &FixedLagSmoother::resetServiceCallback, this); | ||
reset_service_server_ = node_handle_.advertiseService( |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This switch from private_node_handle_
to node_handle_
is intentional. It is in the spirit of locusrobotics/fuse_rl#10
std::string reset_service { "reset" }; | ||
|
||
/** | ||
* @brief The maximum time to wait for motion models to be generated for a received transactions. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
"a received transaction" or "received transactions"?
/** | ||
* @brief The maximum time to wait for motion models to be generated for a received transactions. | ||
* | ||
* Transactions are processes sequentially, so no new transactions will be added to the graph while waiting for |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
*processed
* @param[in] default_value - A default value to use if the provided parameter name does not exist | ||
* @return The loaded (or default) value | ||
*/ | ||
double getPositiveParam(const ros::NodeHandle& node_handle, const std::string& parameter_name, double default_value) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Not for this PR, but if we can't release some of our internal utility methods like this, we should consider adding some of them to a utility header in this package.
Added a FixedLagSmootherParams struct that is responsible for holding and loading all of the Fixed-lag Smoother settings.
This currently does not include the set of parameters loaded and used by the Optimizer base class. A future PR will make an OptimizerParams base class that loads the common parameters.