Bug 358672 - 15.13 git master: White Balance LMS Space looses its parameters on reload
Summary: 15.13 git master: White Balance LMS Space looses its parameters on reload
Status: RESOLVED FIXED
Alias: None
Product: kdenlive
Classification: Applications
Component: Video Effects & Transitions (show other bugs)
Version: unspecified
Platform: Compiled Sources Linux
: NOR normal
Target Milestone: ---
Assignee: Vincent PINON
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2016-01-28 09:46 UTC by Wegwerf
Modified: 2016-02-05 15:02 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 Wegwerf 2016-01-28 09:46:51 UTC
The settings of the White Balance LMS Space looses its settings for the color temperature and falls back to 1000. This happens when reloading the project.

Reproducible: Always
Comment 1 Wegwerf 2016-01-28 17:10:39 UTC
Same happens also for color.
Comment 2 Wegwerf 2016-01-28 17:13:44 UTC
To reproduce:
1. Create new Kdenlive project and add a video clip to it.
2. Place clip on timeline.
3. Add effect "White Balance (LMS color space)"
4. Pick a different color, and change the color temperature to 6000 (K)
5. Save project.
6. Exit Kdenlive, and restart, load project.
7. Select clip in timeline, check the effect's parameters.

Actual:
Color as well as color temperature appear to be reset to defaults. MLT however seems to still use the correct values. "The return of the Komma"? ;)

Expected:
Color and color temperature parameters retain their values set before saving and reloading.
Comment 3 Jean-Baptiste Mardelle 2016-02-01 22:06:35 UTC
Git commit f67d7486d7ab6d9626eee6397571b9e823fbe611 by Jean-Baptiste Mardelle.
Committed on 01/02/2016 at 22:06.
Pushed by mardelle into branch 'master'.

Fix several issues with simplekeyframe (locale, precision)

M  +6    -1    src/effectstack/keyframeedit.cpp
M  +5    -1    src/timeline/timeline.cpp

http://commits.kde.org/kdenlive/f67d7486d7ab6d9626eee6397571b9e823fbe611
Comment 4 Wegwerf 2016-02-05 15:02:14 UTC
Jean-Baptiste, thank you very much for fixing this! I've checked on an existing project and the loaded parameters are now shown correctly. So I set this to resolved fixed.