[VK]: Rework frame interpolation toggle & implement support for PC #340

Open
Gamer64 wants to merge 4 commits from frame-interp-rework into master
Member

Rework & rebase of #217.

This stops using the toggle from JNI (that was veeeery messy) and just uses it from settings.h, making things more clear, including a nuke of the actually disabled frameskip.

if (frame_skipping)
frame_skip_threshold = (target_fps == 30) ? 2 : 2;

if (frame_counter % frame_skip_threshold != 0)
if (frame_interpolation && previous_frame)

Here's a preview of the old logic. We had frame_skipping disabled, so it wasn't working as expected. Frame interpolation would only get initialized or work properly if frame_skipping was enabled, as frame_interpolation was dependent on frame_skip_threshold, which was only being initialized if frame_skipping was enabled. This meant frame interpolation never worked properly (if it even did with the messy JNI implementation). Now with these changes, we've removed the old problematic and useless code of frameskip and brought support to PC, making this feature available beyond just Android.

Rework & rebase of https://git.eden-emu.dev/eden-emu/eden/pulls/217. This stops using the toggle from JNI (that was veeeery messy) and just uses it from settings.h, making things more clear, including a nuke of the actually disabled frameskip. if (frame_skipping) frame_skip_threshold = (target_fps == 30) ? 2 : 2; if (frame_counter % frame_skip_threshold != 0) if (frame_interpolation && previous_frame) Here's a preview of the old logic. We had frame_skipping disabled, so it wasn't working as expected. Frame interpolation would only get initialized or work properly if frame_skipping was enabled, as frame_interpolation was dependent on frame_skip_threshold, which was only being initialized if frame_skipping was enabled. This meant frame interpolation never worked properly (if it even did with the messy JNI implementation). Now with these changes, we've removed the old problematic and useless code of frameskip and brought support to PC, making this feature available beyond just Android.
CamilleLaVey force-pushed frame-interp-rework from 563c3bb289
All checks were successful
eden-license / license-header (pull_request) Successful in 23s
to 3fafbaa00e
All checks were successful
eden-license / license-header (pull_request) Successful in 23s
2025-08-29 03:22:57 +02:00
Compare
All checks were successful
eden-license / license-header (pull_request) Successful in 23s
Required
Details
This pull request has changes conflicting with the target branch.
  • src/android/app/src/main/res/values-ar/strings.xml
  • src/android/app/src/main/res/values-ckb/strings.xml
  • src/android/app/src/main/res/values-cs/strings.xml
  • src/android/app/src/main/res/values-de/strings.xml
  • src/android/app/src/main/res/values-es/strings.xml
  • src/android/app/src/main/res/values-fa/strings.xml
  • src/android/app/src/main/res/values-fr/strings.xml
  • src/android/app/src/main/res/values-he/strings.xml
  • src/android/app/src/main/res/values-hu/strings.xml
  • src/android/app/src/main/res/values-id/strings.xml
  • src/android/app/src/main/res/values-it/strings.xml
  • src/android/app/src/main/res/values-ja/strings.xml
  • src/android/app/src/main/res/values-ko/strings.xml
  • src/android/app/src/main/res/values-nb/strings.xml
  • src/android/app/src/main/res/values-pl/strings.xml
  • src/android/app/src/main/res/values-pt-rBR/strings.xml
  • src/android/app/src/main/res/values-pt-rPT/strings.xml
  • src/android/app/src/main/res/values-ru/strings.xml
  • src/android/app/src/main/res/values-sr/strings.xml
  • src/android/app/src/main/res/values-uk/strings.xml
  • src/android/app/src/main/res/values-vi/strings.xml
  • src/android/app/src/main/res/values-zh-rCN/strings.xml
  • src/android/app/src/main/res/values-zh-rTW/strings.xml
  • src/common/settings.h
  • src/qt_common/config/shared_translation.cpp
View command line instructions

Checkout

From your project repository, check out a new branch and test the changes.
git fetch -u origin frame-interp-rework:frame-interp-rework
git switch frame-interp-rework
Sign in to join this conversation.
No description provided.