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

Duplicate frame preview at keyframes #1616

Closed
5 tasks done
dominikWin opened this issue Jun 5, 2023 · 3 comments
Closed
5 tasks done

Duplicate frame preview at keyframes #1616

dominikWin opened this issue Jun 5, 2023 · 3 comments

Comments

@dominikWin
Copy link

I have a lot of issues to go through, so in order to make it easier for me to help you, I ask that you please try these things first

Operating System

MacOS 13

Steps to reproduce

  1. Open a file in Lossless Cut
  2. Move current time a few frames before a keyframe
  3. Step one frame at a time forward until you pass the keyframe

Occurs on every file I tried including ones directly output from ffmpeg in h264.

A video where frame 499 is skipped in Lossless Cut:

lossless_cut_dup_keyframe.mp4

Expected behavior

The preview window should show the frame matching the timelines timestamp.

Actual behavior

A frame at the key frame is duplicated.

Share log

No response

@mifi
Copy link
Owner

mifi commented Jun 10, 2023

See #156 #585

tldr: with html5 video player, it's not possible to seek to a frame number, but to a time value, so losslesscut has implemented the "seek by frame" as a seek to a time (time divided by fps + 1):

const shortStep = useCallback((direction) => {

might be something wrong there with the calculation, or maybe your file has irregular/variable fps. Hard to say without having a file that reproduces the problem. I tried with an mp4/h264 file and I don't see this behavior

@hqhbaby
Copy link

hqhbaby commented Jun 19, 2023

The same problem, using LosslessCut to cut and combine Youtube's webm format, also has the problem of Duplicate frame preview at keyframes

@mifi
Copy link
Owner

mifi commented Feb 16, 2024

closing in favor of #1216

@mifi mifi closed this as completed Feb 16, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants