Bug 322880

Summary:	Saved Krita 2.6 color adjustment filter doesn't read properly in 2.8pre-alpha
Product:	[Applications] krita	Reporter:	ALeXeY <alexeyn3d>
Component:	File formats	Assignee:	Sahil <nagpal.sahil01>
Status:	RESOLVED FIXED
Severity:	normal	CC:	halla, nagpal.sahil01
Priority:	NOR
Version:	2.8 Pre-Alpha
Target Milestone:	---
Platform:	Ubuntu
OS:	Linux
Latest Commit:		Version Fixed In:
Sentry Crash Report:
Attachments:	Saved in 2.6 snapshot how it looks in 2.6 snapshot how it looks in 2.8pre-a Proposed solution : Checks the number of curves and then adds alpha if it is not present Krita 2.6 images now work with krita 2.8 pre-alpha and vice versa.

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.