Skip to content

Commit

Permalink
refactor: fix lint errors
Browse files Browse the repository at this point in the history
  • Loading branch information
ChandimaFernando committed Feb 28, 2024
1 parent 346fbce commit 71f9119
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 3 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -153,12 +153,13 @@ class PdfBeamtimeServer
/// @brief Set the current state to HOME and move robot to home position
bool reset_fsm();

/// @brief Handles bluesky interrupt to PAUSE
/// @brief Handles bluesky interrupt to PAUSE, STOP, ABORT, and HALT
void handle_pause();
void handle_stop();
void execute_stop();
void handle_abort();
void handle_halt();

/// @brief Handles bluesky interrupt to RESUME
void handle_resume();

Expand Down
4 changes: 2 additions & 2 deletions src/pdf_beamtime/src/pdf_beamtime_server.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -156,7 +156,7 @@ void PdfBeamtimeServer::execute(
feedback->status = get_action_completion_percentage();

RCLCPP_INFO(
node_->get_logger(), "Current state is state %s.",
node_->get_logger(), "Current state is %s.",
external_state_names_[static_cast<int>(current_state_)].c_str());

// Reset inner_state_machine at new goal
Expand Down Expand Up @@ -546,7 +546,6 @@ void PdfBeamtimeServer::handle_stop()

void PdfBeamtimeServer::execute_stop()
{
///@todo @ChandimaFernando This needs testing
switch (current_state_) {
case State::GRASP_SUCCESS:
// handle the decision to pick up something else after successfully grabing the sample
Expand All @@ -568,6 +567,7 @@ void PdfBeamtimeServer::execute_stop()
execute_stop();
break;

/// @todo @ChandimaFernando This needs testing
case State::PLACE:
set_current_state(State::PICKUP_RETREAT);
inner_state_machine_->set_internal_state(Internal_State::STOP);
Expand Down

0 comments on commit 71f9119

Please sign in to comment.