TeamRizu / OutFox

The Bug Reporting Repository for OutFox LTS 0.4, Alpha V and Steam Early Access Builds
https://projectoutfox.com
Apache License 2.0
187 stars 3 forks source link

[FIXED] [Feature Request] Adjust GetNoteData to return correct note type with holds #666

Closed Sudospective closed 1 year ago

Sudospective commented 1 year ago

Please Select the game mode your feature request is about.

All Game Modes

Is your feature request related to a problem?

Currently, when using notedata, all holds return "TapNoteSubType_Hold". There is no way to access information about the type of tail a hold or roll has. This information is potentially useful for themes, noteskins, and modfiles.

Describe the solution you'd like

Adjusting the way GetNoteData returns the notedata table to account for newer hold and roll types is an ideal solution.

Describe alternatives you've considered

No response

Additional context

No response

JoseVarelaP commented 1 year ago

Giving a shot at this, currently I have the output to show this:

return {
    {4.000000,1,"TapNoteSubType::TapNoteSubType_Hold","TapNote_4th",group = "TapNoteGroupType_Normal",length = 4.000000},
    {8.000000,2,"TapNoteSubType::TapNoteSubType_Hold","TapNote_4th",group = "TapNoteGroupType_Mine",length = 4.000000},
    {12.000000,3,"TapNoteSubType::TapNoteSubType_Hold","TapNote_4th",group = "TapNoteGroupType_Normal",length = 4.000000},
    {16.000000,3,"TapNoteType::TapNoteType_LongNoteTail","TapNote_4th",group = "TapNoteGroupType_Lift"},
}

The way it works is that the group item for this data set is present for all the items. However, the change here is that if there is a Lift Hold present, the lift note itself will be added to the table of items as a separate entity to ease parsing.

However, I'm open for suggestions about this implementation.

Sudospective commented 1 year ago

this looks straightforward to me, and it seems like it keeps in line with previous specs, i agree with it