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 GetAngleInBetween parameter references #6525

Merged
merged 7 commits into from
Nov 12, 2024
Merged

Conversation

relent0r
Copy link
Contributor

@relent0r relent0r commented Nov 10, 2024

Description of the proposed changes

Fixes a small bug in the refactor of GetAngleInBetween. It was references indexes but the function that calls it is using named keys.

fixes #6524

p.s this needs someone smarter than me to validate I'm actually fixes this correctly as I'm unsure.

Testing done on the proposed changes

Tested to make sure the scathis does not error and shoots things.

Checklist

  • Changes are annotated, including comments where useful
  • Changes are documented in the changelog for the next game version

@relent0r relent0r requested a review from lL1l1 November 10, 2024 06:45
Copy link
Contributor

@lL1l1 lL1l1 left a comment

Choose a reason for hiding this comment

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

Good find for my refactor messing up the mathematics, it should indeed be z1 * z1 and z2 * z2.

I tested the changed math with my fixed Scathis and they make the fake barrels align with the aim barrel more accurately (when you view the wireframe skeletons).

lua/utilities.lua Outdated Show resolved Hide resolved
lua/utilities.lua Outdated Show resolved Hide resolved
lua/utilities.lua Outdated Show resolved Hide resolved
@lL1l1
Copy link
Contributor

lL1l1 commented Nov 10, 2024

Note: caused by #6438
You can also simply add this PR's number to the snippet other.6438.md for the changelog requirement, since it is essentially part of the same patch.

@relent0r
Copy link
Contributor Author

Good find for my refactor messing up the mathematics, it should indeed be z1 * z1 and z2 * z2.

I tested the changed math with my fixed Scathis and they make the fake barrels align with the aim barrel more accurately (when you view the wireframe skeletons).

Thanks, I can see your making the updates in 6528 for function call, once you merge that in I'll update this to match.

@lL1l1 lL1l1 merged commit 5fa3faf into FAForever:develop Nov 12, 2024
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Bug]: faf develop scathis does not fire
2 participants