-
Notifications
You must be signed in to change notification settings - Fork 310
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
fix multiple system unconfigured rise cleanup #1175
Changes from all commits
63b725d
5ad0199
653f2d0
5184402
40a3d14
1c238f0
7f39020
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change | ||||||||
---|---|---|---|---|---|---|---|---|---|---|
|
@@ -197,6 +197,11 @@ return_type Sensor::read(const rclcpp::Time & time, const rclcpp::Duration & per | |||||||||
{ | ||||||||||
// TODO(destogl): discuss what should be default return value, e.g., "NOT_EXECUTED" | ||||||||||
return_type result = return_type::ERROR; | ||||||||||
if (impl_->get_state().id() == lifecycle_msgs::msg::State::PRIMARY_STATE_UNCONFIGURED) | ||||||||||
{ | ||||||||||
return return_type::OK; | ||||||||||
} | ||||||||||
|
||||||||||
Comment on lines
+200
to
+204
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more.
Suggested change
|
||||||||||
if ( | ||||||||||
impl_->get_state().id() == lifecycle_msgs::msg::State::PRIMARY_STATE_INACTIVE || | ||||||||||
impl_->get_state().id() == lifecycle_msgs::msg::State::PRIMARY_STATE_ACTIVE) | ||||||||||
|
Original file line number | Diff line number | Diff line change | ||||||||||
---|---|---|---|---|---|---|---|---|---|---|---|---|
|
@@ -216,6 +216,11 @@ return_type System::read(const rclcpp::Time & time, const rclcpp::Duration & per | |||||||||||
{ | ||||||||||||
// TODO(destogl): discuss what should be default return value, e.g., "NOT_EXECUTED" | ||||||||||||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more.
Suggested change
|
||||||||||||
return_type result = return_type::ERROR; | ||||||||||||
if (impl_->get_state().id() == lifecycle_msgs::msg::State::PRIMARY_STATE_UNCONFIGURED) | ||||||||||||
{ | ||||||||||||
return return_type::OK; | ||||||||||||
} | ||||||||||||
|
||||||||||||
Comment on lines
+219
to
+223
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more.
Suggested change
|
||||||||||||
if ( | ||||||||||||
impl_->get_state().id() == lifecycle_msgs::msg::State::PRIMARY_STATE_INACTIVE || | ||||||||||||
impl_->get_state().id() == lifecycle_msgs::msg::State::PRIMARY_STATE_ACTIVE) | ||||||||||||
|
@@ -233,6 +238,11 @@ return_type System::write(const rclcpp::Time & time, const rclcpp::Duration & pe | |||||||||||
{ | ||||||||||||
// TODO(destogl): discuss what should be default return value, e.g., "NOT_EXECUTED" | ||||||||||||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more.
Suggested change
|
||||||||||||
return_type result = return_type::ERROR; | ||||||||||||
if (impl_->get_state().id() == lifecycle_msgs::msg::State::PRIMARY_STATE_UNCONFIGURED) | ||||||||||||
{ | ||||||||||||
return return_type::OK; | ||||||||||||
} | ||||||||||||
|
||||||||||||
Comment on lines
+241
to
+245
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more.
Suggested change
|
||||||||||||
if ( | ||||||||||||
impl_->get_state().id() == lifecycle_msgs::msg::State::PRIMARY_STATE_INACTIVE || | ||||||||||||
impl_->get_state().id() == lifecycle_msgs::msg::State::PRIMARY_STATE_ACTIVE) | ||||||||||||
|
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.