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] Incorrect counting of HANDS #710

Open
1 task done
peter-vasut opened this issue Nov 9, 2023 · 1 comment
Open
1 task done

[BUG] Incorrect counting of HANDS #710

peter-vasut opened this issue Nov 9, 2023 · 1 comment
Assignees
Labels
Branch: Alpha-5 Bugs related to Alpha V mainline (0.5.x) Engine: Gameplay Gameplay Related Issues Priority: Low A feature or issue which can be worked on in free time. Status: Fix in Testing We have a fix made, and it's in testing! Type: Upstream Bug Bug inherited from upstream

Comments

@peter-vasut
Copy link

peter-vasut commented Nov 9, 2023

Is there an existing issue for this?

  • I have searched the existing issues and none matched.

Operating System

Linux

CPU

No response

GPU

No response

Storage

No response

Game Version

5.0.0-041

Game Mode

dance

Theme

default

Describe the problem

After playing every note and hold in the song, the stats show only 4/5 HANDS. The song has only 4 hands (see attached simfile).

Here is the relevant part of the simfile (I've removed zero lines and beat separators for clarity):

M202
00M0
1000
0010
1000
0020
0030
0303

Notice there are start of two holds, then 3 taps, then one hold that ends immediately, then the original holds end. So in total there are 3 taps + 1 hold that should be counted as hands.

image

Removing the short hold resolves the issue (making the resulting count 3/3). Replacing it with tap resolves the issue (making the resulting count 4/4). It looks like something weird is going on when there is this short hold.

Describe what should happen

The total number of hands should be 4, so that the stats will show 4/4 hands.

Relevant Log output

simfile (the beginning and end is intentionally blank, only affected part is included for testing)

@JoseVarelaP
Copy link
Member

Have made a fix, but I'm going to perform more tests to see how it affects RadarValues on similar looking charts.

Grabacion.de.pantalla.2024-02-06.a.la.s.7.45.57.p.m.mp4

@JoseVarelaP JoseVarelaP self-assigned this Feb 7, 2024
@JoseVarelaP JoseVarelaP added Type: Upstream Bug Bug inherited from upstream Engine: Gameplay Gameplay Related Issues Priority: Low A feature or issue which can be worked on in free time. Status: Fix in Testing We have a fix made, and it's in testing! Branch: Alpha-5 Bugs related to Alpha V mainline (0.5.x) labels Feb 7, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Branch: Alpha-5 Bugs related to Alpha V mainline (0.5.x) Engine: Gameplay Gameplay Related Issues Priority: Low A feature or issue which can be worked on in free time. Status: Fix in Testing We have a fix made, and it's in testing! Type: Upstream Bug Bug inherited from upstream
Projects
None yet
Development

No branches or pull requests

2 participants