Bug 361773 - Kdenlive freeze - edit external altered project
Summary: Kdenlive freeze - edit external altered project
Status: RESOLVED FIXED
Alias: None
Product: kdenlive
Classification: Applications
Component: User Interface & Miscellaneous (show other bugs)
Version: unspecified
Platform: Other Linux
: NOR normal
Target Milestone: ---
Assignee: Jean-Baptiste Mardelle
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2016-04-14 18:54 UTC by Ovidiu-Florin BOGDAN
Modified: 2016-08-12 15:50 UTC (History)
1 user (show)

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


Attachments
project file with original clip (43.69 KB, application/x-kdenlive)
2016-04-14 18:55 UTC, Ovidiu-Florin BOGDAN
Details
project file with copy clip (40.58 KB, application/x-kdenlive)
2016-04-14 18:55 UTC, Ovidiu-Florin BOGDAN
Details

Note You need to log in before you can comment on or make changes to this bug.
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.