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

[BUGFIX] Remove unnecessary player vocals muting #3012

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

gamerbross
Copy link
Contributor

ghost missing muted the player vocals, theres no point

Copy link
Contributor

@tposejank tposejank left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this also broke songs with only bf vocal track merged.

Copy link

@AbnormalPoof AbnormalPoof left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hoping this gets merged since it gets very awkward on songs that do not have split vocals (Basically every mod)

ghost_miss_bug.mp4

@KutikiPlayz
Copy link

what if, instead of fully removing this, you just unmute the vocals when the opponent sings for songs that have the vocals in one file, like how the game used to work

@tposejank
Copy link
Contributor

what if, instead of fully removing this, you just unmute the vocals when the opponent sings for songs that have the vocals in one file, like how the game used to work

See the problem with that is the game expects people use separate voice tracks as it's more convenient, and isn't likely they want ot do that instead of just easily not muting the bf vocal track when a ghost miss occurs.

@AbnormalPoof
Copy link

AbnormalPoof commented Jul 12, 2024

See the problem with that is the game expects people use separate voice tracks as it's more convenient

The game actually DOES look for Voices.ogg as a last resort. The vocal muting most likely wasn't accounted for that though, unfortunately.
Screenshot 2024-07-12 at 2 34 53 PM

@KutikiPlayz
Copy link

this is my point, I don't mean do it all the time I mean specifically if the vocals are combined
better yet, just put your combined vocals as the opponent vocals and leave the bf vocals empty or missing
solves the problem ez pz

@tposejank
Copy link
Contributor

Indeed it does look, but it may soon feel like a deprecated feature

@Hundrec
Copy link
Contributor

Hundrec commented Jul 12, 2024

this is my point, I don't mean do it all the time I mean specifically if the vocals are combined

better yet, just put your combined vocals as the opponent vocals and leave the bf vocals empty or missing

solves the problem ez pz

#2837 reports that this doesn't solve the problem (unless you replace BF's vocals with silence)

@EliteMasterEric EliteMasterEric added status: pending triage The bug or PR has not been reviewed yet. type: minor bug Involves a minor bug or issue. small A small pull request with 10 or fewer changes labels Jul 12, 2024
@EliteMasterEric
Copy link
Member

Getting this PR to work such that it functions if the player misses on a single vocal track would be great for people looking to port mods.

@tposejank
Copy link
Contributor

Getting this PR to work such that it functions if the player misses on a single vocal track would be great for people looking to port mods.

FNF Porter reference!!!

@gamerbross
Copy link
Contributor Author

gamerbross commented Jul 13, 2024

Getting this PR to work such that it functions if the player misses on a single vocal track would be great for people looking to port mods.

I've been working on a true solution for that. I have 2 solution proposals:

  • Unmute voices when the opponent hits a note (just when the game detects only player vocal track).
    Cons:

    • You will still hear the player voices in a duet part and will just sound with stutter.
    • Will keep the sound stutter in songs with a true player only vocals, like Camellia.
  • (i think its the best) An internal timer in the VoiceGroup object that resets to 0 everytime the player misses, when the timer is done (duration will be hold note lenghth, or for single notes 0.5s) unmute the player vocal track.
    Pros:

    • Less stutter than the other solution in duet parts.
    • Lyrics parts without notes don't get muted, like TwiddleFinger. (most of the PRs I've done lately it's because I've been porting it lol)

Cons:

  • uhh nothing i think

@EliteMasterEric plz i need your opinion on these so i can make a cool pr

@tposejank
Copy link
Contributor

I dunnoooooo new property time??

@EliteMasterEric
Copy link
Member

I've been thinking about it, and maybe vocal muting should just straight up be disabled if there is no opponent vocal track.

Pros:

  • This ensures the opponent's voice is never muted even during duets.

Cons:

  • The player not muting on hit is inconsistent with the normal behavior, but with only one vocal track something will have to be inconsistent from normal anyway.

@tposejank
Copy link
Contributor

Exactly.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
small A small pull request with 10 or fewer changes status: pending triage The bug or PR has not been reviewed yet. type: minor bug Involves a minor bug or issue.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants