Bug 462654

Summary:	A transform effect changes the rotoscoped area
Product:	[Applications] kdenlive	Reporter:	Bernd <berndmj>
Component:	Video Effects & Transitions	Assignee:	Jean-Baptiste Mardelle <jb>
Status:	RESOLVED FIXED
Severity:	normal
Priority:	NOR
Version First Reported In:	22.08.3
Target Milestone:	---
Platform:	Other
OS:	Linux
Latest Commit:		Version Fixed/Implemented In:
Sentry Crash Report:
Attachments:	screen recording

Description Bernd 2022-12-05 01:28:10 UTC

SUMMARY
This may be similar to bug #447786
An image on V2 is rotoscoped and a transform effect is added (in order to move the image around). This changes the rotoscope area (apparently it squishes it horizontally).

STEPS TO REPRODUCE
1. Put a color clip on V1, an image on V2
2. Add rotoscope effect to image on V2 and rotoscope an area
3. Add a transform effect to image on V2

OBSERVED RESULT
Rotoscope area is deformed/squished horizontally and hence a different area of the image is visible.

EXPECTED RESULT
No change to the rotoscoped area

SOFTWARE/OS VERSIONS
Linux: Pop!_OS 22.04 LTS 64-bit
GNOME: 42.5
Windowing System: X11
(available in About System)
KDENLIVE: 22.08.3 (but replicated in  22.04.3 and 21.04.3)
KDE Frameworks Version: 5.98.0
Qt Version: 5.15.5
MLT: 7.10.0

ADDITIONAL INFORMATION

Comment 1 Bernd 2022-12-05 01:36:16 UTC

Created attachment 154322 [details]
screen recording

Comment 2 Bernd 2022-12-05 14:48:01 UTC

This only happens if the rotoscoped image has a different resolution than the project. If the project has the same resolution than the image a transform effect does not change the rotoscoped area.

Comment 3 Bernd 2022-12-05 17:51:39 UTC

Bugs 455296 and 455288 describe the same issue

Comment 4 Jean-Baptiste Mardelle 2022-12-07 08:29:17 UTC

Thanks for your report. I started working on this issue and have an almost working patch. I hope to push the fix to MLT by the end of the week.

Comment 5 Bernd 2022-12-07 15:30:57 UTC

(In reply to Jean-Baptiste Mardelle from comment #4)
> Thanks for your report. I started working on this issue and have an almost
> working patch. I hope to push the fix to MLT by the end of the week.

Awesome! Looking forward to testing it. Thank you!

Comment 6 Jean-Baptiste Mardelle 2022-12-09 15:09:48 UTC

Ok, so I just pushed the fix in MLT. It will be included in today's daily development AppImage build. Would be great if you can test - it should be ready in 5 hours from now (ensure you download build number #427 or more) :
https://binary-factory.kde.org/view/AppImage/job/Kdenlive_Nightly_appimage-centos7/

Comment 7 Bernd 2022-12-09 19:29:25 UTC

(In reply to Jean-Baptiste Mardelle from comment #6)
> Ok, so I just pushed the fix in MLT. It will be included in today's daily
> development AppImage build. Would be great if you can test - it should be
> ready in 5 hours from now (ensure you download build number #427 or more) :
> https://binary-factory.kde.org/view/AppImage/job/Kdenlive_Nightly_appimage-
> centos7/

Works perfectly! Thank you very much!