Bug 361773

Summary:	Kdenlive freeze - edit external altered project
Product:	[Applications] kdenlive	Reporter:	Ovidiu-Florin BOGDAN <kde.kfoar>
Component:	User Interface & Miscellaneous	Assignee:	Jean-Baptiste Mardelle <jb>
Status:	RESOLVED FIXED
Severity:	normal	CC:	wegwerf-1-2-3
Priority:	NOR
Version First Reported In:	unspecified
Target Milestone:	---
Platform:	Other
OS:	Linux
Latest Commit:		Version Fixed/Implemented In:
Sentry Crash Report:
Attachments:	project file with original clip project file with copy clip

Description Ovidiu-Florin BOGDAN 2016-04-14 18:54:26 UTC

I edited a project file (in Kate) to replace one video clip with a copy (see why at the bottom).
After opening the project in Kdenlive, I played through the timeline and it was ok. I tried to increse the speed effect of one clip section and then Kdenlive froze.

I replaced in the project file (via Kate) all occurrences of "VID_20160402_114623.mp4" with "output.mp4". I'm aware that this is not normal and you shouldn't do this, but since the file is a copy, I expected this to work. Anyway, Kdenlive should not freeze and it should complain that the project is invalid, if it actually is.

Why I did this:
I used OpenCamera to make the recoding in landscape mode. The exif of one of the videos showed the resolution as landscape (1280x720) but the Rotation got set to 90 for some reason.

This wasn't a problem until a recent update when video players (including Kdenlive) started reading this Rotation flag, and rotating the clip everywhere.

Research on the problem eventually got me here: http://u88.n24.queensu.ca/exiftool/forum/index.php?topic=6670.0 and then here: http://u88.n24.queensu.ca/exiftool/forum/index.php?topic=6010.0
So I copied the clip with ffmpeg `ffmpeg -i VID_20160402_114623.mp4 -c copy -metadata:s:v:0 rotate=0 output.mp4` in order to rewrite the rotation flag.

I had already don lot's of splitting and effect adding to the project on the original clip (which wasn't rotated until some time ago). It was too much work to be repeated. Since it's basically a copy, renaming the file in the project file should have worked fine.

Reproducible: Always

Comment 1 Ovidiu-Florin BOGDAN 2016-04-14 18:55:01 UTC

Created attachment 98398 [details]
project file with original clip

Comment 2 Ovidiu-Florin BOGDAN 2016-04-14 18:55:24 UTC

Created attachment 98399 [details]
project file with copy clip

Comment 3 Ovidiu-Florin BOGDAN 2016-04-14 18:56:37 UTC

When I opened the project after editing (with Kate) Kdenlive complained that the video clips were missing and I clicked a button that said something like "Use replacements", and then it all worked fine.

Comment 4 Jean-Baptiste Mardelle 2016-04-14 20:54:15 UTC

Git commit a79e692236d2718abeb3e74d3cac4a177d3e1eec by Jean-Baptiste Mardelle.
Committed on 14/04/2016 at 20:53.
Pushed by mardelle into branch 'master'.

Add clip property to disable autorotate

M  +1    -1    src/bin/projectclip.cpp
M  +2    -2    src/mltcontroller/clipcontroller.cpp
M  +19   -2    src/mltcontroller/clippropertiescontroller.cpp

http://commits.kde.org/kdenlive/a79e692236d2718abeb3e74d3cac4a177d3e1eec

Comment 5 Jean-Baptiste Mardelle 2016-04-14 20:57:44 UTC

I just added a new option in Clip Properties to disable autorotate (in git master).
Without source clips it's hard to know why your edits caused a freeze. However, the correct and simpler way to disable autorotate from an existing .kdenlive project file is to edit original project, and in every <producer> tags that use the clip, add this:

<property name="autorotate">0</property>

For example:
<property name="resource">path/to/your/file/VID_20160402_114623.mp4</property>
<property name="autorotate">0</property>

This should disable autorotate without messing with the project

Comment 6 Ovidiu-Florin BOGDAN 2016-04-19 05:41:09 UTC

The source clip in question is 3,8GB. If you want I can send it to you somehow, in private.

Comment 7 Ovidiu-Florin BOGDAN 2016-04-19 15:04:52 UTC

I found no GUI option for this, so I edited the project file in Kate. I guess that's what I was supposed to do. Is there a plan to add a GUI option for this, before the release?

Also, the clip properties still shows the resolution of the clip rotated: 720x1280, when the actual resolution is 1280x720. I see this resolution even on the clips that don't have the rotation problem. Is this intentional?

Comment 8 Wegwerf 2016-08-12 15:50:12 UTC

Ovidiu-Florin, recent stable Kdenlive 16.04.2 has a GUI option "disable autorotate". This should solve your original problem.