Funkin icon indicating copy to clipboard operation
Funkin copied to clipboard

[FIX] Finally fixes the stuttering that was caused via offsets

Open KoloInDaCrib opened this issue 1 year ago • 4 comments

Does this PR close any issues? If so, link them below.

3667 + like 3 others that were closed due to being duplicates

Briefly describe the issue(s) fixed.

Considering traces are back (hooray!) I was able to determine the culprit of the "minuscule" lags, ty eric image

so yeah i changed the difference check from 5ms to 10ms, hopefully there shouldn't be any more input errors until another update to offsets lol Rest in Peace the "Swing Mode" glitch saga you were too funny to last

Include any relevant screenshots or videos.

Before: https://github.com/user-attachments/assets/23cee938-390c-40fe-b9b5-f8260c913adc

After: https://github.com/user-attachments/assets/39023c8f-c87d-49cb-862a-128a9b6500c9

KoloInDaCrib avatar Oct 19 '24 19:10 KoloInDaCrib

This is so peak

NotHyper-474 avatar Oct 19 '24 19:10 NotHyper-474

It's not caused primarily by offsets

biomseed avatar Oct 19 '24 21:10 biomseed

Can we have more people test this, even while also messing with the input offset settings to make sure it's finally fixed fr this time?

trayfellow avatar Oct 20 '24 01:10 trayfellow

Can we have more people test this, even while also messing with the input offset settings to make sure it's finally fixed fr this time?

You can just compile the game with these changes and test it out yourself - I'll definitely do a bit more testing myself and archive this if theres more errors than I could fix Edit: I have ran a test of all the erect songs and have not found a single time the offspring of the swing mode glitch occurred. The worst of the glitches that happened was a bit of lag at the start of the song, but its probably due to caching the song, or rather lack there of. I should probably make all the insts and vocals get precached during the preload sequence at some point.

KoloInDaCrib avatar Oct 20 '24 07:10 KoloInDaCrib

I should probably make all the insts and vocals get precached during the preload sequence at some point.

I think that isn't in the scope of this PR

biomseed avatar Oct 20 '24 13:10 biomseed

I think that isn't in the scope of this PR

ye i meaint in another pr lol, but yeah no offset math should now be of any problem as i have tested a bunch of songs both with additional offsets and without em

KoloInDaCrib avatar Oct 20 '24 14:10 KoloInDaCrib

so yeah i changed the difference check from 5ms to 10ms, hopefully there shouldn't be any more input errors until another update to offsets lol Rest in Peace the "Swing Mode" glitch saga you were too funny to last

Is this the way to reproduce the bug? If so, I can gh checkout the PR and see if it's fixed on my end.. If there's anything else you want me to replicate, please do let me know 👍

doggogit avatar Oct 20 '24 14:10 doggogit

Is this the way to reproduce the bug? If so, I can gh checkout the PR and see if it's fixed on my end.. If there's anything else you want me to replicate, please do let me know 👍

Literally just play any song. If you notice any stuttering during the song where the notes would just suddenly move backwards, thats the bug that I was trying to fix

KoloInDaCrib avatar Oct 20 '24 14:10 KoloInDaCrib

Is this the way to reproduce the bug? If so, I can gh checkout the PR and see if it's fixed on my end.. If there's anything else you want me to replicate, please do let me know 👍

Literally just play any song. If you notice any stuttering during the song where the notes would just suddenly move backwards, thats the bug that I was trying to fix

Sorry, let me rephrase, I wanted to ask what you set the input offset to, didn't mean to overcomplicate my question

doggogit avatar Oct 20 '24 14:10 doggogit

Sorry, let me rephrase, I wanted to ask what you set the input offset to, didn't mean to overcomplicate my question

Oh, this bug was ongoing for any offset, including 0

KoloInDaCrib avatar Oct 20 '24 14:10 KoloInDaCrib

Sorry, let me rephrase, I wanted to ask what you set the input offset to, didn't mean to overcomplicate my question

Oh, this bug was ongoing for any offset, including 0

Oh, huh, weird, I haven't encountered that bug on 0.5.2 or 0.5.1 as a matter of fact

doggogit avatar Oct 20 '24 14:10 doggogit

Oh, huh, weird, I haven't encountered that bug on 0.5.2 or 0.5.1 as a matter of fact

Try a different FPS limit, preferably lower

biomseed avatar Oct 20 '24 15:10 biomseed

If the stutter varies from song to song, then modded songs could still be affected

Agreed, if Fresh pico mix caused a difference leading to the PR getting edited again, then there's no telling what other modded songs could do

trayfellow avatar Oct 24 '24 07:10 trayfellow

this should get the high priority tag

M7theguy avatar Oct 25 '24 15:10 M7theguy

Agreed, if Fresh pico mix caused a difference leading to the PR getting edited again, then there's no telling what other modded songs could do

This also applies to new songs added in the base game

biomseed avatar Oct 25 '24 16:10 biomseed

yes dude this is the only reason ive been playing on v0.5.0 instead of the latest (v0.5.3)

Beelijah avatar Nov 21 '24 22:11 Beelijah