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

how to drive the multirotor rotate on pitch angle and roll angle #4304

Closed
Cming7 opened this issue Jan 20, 2022 · 4 comments · Fixed by #4338
Closed

how to drive the multirotor rotate on pitch angle and roll angle #4304

Cming7 opened this issue Jan 20, 2022 · 4 comments · Fixed by #4338

Comments

@Cming7
Copy link

Cming7 commented Jan 20, 2022

Question

What's your question?

I want to drive the multirotor rotate on pitch angle and roll angle. Does multirotor support moveByAngleZAsync? What's the TODO meaning in the Unity APIs?

Include context on what you are trying to achieve

I just find the rotateToYawAsync API to drive the multirotor rorate on yaw angle. And the multirotor pose doesn't change using moveByAngleZAsync API.

client.moveByAngleZAsync(0, 0, -2, 30 / 180 * math.pi, 5).join()

Looking forward to your reply. Thank you so much!

@Cming7 Cming7 changed the title Does multirotor support moveByAngleZAsync? how to drive the multirotor rotate on pitch angle and roll angle Jan 20, 2022
@Gastastrophe
Copy link
Contributor

Are you building Airsim on Unity? If an API says TODO in the Unity APIs, then that means it works in Unreal Engine but not Unity.

@Cming7
Copy link
Author

Cming7 commented Jan 21, 2022

Are you building Airsim on Unity? If an API says TODO in the Unity APIs, then that means it works in Unreal Engine but not Unity.

I build the project on UE4.27.

@zimmy87
Copy link
Contributor

zimmy87 commented Feb 2, 2022

Hi @Cming7, the server binding for this API was removed in #2297. You'll want to use the moveByRollPitchYawZAsync API instead.

@zimmy87
Copy link
Contributor

zimmy87 commented Feb 2, 2022

I put out PR #4338 to fix this.

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

Successfully merging a pull request may close this issue.

4 participants