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

Fix the inconsistency between the aiming prediction time and the actual time #71406

Merged
merged 1 commit into from
Feb 1, 2024

Conversation

wwkk222208
Copy link
Contributor

@wwkk222208 wwkk222208 commented Feb 1, 2024

Summary

None

Purpose of change

When players use the "to steady your aim" method to approach the accuracy limit, they will find that the aiming time prediction of precision shooting mode is different from the actual time spent. The purpose of this PR is to fix this problem.

Describe the solution

The function "Character::gun_engagement_moves" called in the original code is used to roughly estimate the gun's aiming performance in the item interface. It is different from the algorithm used in the actual aiming process. So I use the function “predict_recoil” to replace it.

Describe alternatives you've considered

Testing

Tested in game and find the bug has been fixed.

Additional context

@github-actions github-actions bot added [C++] Changes (can be) made in C++. Previously named `Code` astyled astyled PR, label is assigned by github actions json-styled JSON lint passed, label assigned by github actions BasicBuildPassed This PR builds correctly, label assigned by github actions labels Feb 1, 2024
@Maleclypse Maleclypse merged commit 4aee1f3 into CleverRaven:master Feb 1, 2024
23 of 30 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
astyled astyled PR, label is assigned by github actions BasicBuildPassed This PR builds correctly, label assigned by github actions [C++] Changes (can be) made in C++. Previously named `Code` json-styled JSON lint passed, label assigned by github actions
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants