Bug 358583

Summary:	Gt Master 2016-01-26 - cannot make a proxy of a stabilised .mlt file.
Product:	[Applications] kdenlive	Reporter:	Evert Vorster <evorster>
Component:	User Interface & Miscellaneous	Assignee:	Jean-Baptiste Mardelle <jb>
Status:	RESOLVED FIXED
Severity:	normal
Priority:	NOR
Version First Reported In:	unspecified
Target Milestone:	---
Platform:	Arch Linux
OS:	Linux
Latest Commit:		Version Fixed In:
Sentry Crash Report:
Attachments:	mlt file that does not load

Description Evert Vorster 2016-01-26 12:11:09 UTC

I work with proxy clips, as my filter stacks are big, and my footage bigger. 

I have just discovered that one can't make a proxy clip of a stabilised .mlt file, but it might be true of all .mlt files. 

Seeing as how .mlt files already introduce an overhead, having a proxy of such a file would be a good thing, unless it's not supported at all, and then one should not have the option of making a proxy of an .mlt file. 

Here is a short clip, showing the issue:

https://youtu.be/5BN7-Ek5cWk

Reproducible: Always

Steps to Reproduce:
1. Run "vidstab" on a clip
2. Add the output .mlt file to the project. 
3. Notice that even though "Generate proxies by default" is selected, no proxy is generated.
4. Manually generate a proxy

Actual Results:  
Proxy generation fails with "Clip is invalid"

Expected Results:  
Either a proxy clip of the .mlt file, or the proxy clip option greyed out.

Comment 1 Jean-Baptiste Mardelle 2016-01-29 11:03:08 UTC

Git commit 50b186e2268e61577680d40ebad0a1ead6cb8dcb by Jean-Baptiste Mardelle.
Committed on 29/01/2016 at 11:02.
Pushed by mardelle into branch 'master'.

Fix proxy for playlist clips

M  +1    -1    src/mltcontroller/producerqueue.cpp

http://commits.kde.org/kdenlive/50b186e2268e61577680d40ebad0a1ead6cb8dcb

Comment 2 Evert Vorster 2016-02-01 17:59:31 UTC

Created attachment 96959 [details]
mlt file that does not load

I'm sorry but the problem is not fixed, and some more bugs were introduced. 
Here is a short video of me running into this issue again. 

https://youtu.be/wuQqVwwhub0
Also new now is that the clip monitor does not show the currently selected clip in the monitor, as well as acting squirrely, like showing full screen without me having clicked on it

Comment 3 Evert Vorster 2016-02-01 18:01:01 UTC

Issue not fixed as of git master 2016-02-01, and new issues introduced on clip monitor. 
I do like the clip used count being back now, I missed it with the rewrite. 

:)

Comment 4 Evert Vorster 2016-02-01 18:03:30 UTC

Oh, the clip is 4k, and the project is 1080p, which might also be causing the issue.

Comment 5 Jean-Baptiste Mardelle 2016-02-01 22:46:51 UTC

Git commit 71392a70371531cd47e5cddb622eb96df00df08e by Jean-Baptiste Mardelle.
Committed on 01/02/2016 at 22:46.
Pushed by mardelle into branch 'master'.

Fix stabilize on clips with profile != project profile

M  +5    -0    src/definitions.cpp
M  +2    -0    src/definitions.h
M  +1    -1    src/mltcontroller/producerqueue.cpp
M  +6    -0    src/project/jobs/meltjob.cpp

http://commits.kde.org/kdenlive/71392a70371531cd47e5cddb622eb96df00df08e

Comment 6 Evert Vorster 2016-02-02 06:34:49 UTC

Works fine now, thanks