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

Fix songs ending at incorrect times if the ini length was wrong #86

Merged
merged 1 commit into from
Apr 14, 2023

Conversation

RileyTheFox
Copy link
Collaborator

This fixes a bug with some tracks with incorrect song lengths in the song.ini from ending at the wrong time. It also implements the use of [end] events.

@EliteAsian123 EliteAsian123 merged commit dd6fe48 into YARC-Official:master Apr 14, 2023
EliteAsian123 added a commit that referenced this pull request Apr 15, 2023
#97)

* initial work on score mechanics

* initial work on score mechanics

* add tempo keeping, sustain score adding with tempo

* fix styling, sustain-tempo math

* fix line endings

* add condition to prevent out of bounds crash in updating curBeatPerSecond

* implement MaxSustainScore method

to be used in capping achievable sustain points

* fix MaxSustainPoints logic

* refactoring ScoreKeeper instance management

* try to make sustain scoring deterministic

* add Score_Box graphic; fix sustain scoring for if the note goes beyond beat markers

* refactoring

* add conventional BPM Play.tempo property

* Move some Utils methods to more relevant locations, and other misc. changes (#79)

* Move VolumeFromLinear from Utils to AudioManager

* Move SplitNoteToOctaveAndNote from Utils to LyricInfo

* Turn RectTransform utility methods into extension methods

* Move Hash from Utils to SongLibrary

* Use const case for sources URL constant name

* Unity keeps modifying this so w/e

* Add strum leniency to five fret guitar (#83)

* Ignore .idea folder

* Strum leniency added to FiveFretTrack

* Adjusted strum leniency to 65ms

* Comment fixes

* Update README.md

* Added some sources

* Allow anchoring chord HO/POs (togglable) (#85)

* Removed Patreon from Main Menu

* Added credits

* Add song length to Play.cs for song end (#86)

* Fix bot overstrumming (#87)

* Small comment for later

* Add update check on startup (#91)

* Created update checker, runs on startup

* Add current version and update notification to main menu

* Update version back to 0.8.2

* Moved engine constants to `Constants`

* Awesomeness Detection (#89)

* Awesomeness detection

EXTREMLY IMPORTANT

* Awesomeness Detection

Very important!

* Pretty awesome if you ask me

* move sustain tracking to separate file, other sus refactoring

* add scoring for drums

* add BarlowBold reference for scorebox

* Adding generation of currentsong.text (#93)

This is mainly for OBS to show on twitch streams.
More stuff can be added but let's see if this works right first, ya? :)

* "ThirdParty" folder

* Fix lefty flip on 4 and 5 lane drums (#96)

* 4 lane drum lefty flip fix

* 5 lane drum lefty flip fix

* Updated credits

* Add PlasticBand-Unity for input mappings, and HIDrogen for Linux HID support (#95)

* Add PlasticBand-Unity package

* Switch Pro Guitar support to PlasticBand's

* Update input actions
Guitar actions now use PlasticBand
Removed most guitar UI actions, as they are covered by usages in PlasticBand

* Add HIDrogen package for HID support on Linux

* implement scoring for all, move ScoreKeeper initialization, refactor

---------

Co-authored-by: Nathan <nathanhurst03@gmail.com>
Co-authored-by: RileyTheFox <jacobchambers@hotmail.co.uk>
Co-authored-by: EliteAsian <lavasnakegaming@gmail.com>
Co-authored-by: Raphael Goulart <contact@raphaelgoul.art>
Co-authored-by: TheFatBastid <tfb@sof.ws>
Co-authored-by: Rodrigo Oliveira <39157156+santorfo@users.noreply.github.com>
EliteAsian123 added a commit that referenced this pull request Apr 16, 2023
* initial work on score mechanics

* initial work on score mechanics

* add tempo keeping, sustain score adding with tempo

* fix styling, sustain-tempo math

* fix line endings

* add condition to prevent out of bounds crash in updating curBeatPerSecond

* implement MaxSustainScore method

to be used in capping achievable sustain points

* fix MaxSustainPoints logic

* refactoring ScoreKeeper instance management

* try to make sustain scoring deterministic

* add Score_Box graphic; fix sustain scoring for if the note goes beyond beat markers

* refactoring

* add conventional BPM Play.tempo property

* Move some Utils methods to more relevant locations, and other misc. changes (#79)

* Move VolumeFromLinear from Utils to AudioManager

* Move SplitNoteToOctaveAndNote from Utils to LyricInfo

* Turn RectTransform utility methods into extension methods

* Move Hash from Utils to SongLibrary

* Use const case for sources URL constant name

* Unity keeps modifying this so w/e

* Add strum leniency to five fret guitar (#83)

* Ignore .idea folder

* Strum leniency added to FiveFretTrack

* Adjusted strum leniency to 65ms

* Comment fixes

* Update README.md

* Added some sources

* Allow anchoring chord HO/POs (togglable) (#85)

* Removed Patreon from Main Menu

* Added credits

* Add song length to Play.cs for song end (#86)

* Fix bot overstrumming (#87)

* Small comment for later

* Add update check on startup (#91)

* Created update checker, runs on startup

* Add current version and update notification to main menu

* Update version back to 0.8.2

* Moved engine constants to `Constants`

* Awesomeness Detection (#89)

* Awesomeness detection

EXTREMLY IMPORTANT

* Awesomeness Detection

Very important!

* Pretty awesome if you ask me

* move sustain tracking to separate file, other sus refactoring

* add scoring for drums

* add BarlowBold reference for scorebox

* Adding generation of currentsong.text (#93)

This is mainly for OBS to show on twitch streams.
More stuff can be added but let's see if this works right first, ya? :)

* "ThirdParty" folder

* Fix lefty flip on 4 and 5 lane drums (#96)

* 4 lane drum lefty flip fix

* 5 lane drum lefty flip fix

* Updated credits

* Add PlasticBand-Unity for input mappings, and HIDrogen for Linux HID support (#95)

* Add PlasticBand-Unity package

* Switch Pro Guitar support to PlasticBand's

* Update input actions
Guitar actions now use PlasticBand
Removed most guitar UI actions, as they are covered by usages in PlasticBand

* Add HIDrogen package for HID support on Linux

* implement scoring for all, move ScoreKeeper initialization, refactor

* Fix lefty flip drum inputs (#99)

* fixed lefty flip drum inputs

* fixed bot drum lefty flip

* Re-enable incremental garbage collection (#101)

* Improve HO/PO overstrum forgiveness (#100)

* Allow anchoring chord HO/POs (togglable)

* Improve HO/PO overstrum forgiveness

* Small fix

* swap for adjusted score box sprite

* implement StarKeeper and set up with player tracks (sans vocals)

* add StarDisplay for Play UI

* add vocals and harmVocals star thresholds

* fix play scorebox graphic import

* add StarDisplay, refactor

score display is now its own class, now only updates on event (score add)

---------

Co-authored-by: Nathan <nathanhurst03@gmail.com>
Co-authored-by: RileyTheFox <jacobchambers@hotmail.co.uk>
Co-authored-by: EliteAsian <lavasnakegaming@gmail.com>
Co-authored-by: Raphael Goulart <contact@raphaelgoul.art>
Co-authored-by: TheFatBastid <tfb@sof.ws>
Co-authored-by: Rodrigo Oliveira <39157156+santorfo@users.noreply.github.com>
@RileyTheFox RileyTheFox deleted the end-event-fix branch April 16, 2023 21:10
Purplo-cf pushed a commit to Purplo-cf/YARG that referenced this pull request May 14, 2024
* Convert Lists of cache groups to dictionaries

Extracts the would-be key value out of the group types, instead passing the keys as function parameters.

* Encapsulate cachegroup locks in a LockedCacheDictionary type

Simplifies the "AddGroup" calling conventions
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants