Bug 357710 - wipe transition uses incorrect shape size when using a png lumas file
Summary: wipe transition uses incorrect shape size when using a png lumas file
Status: RESOLVED FIXED
Alias: None
Product: kdenlive
Classification: Applications
Component: Video Effects & Transitions (show other bugs)
Version: unspecified
Platform: Gentoo Packages Linux
: NOR normal
Target Milestone: ---
Assignee: Vincent PINON
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2016-01-08 20:58 UTC by Roberto Castagnola
Modified: 2016-08-04 20:05 UTC (History)
3 users (show)

See Also:
Latest Commit:
Version Fixed In:
Sentry Crash Report:


Attachments
kdenlive project used as test (7.56 KB, application/xml)
2016-01-08 21:00 UTC, Roberto Castagnola
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Roberto Castagnola 2016-01-08 20:58:34 UTC
If I use a png lumas file with a different aspect ratio than the project one, the shape of transition is resized keeping the aspect-ratio of the png file, so I get 2 bar on left and right of the clip where the transition is not applied.
I will attach a simple test project in dv_pal format where I have created a black and a white clip color, starting with the balck clip and ending with the white one. During the transition, using file radial.png found into kdenlive package, the black clip is resized leaving 2 white bars on left and right.
If I convert the file to pgm format:
   convert -strip radial.png radial.pgm
the transition covers all the clip size as expected
(I have to use -strip option to strip comment, or the transition doesn't work: maybe I shall open an other bug for this?)

Furthermore, if I enable 'Use GPU processing' option, kdenlive crashes when using png file, but not when using pgm file


Reproducible: Always




kdenlive version: 15.12.0
mlt version: 0.9.8
Comment 1 Roberto Castagnola 2016-01-08 21:00:53 UTC
Created attachment 96532 [details]
kdenlive project used as test
Comment 2 Jean-Baptiste Mardelle 2016-01-17 20:40:55 UTC
Thanks for your report and feedback.
We are going to switch back to ppm files for the next 15.12.2 release.
Comment 3 Jean-Baptiste Mardelle 2016-01-17 20:42:00 UTC
err.. I mean pgm files.
Comment 4 Jean-Baptiste Mardelle 2016-01-20 08:21:04 UTC
Git commit 331fd851d7a99afe6ef244c49630034f3e1dc541 by Jean-Baptiste Mardelle.
Committed on 20/01/2016 at 08:19.
Pushed by mardelle into branch 'master'.

Revert luma files to pgm, faster and solves scaling issues
Related: bug 358177

M  +18   -18   data/lumas/CMakeLists.txt
A  +-    --    data/lumas/bi-linear_x.pgm
D  +-    --    data/lumas/bi-linear_x.png
A  +1445 -0    data/lumas/bi-linear_y.pgm
D  +-    --    data/lumas/bi-linear_y.png
A  +5    -0    data/lumas/burst.pgm
D  +-    --    data/lumas/burst.png
A  +5005 -0    data/lumas/checkerboard_small.pgm
D  +-    --    data/lumas/checkerboard_small.png
A  +5    -0    data/lumas/clock.pgm
D  +-    --    data/lumas/clock.png
A  +1379 -0    data/lumas/cloud.pgm
D  +-    --    data/lumas/cloud.png
A  +1213 -0    data/lumas/curtain.pgm
D  +-    --    data/lumas/curtain.png
A  +-    --    data/lumas/horizontal_blinds.pgm
D  +-    --    data/lumas/horizontal_blinds.png
A  +-    --    data/lumas/linear_x.pgm
D  +-    --    data/lumas/linear_x.png
A  +-    --    data/lumas/linear_y.pgm
D  +-    --    data/lumas/linear_y.png
A  +-    --    data/lumas/radial-bars.pgm
D  +-    --    data/lumas/radial-bars.png
A  +209  -0    data/lumas/radial.pgm
D  +-    --    data/lumas/radial.png
A  +5    -0    data/lumas/spiral.pgm
D  +-    --    data/lumas/spiral.png
A  +5    -0    data/lumas/spiral2.pgm
D  +-    --    data/lumas/spiral2.png
A  +125  -0    data/lumas/square.pgm
D  +-    --    data/lumas/square.png
A  +-    --    data/lumas/square2-bars.pgm
D  +-    --    data/lumas/square2-bars.png
A  +389  -0    data/lumas/square2.pgm
D  +-    --    data/lumas/square2.png
A  +-    --    data/lumas/symmetric_clock.pgm
D  +-    --    data/lumas/symmetric_clock.png

http://commits.kde.org/kdenlive/331fd851d7a99afe6ef244c49630034f3e1dc541