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

Disable g-forces for UAV AI #5094

Merged
merged 1 commit into from
May 12, 2017
Merged

Disable g-forces for UAV AI #5094

merged 1 commit into from
May 12, 2017

Conversation

PabstMirror
Copy link
Contributor

A zeus can remote control UAV
Doing so means that typeOf ace_player = B_UAV_AI
(has some other weird effects)

This exits the gfroce pfeh if the unit is virtual or AI

@PabstMirror PabstMirror added the kind/bug-fix Release Notes: **FIXED:** label Apr 21, 2017
@PabstMirror PabstMirror added this to the 3.10.0 milestone Apr 21, 2017
@Zakant
Copy link
Contributor

Zakant commented Apr 22, 2017

How about moving the virtual unit detection to a more central component such as ace_common? Only one unit event handler would be needed an all components could grab the data. I think it would be useful for a few other cases as well.

@commy2
Copy link
Contributor

commy2 commented Apr 22, 2017

So #5093 only applies when remote controlling the drone, not when managing it as drone operator?

@Zakant
Copy link
Contributor

Zakant commented Apr 22, 2017

Can confirm. Just happening from zeus. UAV operators are uneffected.

Copy link
Member

@kymckay kymckay left a comment

Choose a reason for hiding this comment

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

Code all looks good

@PabstMirror PabstMirror merged commit b8a6a60 into master May 12, 2017
@PabstMirror PabstMirror deleted the gforces-uavAI branch May 12, 2017 18:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/bug-fix Release Notes: **FIXED:**
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants