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

Headdirection based wind indication #1980

Merged
merged 2 commits into from
Aug 8, 2015

Conversation

MikeMatrix
Copy link
Contributor

Replaced direction detection with CBA_fnc_headDir to calculate the indicator direction off of the head direction instead of the player direction.

This solves #1892.

@PabstMirror PabstMirror added the kind/bug-fix Release Notes: **FIXED:** label Aug 2, 2015
@PabstMirror PabstMirror added this to the 3.3.0 milestone Aug 2, 2015
@PabstMirror PabstMirror added the kind/enhancement Release Notes: **IMPROVED:** label Aug 2, 2015
@PabstMirror
Copy link
Contributor

Great job, this should be perfect.

Only thing I can think of is that someone with TrackIR might want to try this, it might effect you differently with freelook always on.

@MikeMatrix
Copy link
Contributor Author

@PabstMirror I tried with TrackIR with the ACE Test Mission and it worked just fine.

@kymckay
Copy link
Member

kymckay commented Aug 2, 2015

👍 This is why CBA is so handy

@jonpas
Copy link
Member

jonpas commented Aug 2, 2015

Intruding, but is there set head direction function as well?

@kymckay
Copy link
Member

kymckay commented Aug 2, 2015

Not possible @jonpas

@jonpas
Copy link
Member

jonpas commented Aug 2, 2015

Thought so, I guess this concludes my search for it(cfgMoves better get ready for me lol), thanks @SilentSpike

thojkooi added a commit that referenced this pull request Aug 8, 2015
@thojkooi thojkooi merged commit ff421db into acemod:master Aug 8, 2015
@MikeMatrix MikeMatrix deleted the headDirBasedWindIndication branch August 25, 2015 01:21
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:** kind/enhancement Release Notes: **IMPROVED:**
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants