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

Several bugfixes & improvements to stats #1461

Merged
merged 1 commit into from
Dec 23, 2024
Merged

Several bugfixes & improvements to stats #1461

merged 1 commit into from
Dec 23, 2024

Conversation

Pablete1234
Copy link
Member

@Pablete1234 Pablete1234 commented Dec 23, 2024

Fixes a few issues with stats:

  • Custom projectiles hitting players with resistance would cause negative damage to be recorded
    • Extreme cases (resistance level 255 and rage projectiles) could cause damages in the -300k range per projectile
  • Bow shots and bow lands were tracking differently, causing mismatches like:
    • Shooting a custom projectile that launches an arrow (eg: La Piazza's crossbow) would increase your landed shots but not your "shot" arrows, meaning you could get 100% bow accuracy.
    • Shooting a bow that modifies the projectile (eg: Glacial impact shooting snowballs) would increase your shots but would never count towards landed projectiles, meaning your accuracy would drop even if you land your projectiles.
  • The stats menu now supports FFA, prior to this it wouldn't give you the item at all.
  • The stats menu was not centered for a bunch of team sizes (it really only worked well for even amounts), so 3-team or 5-team maps would be off-centered. Now they all display in a more reasonable manner.
  • The stats menu always reserved just 2 rows for teams, now it will be flexible from 2 to 4 rows depending on how many teams there are. This applies to FFA too.

@Pablete1234 Pablete1234 merged commit 8808c28 into dev Dec 23, 2024
2 checks passed
@Pablete1234 Pablete1234 deleted the fix-stats branch December 23, 2024 23:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants