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

Implement localization as part of a mission #1277

Merged
merged 19 commits into from
Dec 19, 2023
Merged

Conversation

aeshub
Copy link
Contributor

@aeshub aeshub commented Dec 19, 2023

No description provided.

aeshub and others added 18 commits December 19, 2023 16:04
The localization procedure is now implemented and will run as part of
regular mission scheduling.

If the queue is empty a localization mission will be started for the
current deck. This assumes that the operator has confirmed that the
robot is on the deck of the mission that has been scheduled.

If there is an existing mission the system will check if a new mission
is in the same deck as that mission and if so schedule it. If not it
will be rejected.

If the last mission finishes a return to home mission will be scheduled
which puts the robot back at the default localization pose. If a
mission is scheduled in between the return to home mission another
localization will not be required.
@aeshub aeshub self-assigned this Dec 19, 2023
Copy link

🔔 Changes in database folder detected 🔔
Do these changes require adding new migrations? 🤔 In that case follow these steps.
If you are uncertain, ask a database admin on the team 😄

Copy link

🔔 Migrations changes detected 🔔
📣 Remember to comment "/UpdateDatabase" after review approval for migrations to take effect!

@github-actions github-actions bot added the database-change Will require migration label Dec 19, 2023
Copy link
Contributor

@andchiind andchiind left a comment

Choose a reason for hiding this comment

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

LGTM

@aeshub aeshub changed the title Localization Implement localization as part of a mission Dec 19, 2023
@aeshub
Copy link
Contributor Author

aeshub commented Dec 19, 2023

/UpdateDatabase

Copy link

👀 Running migration command... 👀

@aeshub aeshub added feature New feature or request breaking-change A breaking change which introduces changes to the public APIs labels Dec 19, 2023
Copy link

❌ Migration failed, please see action log below for details ❌
https://github.com/equinor/flotilla/actions/runs/7263778710

@aeshub
Copy link
Contributor Author

aeshub commented Dec 19, 2023

/UpdateDatabase

Copy link

👀 Running migration command... 👀

Copy link

✨ Successfully ran migration command! ✨

@aeshub aeshub merged commit 2b90e68 into equinor:main Dec 19, 2023
10 checks passed
@aeshub aeshub deleted the localization branch December 19, 2023 16:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
breaking-change A breaking change which introduces changes to the public APIs database-change Will require migration feature New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants