Summary: | Kdenlive-git - Issue with Wipe transition, not working properly. (video example included) | ||
---|---|---|---|
Product: | [Applications] kdenlive | Reporter: | Unknown <null> |
Component: | Video Effects & Transitions | Assignee: | Vincent PINON <vpinon> |
Status: | RESOLVED FIXED | ||
Severity: | major | CC: | jb, null, wegwerf-1-2-3 |
Priority: | NOR | ||
Version First Reported In: | unspecified | ||
Target Milestone: | --- | ||
Platform: | Arch Linux | ||
OS: | Linux | ||
URL: | https://youtu.be/wL7nQXmN644 | ||
Latest Commit: | Version Fixed In: | ||
Sentry Crash Report: | |||
Attachments: | Example of wipe issue (the black color behind the text in this example should be showing the helicopter, not a black solid. |
Description
Unknown
2016-01-18 18:48:29 UTC
This is probably related to bug #357710. I am going to revert the luma files to pgm now... 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 357710 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 Looks like it's 90% fixed! For video clips, it works like a charm; wipes clean, and does a linear wipe right off the bat. One more issue. When applying to title clips (could be because title clips have an alpha channel/transparent background), but when a title clip is over a video clip, and you put the Wipe transition on the beginning of the title clip, the background during the wipe is black until it reaches the end of the yellow transition block, then the transparency returns to normal (see screenshot I'm about to attach). Created attachment 96758 [details]
Example of wipe issue (the black color behind the text in this example should be showing the helicopter, not a black solid.
Ok, I just checked and MLT's luma transition does not support alpha channel. To fix the proble, we can internally, we use MLT's "composite" transition with a luma, which works with alpha instead of using MLT's "luma" transition. I managed to make it work, however that means that when opening projects with the previous real "luma" transition we have to convert all parameters to make it work with the new "composite" way... will try to finish implementing it this week-end. As a workaround you can manually use the composite transition with a luma. Will do. Thanks for looking into it JB! Git commit 95c68bd99d510cad6dd26d066ecefeda7cf3b4e6 by Jean-Baptiste Mardelle. Committed on 22/01/2016 at 16:32. Pushed by mardelle into branch 'master'. Use "composite" transition for wipe instead of luma, because alpha channel was not supported. We now convert older project files to use the composite transition to have a coherent wipe transition M +4 -0 data/effects/README M +1 -0 data/transitions/CMakeLists.txt M +1 -1 data/transitions/frei0r_cairoaffineblend.xml M +1 -1 data/transitions/frei0r_cairoblend.xml A +21 -0 data/transitions/wipe.xml M +27 -1 src/doc/documentvalidator.cpp M +1 -1 src/doc/kdenlivedoc.cpp M +49 -47 src/effectslist/initeffects.cpp M +2 -4 src/effectslist/initeffects.h M +22 -0 src/effectstack/parametercontainer.cpp M +0 -1 src/project/transitionsettings.cpp M +2 -1 src/timeline/timeline.cpp M +2 -3 src/timeline/transitionhandler.cpp http://commits.kde.org/kdenlive/95c68bd99d510cad6dd26d066ecefeda7cf3b4e6 Looks like the bug is resolved. Works great! Thanks JB and Vincent! Seeing this I now understand why I never got the stand-alone wipe transition working properly with svg titling clips with transparency. Many thanks to all involved in fixing this one! |