Skip to content
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

Warning in flotilla when obstacle is detected #893

Closed
anetteu opened this issue Jul 18, 2023 · 9 comments
Closed

Warning in flotilla when obstacle is detected #893

anetteu opened this issue Jul 18, 2023 · 9 comments
Assignees
Labels
backend Backend related functionality feature New feature or request frontend Frontend related functionality

Comments

@anetteu
Copy link
Member

anetteu commented Jul 18, 2023

Describe the new feature you would like to see
Consider adding a warning/pop-up in flotilla when the robot is detecting an obstacle in its path and waiting for it to move.

Describe the solution you'd like
Solution would propagate through ISAR. Frontend solution in flotilla could be for instance a pop-up that dissapears if the obstacle moves out of the way.

How will this feature affect the current Threat Model?
Leave blank for maintainers to fill out if you are uncertain about this.

@anetteu anetteu added feature New feature or request backend Backend related functionality frontend Frontend related functionality ux Issue needs input from UX designer labels Jul 18, 2023
@mariannewiggen
Copy link

mariannewiggen commented Aug 2, 2023

Suggest adding:

  • A “Sitter fast”/”Stuck” status on robot card
  • Banner with error message describing the problem
  • Notification button showing the latest error messages
  • A "See all notifications" button which leads to an overview of all notifications

Link to Figma: https://www.figma.com/file/q6vsNRogzV6r9vTCCDsocd/Robot-Missions?type=design&node-id=4640%3A716543&mode=dev

Image

Image

Image

@UsamaEquinorAFK UsamaEquinorAFK self-assigned this Aug 11, 2023
@UsamaEquinorAFK
Copy link
Contributor

@UsamaEquinorAFK
Copy link
Contributor

Flotilla - #989

@UsamaEquinorAFK
Copy link
Contributor

UsamaEquinorAFK commented Oct 6, 2023

New Implementaion as suggested by @oysand

Isar-robot: equinor/isar-robot#89
Flotilla: #1047
isar: equinor/isar#491

All of these are draft from now. Can you test it and review please. I will clean up the commit history soon as well :)

@UsamaEquinorAFK
Copy link
Contributor

Fixed this guy :isar -taurob one is here https://github.com/equinor/isar-taurob/pull/319

@UsamaEquinorAFK
Copy link
Contributor

The "See all notification " part can be handled by a seperate card. I will make an issue for that

@aeshub
Copy link
Contributor

aeshub commented Dec 31, 2023

This issue has automatically been marked as stale as there has been no activity for 60 days.

@aeshub aeshub added the stale This issue or pull request already exists label Dec 31, 2023
@Eddasol Eddasol removed stale This issue or pull request already exists ux Issue needs input from UX designer labels Mar 20, 2024
@anetteu
Copy link
Member Author

anetteu commented Apr 6, 2024

Can this issue be closed? Restoring issue and moving to new for team to consider if it can be closed.

@Eddasol
Copy link
Contributor

Eddasol commented Apr 8, 2024

Already solved

@Eddasol Eddasol closed this as completed Apr 8, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backend Backend related functionality feature New feature or request frontend Frontend related functionality
Projects
None yet
Development

No branches or pull requests

5 participants