FunkinCrew / Funkin

A rhythm game made with HaxeFlixel
https://www.newgrounds.com/portal/view/770371
Other
2.95k stars 2.29k forks source link

Bug Report: "Total notes" on the results screen does not reflect the actual number of notes in a song. #3016

Open NewCs386148 opened 3 months ago

NewCs386148 commented 3 months ago

Issue Checklist

What platform are you using?

Newgrounds (Web)

If you are playing on a browser, which one are you using?

Microsoft Edge

Version

0.4.1

Context (Provide images, videos, etc.)

I played 2hot and ended up getting 89% despite my "sicks" and "goods" totaling to about 91% of the total notes. I added up the numbers of each category, and it turns out that somehow there are 17 more notes there than there are in the total notes section of the screen.

Score: 2hotScore

Math: 2hotMath

After playing the song again, the "total notes" was an entirely different number! However, adding up the numbers of each individual accuracy category still adds up to 585, and the % cleared still uses 585 as the number of notes in the song to calculate its percentage. 2hotScore2 2hotMath2

Steps to reproduce (or crash logs, errors, etc.)

Play any song in the game and don't get a full combo. In this example here, I played the tutorial on normal and missed the whole first turn. TutorialLMAO

Hundrec commented 3 months ago

This is a really strange bug that seems to only affect the displayed value for "Total Notes".

From your report, I'm getting that none of the calculations related to the total number of notes were affected, right?

Keoiki commented 3 months ago

Based on the 3rd picture, you'd think Total Notes means the total notes you HIT in the song, not total notes in the song in general.

But putting together the numbers, excluding misses, gives 566 in the first picture (2 lower than what Total Notes says) and 561 in the second picture (1 lower than what Total Notes says)

What gives?

NewCs386148 commented 3 months ago

From your report, I'm getting that none of the calculations related to the total number of notes were affected, right?

Yes. Despite the displayed number being wrong, it still calculates percentages with the real number of total notes (which seems to be the number of all five accuracy categories added up).

Hundrec commented 3 months ago

Maybe there's a rare chance for the game to skip counting a hit note if there's a lagspike?

NewCs386148 commented 3 months ago

Possibly. I find it strange that the game seems to calculate total notes while you're playing the song instead of deriving that value from the chart itself.

Keoiki commented 3 months ago

OK This isn't even a bug, it's like that on purpose as that number shows the totalNotesHit: https://github.com/FunkinCrew/Funkin/blob/9b483de1da9c1b72b02ec30f4d13923470cf32b9/source/funkin/play/ResultState.hx#L368 https://github.com/FunkinCrew/Funkin/blob/9b483de1da9c1b72b02ec30f4d13923470cf32b9/source/funkin/play/ResultState.hx#L371

But I still don't understand why either: the total number is wrong or the sick/good/bad/shit numbers are wrong (based on what I said in my previous comment)

Hundrec commented 3 months ago

So that leaves the off-by-a-few issue as well as a need for clarification that it's "Total Notes Hit" rather than "Total Notes in the song"

NewCs386148 commented 3 months ago

a need for clarification that it's "Total Notes Hit" rather than "Total Notes in the song"

They could make it say something like "Total Notes: [Total notes hit]/[Total notes in song]" instead of "Total Notes: [Total notes hit]". They'd have to make it a scrolling textbox or something to keep it off the yellow background, but it would properly communicate both values.

amyspark-ng commented 1 week ago

true! this is pretty weird