Bug 322880 - Saved Krita 2.6 color adjustment filter doesn't read properly in 2.8pre-alpha
Summary: Saved Krita 2.6 color adjustment filter doesn't read properly in 2.8pre-alpha
Status: RESOLVED FIXED
Alias: None
Product: krita
Classification: Applications
Component: File formats (show other bugs)
Version: 2.8 Pre-Alpha
Platform: Ubuntu Linux
: NOR normal
Target Milestone: ---
Assignee: Sahil
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2013-07-27 12:38 UTC by ALeXeY
Modified: 2013-10-02 17:51 UTC (History)
2 users (show)

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


Attachments
Saved in 2.6 (164.47 KB, application/octet-stream)
2013-07-27 12:40 UTC, ALeXeY
Details
snapshot how it looks in 2.6 (104.00 KB, image/png)
2013-07-27 12:42 UTC, ALeXeY
Details
snapshot how it looks in 2.8pre-a (66.72 KB, image/png)
2013-07-27 12:42 UTC, ALeXeY
Details
Proposed solution : Checks the number of curves and then adds alpha if it is not present (1.74 KB, patch)
2013-10-02 15:07 UTC, Sahil
Details
Krita 2.6 images now work with krita 2.8 pre-alpha and vice versa. (4.30 KB, patch)
2013-10-02 17:23 UTC, Sahil
Details

Note You need to log in before you can comment on or make changes to this bug.
Description ALeXeY 2013-07-27 12:38:56 UTC
Saved in Krita 2.6 .kra file filter layer with color adjustment looses it's settings (curves reseted to default) when open in 2.8pre-alpha. 

By the way, extracting a content of *.kra archives, I have found that in *.filterconfig 2.6 format it's only 3 curve param (obviously no alpha), but in 2.8 there are 4.

Reproducible: Always
Comment 1 ALeXeY 2013-07-27 12:40:21 UTC
Created attachment 81378 [details]
Saved in 2.6
Comment 2 ALeXeY 2013-07-27 12:42:27 UTC
Created attachment 81379 [details]
snapshot how it looks in 2.6
Comment 3 ALeXeY 2013-07-27 12:42:58 UTC
Created attachment 81380 [details]
snapshot how it looks in 2.8pre-a
Comment 4 ALeXeY 2013-07-27 17:58:32 UTC
Edit the *.filterconfig can make that *.kra compatible with 2.6 OR 2.8.

For 2.6 should be: 

<!DOCTYPE params>
<params version="1">
 <param name="nTransfers">3</param>
 <param name="curve0">0,0;0.502008,1;1,0;</param>
 <param name="curve1">0,0;0.297189,1;0.714859,0;1,1;</param>
 <param name="curve2">0,1;0.248996,0.493976;1,0;</param>
</params>

And for 2.8:

<!DOCTYPE params>
<params version="1">
 <param name="nTransfers">4</param>
 <param name="curve0">0,0;0.502008,1;1,0;</param>
 <param name="curve1">0,0;0.297189,1;0.714859,0;1,1;</param>
 <param name="curve2">0,1;0.248996,0.493976;1,0;</param>
 <param name="curve3">0,0;1,1;</param> 
</params>
Comment 5 Sahil 2013-07-27 18:15:47 UTC
Thank you for all the information. We are going to look into it and fix it soon.
Comment 6 ALeXeY 2013-07-27 18:24:12 UTC
Also, the 4-th curve (alpha channel) don't work properly for now. 
I entered another bug: https://bugs.kde.org/show_bug.cgi?id=322895
Comment 7 Sahil 2013-10-02 15:07:02 UTC
Created attachment 82609 [details]
Proposed solution : Checks the number of curves and then adds alpha if it is not present
Comment 8 Sahil 2013-10-02 17:23:37 UTC
Created attachment 82625 [details]
Krita 2.6 images now work with krita 2.8 pre-alpha and  vice versa.