[VK]: Rework frame interpolation toggle & implement support for PC #340
No reviewers
Labels
No labels
android
audio
bug
ci
cmake
confirmed
core
desktop
duplicate
enhancement
frontend
fw
git
graphics
loader
nv gpu
ui
vulkan
No milestone
No project
No assignees
2 participants
Notifications
Due date
No due date set.
Dependencies
No dependencies set.
Reference
eden-emu/eden!340
Loading…
Add table
Add a link
Reference in a new issue
No description provided.
Delete branch "frame-interp-rework"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
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.
563c3bb2893fafbaa00eView command line instructions
Checkout
From your project repository, check out a new branch and test the changes.