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

Support for import_qgroundcontrol_mission #106

Closed
petergerten opened this issue Sep 4, 2019 · 2 comments
Closed

Support for import_qgroundcontrol_mission #106

petergerten opened this issue Sep 4, 2019 · 2 comments
Labels

Comments

@petergerten
Copy link
Contributor

Hi,

it seems there is currently no support for the import_qgroundcontrol_mission() method.
I am a bit confused on how this works. If the wrapper is auto-generated, wouldn't it have the same functionality as the "main" MAVSDK?

Peter

@JonasVautherin
Copy link
Collaborator

Hello!

The support exists in C++, but not yet in the other languages. Maybe a part of this presentation can help on the wording there.

To summarize, it is planned to autogenerate everything down to the C++ API, meaning that everything we have in C++ will always exist in all other languages. Currently, the "glue code" (or what I call "black box" in the presentation) is not auto-generated, so it has to be written manually. And QGC mission import was not added there (yet).

We would be happy if you could contribute that! The first step would be to add the definition in the corresponding proto file, and then write the "glue code" (gRPC) 👍.

Does that make sense?

@JonasVautherin
Copy link
Collaborator

It is now in 0.7.0, though I realize now that it works in a weird way: await drone.mission.import_qgroundcontrol_mission(qgc_plan_path) takes the path to the qgc plan, which will be opened by mavsdk_server. I'm not sure what's the best way to improve that. But assuming that you run MAVSDK-Python and mavsdk-server on the same machine, it should be fine.

We would be glad if you could try it and let us know if it works!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants