Bug 439748 - Composite and transform keyframe timestamps are off
Summary: Composite and transform keyframe timestamps are off
Status: RESOLVED FIXED
Alias: None
Product: kdenlive
Classification: Applications
Component: User Interface (show other bugs)
Version: 22.08.1
Platform: Appimage Linux
: NOR normal
Target Milestone: ---
Assignee: Jean-Baptiste Mardelle
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2021-07-11 08:00 UTC by Andrej M.
Modified: 2022-09-23 07:16 UTC (History)
1 user (show)

See Also:
Latest Commit:
Version Fixed In: 22.08.2


Attachments
Broken keyframe timestamps (43.69 KB, image/png)
2021-07-11 08:00 UTC, Andrej M.
Details
Broken Composite and transform timestamp in 22.04.02 (59.79 KB, image/png)
2022-07-07 14:48 UTC, Andrej M.
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Andrej M. 2021-07-11 08:00:17 UTC
Created attachment 139988 [details]
Broken keyframe timestamps

Dear maintainers,

since kdenlive 21.04.x the Composite and transform keyframe timestamps don't represent the actual selected timestamps.

Examples:
- clip A is 3 seconds long, starting keyframe is 00:00:03:00 ending is 00:00:05:59
- clip B is 3 seconds long, starting keyframe is 00:00:03:23 ending is 00:00:06:23
- clip C is 5 seconds long, starting keyframe is 00:00:03:42 ending is 00:00:08:41

Please see attachment for more info.

The timestamps have thus become unusable for quick navigation as calculations are too time consuming.

Best regards,
Andrej
Comment 1 Andrej M. 2021-09-11 07:35:08 UTC
Still a problem with 21.08.x. Tested on both: Distro package and AppImage.
Comment 2 Andrej M. 2022-03-30 15:30:59 UTC
Still a problem with 21.12.0
Comment 3 Andrej M. 2022-07-07 14:43:46 UTC
Hi,

still an issue with 22.04, and on further testing I can confirm that the calculations behind this follow a nonsense linear equation.

In 22.04 I created a clip with duration 00:00:00:10 added Composite and transform:
Here is the map of timeline position => value in keyframe box
position 00:00:00:00 => value 00:00:00:00
position 00:00:00:10 => value 00:00:00:01
position 00:00:00:25 => value 00:00:00:02
position 00:00:00:41 => value 00:00:00:03
position 00:00:00:59 => value 00:00:00:04
position 00:00:01:15 => value 00:00:00:05

Etc.

If position is X and value is Y, the formula is something like Y=0.06134*X+0.3991 ...

Best regards,
Andrej
Comment 4 Andrej M. 2022-07-07 14:48:36 UTC
Created attachment 150468 [details]
Broken Composite and transform timestamp in 22.04.02
Comment 5 Julius Künzel 2022-09-21 17:45:53 UTC
Thanks for your report and sorry for the late reply!

The dev team is very small and there lots of things to do. Sometimes we miss important reports unfortunately. However in such cases it is good if you bring it up the chats or the mailinglist like you did. Hopefully we can fix this soon.
Comment 6 Julius Künzel 2022-09-21 22:06:48 UTC
Git commit 246682a095700df3081b258bb0623659209336b8 by Julius Künzel.
Committed on 21/09/2022 at 22:05.
Pushed by jlskuz into branch 'release/22.08'.

Fix wrong timecode offset in keyframewidget of transitions
FIXED-IN: 22.08.2

M  +0    -3    src/assets/view/widgets/keyframewidget.cpp

https://invent.kde.org/multimedia/kdenlive/commit/246682a095700df3081b258bb0623659209336b8
Comment 7 Andrej M. 2022-09-23 07:16:11 UTC
@ Julius Künzel 

Thank you so much for this fix. I tested the 22.11.70 git build and it works as expected now.

Thanks again and best regards,
Andrej