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.

Sign up for GitHub

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

Jump to bottom

Drone flight status update #16

Open
erwang01 opened this issue Jan 23, 2021 · 7 comments
Open

Drone flight status update #16

erwang01 opened this issue Jan 23, 2021 · 7 comments

Comments

@erwang01
Copy link
Contributor

No description provided.

@erwang01
Copy link
Contributor Author

Is this even needed?

@erwang01
Copy link
Contributor Author

What kind of fail safes can this provide us? Is this worth it? How will we keep the flight status updated esp considering DJI doesn't provide us with updates.

@erwang01
Copy link
Contributor Author

LOL anyone know know what I was talking about?

@DragonKyro
Copy link
Contributor

I think you were referring to the flight_status enum in drone base class

@DragonKyro
Copy link
Contributor

Currently unsure when a flight_status should be changed. Need to discuss

@erwang01
Copy link
Contributor Author

erwang01 commented May 1, 2021

flight_status enum tracks the drone's current state (in a state machine type manner)

Do we need to track this? If so, how do we ensure that this is consistent with both VR and the physical drone?

@erwang01
Copy link
Contributor Author

erwang01 commented May 1, 2021

Check out VR interface's matrice_ros_drone_connection.cs has some version of this implementation as well.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@erwang01 @DragonKyro