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

[Android] Fix fps issue when certain animation intervals are set #2162

Merged
merged 1 commit into from
Sep 18, 2024

Conversation

rh101
Copy link
Contributor

@rh101 rh101 commented Sep 18, 2024

Describe your changes

An issue was introduced with this PR #1504 that causes incorrect FPS when certain animation interval values are used, such as 1/30, which results in 50-53FPS on certain devices.

Fix was suggested by @halx99, and testing on both 60hz and 120Hz devices, the FPS now correctly matches the animation interval.

Issue ticket number and link

Checklist before requesting a review

For each PR

  • Add Copyright if it missed:
    - "Copyright (c) 2019-present Axmol Engine contributors (see AUTHORS.md)."

  • I have performed a self-review of my code.

    Optional:

    • I have checked readme and add important infos to this PR.
    • I have added/adapted some tests too.

For core/new feature PR

  • I have checked readme and add important infos to this PR.
  • I have added thorough tests.

@rh101 rh101 changed the title Fix fps issue when certain animation intervals are set [Android] Fix fps issue when certain animation intervals are set Sep 18, 2024
@halx99 halx99 added this to the 2.2.0 milestone Sep 18, 2024
@halx99 halx99 added the bug Something isn't working label Sep 18, 2024
@halx99 halx99 merged commit ef52043 into axmolengine:dev Sep 18, 2024
15 checks passed
@halx99 halx99 linked an issue Sep 18, 2024 that may be closed by this pull request
@rh101 rh101 deleted the android-fps-fix branch September 18, 2024 15:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Android - Wrong FPS/update on high rate devices
2 participants