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

Short R3F compat update according with the new AB drag models #5587

Merged
merged 1 commit into from
Oct 6, 2017
Merged

Short R3F compat update according with the new AB drag models #5587

merged 1 commit into from
Oct 6, 2017

Conversation

Laid3acK
Copy link
Contributor

@Laid3acK Laid3acK commented Oct 4, 2017

  • Revert to the default G7 drag model for the 5.56x45 and 7.62x51 bullets.
  • Update all airFriction according with the new ACE3's ones.

@jonpas jonpas added area/compatibility kind/enhancement Release Notes: **IMPROVED:** labels Oct 4, 2017
@jonpas jonpas added this to the 3.11.0 milestone Oct 4, 2017
@jonpas
Copy link
Member

jonpas commented Oct 4, 2017

Merge after #5566?

@dedmen
Copy link
Contributor

dedmen commented Oct 4, 2017

Doesn't have to be after #5566 But wouldn't hurt.
No dependencies here.

@jonpas
Copy link
Member

jonpas commented Oct 4, 2017

Yes that's what I meant, it will work better after #5566, in case of any issues that might prevent merge of it (hope not!).

@Laid3acK
Copy link
Contributor Author

Laid3acK commented Oct 4, 2017

The R3F airFriction are designed to work with the vanilla ballistic (R3F SCROME scopes and BDC reticles).
The ballistic in-game is more accurate with the old or the new ACE3 airFriction in all cases.
So, you can merge it when you want : now, after #5566 or with the ACE3 3.12.
It's not really a problem, the result in-game will be necessarily better with the ACE3 values. :)

@jonpas jonpas merged commit 3a5fcda into acemod:master Oct 6, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/compatibility kind/enhancement Release Notes: **IMPROVED:**
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants