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

[BUG] Accuracy overlay shows wrong values for Nailgun #164

Open
The-Gig opened this issue Jun 15, 2022 · 9 comments
Open

[BUG] Accuracy overlay shows wrong values for Nailgun #164

The-Gig opened this issue Jun 15, 2022 · 9 comments
Labels
bug Something went wrong. cgame Possible issue related to cgame (the clientside of things) game Possible issue related to game (the serverside of things) possibly native Nope, this wrong behavior turns out to be not our fault, but those of the original makers.

Comments

@The-Gig
Copy link
Contributor

The-Gig commented Jun 15, 2022

It's possible to bind "+acc" command to a key in order to get an "Accuracy overlay" shown while holding that key.
Example /bind l "+acc" will make the table show when you will be holding the "l" key.

But the percentages shown for Nailgun are clearly wrong. You may fire a single shot at close range and then see a 600% accuracy for that weapon, which makes no sense. I mean, if one or more nails of a single shot hit, you should consider it 1 hit/1 shot; if no nails hit, you should consider 0 hit/1 shot.

Maybe the problem could be somehow related to the fact that a single Nailgun shot actually fires 15 nails, which reach the target at different times.

Considerations:

  • Wondering whether "Accuracy" medal (which is single player deathmatch only, which is earned if you land more than 50% of shots) calculations are affected, too. Harder to test, I fear... By the way, considering Accuracy Overlay does not consider Gauntlet and Proximity Mines, I wonder whether Accuracy Medal considers them or not.
  • Shotgun seems to do not have the same issue. Considering that's an hitscan (instant hit) weapon, I guess the code is quite different.
  • While doing the testing, be aware that to update the stats shown there, you have to release and push the button again, as the table is not updated automatically. IDK if that's on purpose to be able to see the stats of a "static" moment instead of ever-changing values or to prevent excessive computational work... or just an overlook. Thinking about it, if the table would have been updated automatically, people would have been able to enable/disable the overlay by typing /+acc and /-acc in console, which may have been nice, although not really definitive anyway (just a simple vid_restart would have disabled it again).

shot0247

@NeonKnightOA

This comment was marked as abuse.

@NeonKnightOA NeonKnightOA added cgame Possible issue related to cgame (the clientside of things) game Possible issue related to game (the serverside of things) bug Something went wrong. labels Mar 14, 2024
@NeonKnightOA

This comment was marked as abuse.

@NeonKnightOA NeonKnightOA added the requires testing We need to be sure this has been fixed. Posts with this tag have D/L links. label Mar 22, 2024
@The-Gig
Copy link
Contributor Author

The-Gig commented Mar 25, 2024

Yes, I just shot a close range nailgun in OAX B53 and got 300% accuracy.

@NeonKnightOA

This comment was marked as abuse.

@NeonKnightOA NeonKnightOA removed the requires testing We need to be sure this has been fixed. Posts with this tag have D/L links. label Mar 25, 2024
@The-Gig
Copy link
Contributor Author

The-Gig commented Mar 26, 2024

IIRC, it's just the nailgun.
(Things mentioned in "considerations" part might refer to other weapons, though).

@NeonKnightOA

This comment was marked as abuse.

@The-Gig
Copy link
Contributor Author

The-Gig commented Mar 29, 2024

About "the situations the bug is triggered", to put it very shortly, just hit with nailgun point-blank and then check the accuracy overlay, it will show a value above 100%, which shouldn't be possible. Please read the opening post for more information.

Tested in 0.8.8, oax B53 and nightly build 2024-03-27_1c0587e. It can be safely assumed it's there also in all intermediate releases.

@NeonKnightOA

This comment was marked as abuse.

@NeonKnightOA

This comment was marked as abuse.

@NeonKnightOA NeonKnightOA added the possibly native Nope, this wrong behavior turns out to be not our fault, but those of the original makers. label Mar 29, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something went wrong. cgame Possible issue related to cgame (the clientside of things) game Possible issue related to game (the serverside of things) possibly native Nope, this wrong behavior turns out to be not our fault, but those of the original makers.
Projects
None yet
Development

No branches or pull requests

2 participants