Skip to content

Add relative altitude on GoToAction - #116

Draft
angel-ayala wants to merge 1 commit into
developfrom
fix/takeoff-climb
Draft

Add relative altitude on GoToAction#116
angel-ayala wants to merge 1 commit into
developfrom
fix/takeoff-climb

Conversation

@angel-ayala

Copy link
Copy Markdown
Collaborator

Introduces a POC to set the takeoff altitude relative to the current aircrat location as demanded in the SDK documentation for the GoToAction

Related to #107

The public logger of WenuLinkHandler is only to check if everything is set

Introduces a POC to set the takeoff altitude relative to the current aircrat location as demanded in the SDK documentation for the GoToAction

Related to #107
@TimonSchreiber

Copy link
Copy Markdown
Collaborator

I am not sure about this pr @angel-ayala.

Based on the dji documentation I see no reason to believe that the altitude in the GoToAction is a relative altitude, see dji-api
The phrase "from the current aircraft position" describes the start of the movement, not the reference frame of the altitude. It is not ideally worded, but otherwise the paramater would be named "relativeAltitude" or something similar.

Testing this in a simulator might help, but I am not convinced this solves the problem.

@angel-ayala

Copy link
Copy Markdown
Collaborator Author

I understand what you mean.
I just tested with the simulation feature and works the same way that the version without this relative altitude, so it must be tested with the real aircraft to see the actual behavior.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants