Skip to content
This repository has been archived by the owner on Jul 22, 2021. It is now read-only.

Add alarm emergency response #162

Merged
merged 5 commits into from
Aug 24, 2020
Merged

Add alarm emergency response #162

merged 5 commits into from
Aug 24, 2020

Conversation

mxgrey
Copy link
Member

@mxgrey mxgrey commented Aug 24, 2020

This PR subscribes to the emergency topic so the full control robots can respond appropriately when an alarm is triggered.

@FloodShao
Copy link
Collaborator

Tested in office.world.
The following actions are performed:

  1. Send loop request to robots
  2. After the robots started moving, trigger the active alarm checkbox in the nav graph.
  3. The two magnis will stop and returned to the nearest waypoint that marked as "is_parking_spot".
    Most of the parking spots have a name on it in the nav graph except one:
- [814.828, 239.25, 0, "", {is_parking_spot: [4, true]}]

The robot once stopped at a non-named waypoint, and that waypoint is this non-named parking_spot.

The emergency_response should works as intended.

@FloodShao FloodShao self-requested a review August 24, 2020 08:13
Copy link
Collaborator

@FloodShao FloodShao left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The PR works as intended.

Copy link
Collaborator

@FloodShao FloodShao left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Tested in office.world.
The following actions are performed:

Send loop request to robots
After the robots started moving, trigger the active alarm checkbox in the nav graph.
The two magnis will stop and returned to the nearest waypoint that marked as "is_parking_spot".
Most of the parking spots have a name on it in the nav graph except one:

  • [814.828, 239.25, 0, "", {is_parking_spot: [4, true]}]
    The robot once stopped at a non-named waypoint, and that waypoint is this non-named parking_spot.

The emergency_response should works as intended.

@mxgrey mxgrey merged commit b864d20 into master Aug 24, 2020
@mxgrey mxgrey deleted the feature/emergency_response branch August 24, 2020 08:46
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants