Bug 369455 - Version 16.08.1 breaks the Position and Zoom plugin
Summary: Version 16.08.1 breaks the Position and Zoom plugin
Status: RESOLVED FIXED
Alias: None
Product: kdenlive
Classification: Applications
Component: Video Effects & Transitions (show other bugs)
Version: 16.08.0
Platform: Fedora RPMs Linux
: NOR normal
Target Milestone: ---
Assignee: Vincent PINON
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2016-09-28 01:46 UTC by ryeb
Modified: 2016-09-29 02:57 UTC (History)
1 user (show)

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


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description ryeb 2016-09-28 01:46:41 UTC
After updating to Kdenlive version 16.08.1 the Position and Zoom plugin no longer works.

Reproducible: Always

Steps to Reproduce:
1. Click on a clip in the timeline
2. In the video effects tab choose Crop and Transform => Position and Zoom
3. Add a couple of keyframes and then zoom in and out
4. Preview your clip (or render)

Actual Results:  
The red boarder and white dot will move, but the actual video clip is not effected.

Expected Results:  
A zoom effect in the video preview and exported file.

Running up to date Fedora 24. Kdenlive installed via RPM Fusion. If you downgrade Kdenlive to version 16.04.2, the plugin works.
Comment 1 Jean-Baptiste Mardelle 2016-09-28 08:52:07 UTC
I can confirm an issue when using a locale with a comma as decimal separator. Working on it. Can you please paste here the output of the following command in a terminal:

locale
Comment 2 Jean-Baptiste Mardelle 2016-09-28 09:08:14 UTC
Git commit aed3cd0a8a65f3021c71ec92df35ace52953d9d7 by Jean-Baptiste Mardelle.
Committed on 28/09/2016 at 09:07.
Pushed by mardelle into branch 'Applications/16.08'.

Fix several geometry effects broken on locale with comma separator (french, german,...)

M  +1    -1    data/effects/audiowaveform.xml
M  +1    -1    data/effects/dynamictext.xml
M  +1    -1    data/effects/pan_zoom.xml
M  +1    -1    data/effects/region.xml
M  +1    -1    data/effects/rotation.xml

http://commits.kde.org/kdenlive/aed3cd0a8a65f3021c71ec92df35ace52953d9d7
Comment 3 ryeb 2016-09-29 02:57:00 UTC
It looks like you don't need this anymore, but just in case, the output of locale is:

LANG=en_CA.UTF-8
LC_CTYPE="en_CA.UTF-8"
LC_NUMERIC="en_CA.UTF-8"
LC_TIME="en_CA.UTF-8"
LC_COLLATE="en_CA.UTF-8"
LC_MONETARY="en_CA.UTF-8"
LC_MESSAGES="en_CA.UTF-8"
LC_PAPER="en_CA.UTF-8"
LC_NAME="en_CA.UTF-8"
LC_ADDRESS="en_CA.UTF-8"
LC_TELEPHONE="en_CA.UTF-8"
LC_MEASUREMENT="en_CA.UTF-8"
LC_IDENTIFICATION="en_CA.UTF-8"
LC_ALL=