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

Removing equality sign from _accumulator in FlxGame #202

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

Conversation

winterismute
Copy link

Hi! As far as I know, this is a known fix for the FlxGame class. Removing the equality sign prevents some frames from being erroneously skipped, and could result in smoother games.
I was wondering: why is it not in the updated community version?
Also, I know that what I am going to do is extremely but practice, but: I have also issues this pool requests for the power tools, which are also maintained by this account: FlixelCommunity/Flixel-Power-Tools#22
Although probably the FPT are not so used anymore, those bugs are pretty big, so it would be cool if someone could merge them and prevent characters from moving at huge speeds with no reason :D
Thank you in advance, I promise I will work on some scheduled issue next time!

… prevents some frames from being erroneously skipped, and could result in smoother games.
@WingEraser
Copy link

#32

@Dovyski
Copy link
Member

Dovyski commented Mar 19, 2014

Hi there! First of all every contribution is always welcome, no need to work on a scheduled issue :)

About your pull request, as @WingEraser mentioned, you didn't include that in that last release because there was not enough testing. You could just change the <= part, but we must check everything first.

Regarding your Flixel Power Tools pull, that's awesome! We haven't decided quite yet how to work and maintain FPT. For now your pull request will remain the waiting queue until we discuss and plan the next steps.

@winterismute
Copy link
Author

Hi again!
Well, it's really a minority anyway... I have read the issue referenced but it does not really say much, in general the idea is that you don't want to "step" twice or more unless is very necessary (you need the simulation to catch-up with the real time), and suppose that updating and drawing take the exact time, you can end up calling an extra step often while instead you would like to call immediately a draw to "push" immediately a frame.

For the FPT, I think they are pretty useful. If you decide to maintain them, I think that commit (or an improved version of it :D) should be in - those bugs are just crazy.
Hope I can help with something more in the future.

@IQAndreas IQAndreas added this to the Future release milestone May 1, 2014
@DigiEggz
Copy link

DigiEggz commented Jun 2, 2016

I can confirm that a frame-by-frame analysis shows big improvements to scrolling smoothness just by removing that pesky equality sign.

Even before I tested each frame, there was a very visible sort of jerking or heaving motion that background sprites make about every second when moving left/right at a constant speed. After removing the equality sign, the problem goes away immediately.

I've also been testing this fix for over a year now and I haven't experienced any negative effects from this change.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants