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

[49.18] Secondary turret firing arc display does not reflect turret rotation #5215

Closed
gsparks3 opened this issue Mar 4, 2024 · 3 comments
Closed
Assignees
Labels

Comments

@gsparks3
Copy link
Collaborator

gsparks3 commented Mar 4, 2024

On a multi-turret vehicle (such as the Gulltoppr OmniMonitor), rotation of the FT (front turret) using the turret twist mini-menu does not visually update the firing arcs for the weapon. The primary turret updates as normal.

In this image, the turret was rotated to point northeast prior to re-opening the FT rotation menu:
image

While the visual display is incorrect, the actual turret rotation appears to be correct.
image

This is likely related to #2659 , as the FT here uses the same rotation menu as the mech turrets there.

@HammerGS HammerGS added the Bug label Apr 11, 2024
@HoneySkull HoneySkull self-assigned this Apr 17, 2024
@HoneySkull
Copy link
Collaborator

If TacOps Vehicle Firing Arcs is enabled, should the second turret also have the same arc width as the main turret?

@gsparks3
Copy link
Collaborator Author

TO:AUE page 159 says they both follow the standard turret rules except that the forward turret can't fire through the rear hexside, so seems like that is correct.

@HoneySkull
Copy link
Collaborator

Okay - that's what I thought, in the code the 2nd turret is hard coded to a forward arc regardless of the tacOps setting - I think this should be the same logic as the main turret. (note: This is the arc type not facing)

image

HammerGS added a commit that referenced this issue Apr 17, 2024
+ PR #5366: Added Faction Logos from UCC (With Permission)
+ Fix #5215 - Secondary turret firing arc display to reflect tanks
turret rotation.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants