Bug 342141 - Export filters should warn if the image contains unsupported features.
Summary: Export filters should warn if the image contains unsupported features.
Status: RESOLVED FIXED
Alias: None
Product: krita
Classification: Applications
Component: File formats (show other bugs)
Version: unspecified
Platform: Other Linux
: NOR normal
Target Milestone: ---
Assignee: Krita Bugs
URL:
Keywords:
: 344810 (view as bug list)
Depends on:
Blocks:
 
Reported: 2014-12-22 22:29 UTC by Sven Langkamp
Modified: 2016-10-13 08:18 UTC (History)
4 users (show)

See Also:
Latest Commit:
Version Fixed In:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Sven Langkamp 2014-12-22 22:29:44 UTC
Saving 16bit float images in some formats doesn't work. 

Tif and psd files should be able to save it, but they refuse to save.
For png it should automatically convert, but also refuses to save like it does e.g. for jpeg.

Reproducible: Always

Steps to Reproduce:
1. Create 16 bit float image
2. export it

Actual Results:  
Should be exported.

Expected Results:  
Sometimes not exported.
Comment 1 Halla Rempt 2014-12-23 09:08:42 UTC
I'm not sure that automatic conversion is desirable, though. And are you sure tiff supports 16f? As for photoshop, at least the documentation I have only says it's got 16 bit integer: http://www.adobe.com/devnet-apps/photoshop/fileformatashtml/#50577409_19840 "Depth: the number of bits per channel. Supported values are 1, 8, 16 and 32."
Comment 2 Sven Langkamp 2014-12-23 13:30:00 UTC
I think it should convert, but warn. Tiff has 32 bit float, although it's not that important anymore. Photoshop has 32bit float so it should fall back to that.

Btw saving 32bit to psd also doesn't work.
Comment 3 pcchou 2014-12-23 22:57:12 UTC
I'm a google code-in who is trying to do bug triaging task.
Can you provide which version of krita are you using?
Also, can you give more description about how it "refuse to save"?
Comment 4 Dmitry Kazakov 2014-12-29 12:09:48 UTC
Git commit a10298726b78b5e57a4642754ab4b1e66a127ed3 by Dmitry Kazakov.
Committed on 29/12/2014 at 13:04.
Pushed by dkazakov into branch 'calligra/2.9'.

Fix saving of RGBF16/F32 images into TIFF

Also add a unittest for that

M  +1    -1    krita/plugins/formats/tiff/kis_tiff_writer_visitor.cpp
M  +44   -0    krita/plugins/formats/tiff/tests/kis_tiff_test.cpp
M  +1    -0    krita/plugins/formats/tiff/tests/kis_tiff_test.h

http://commits.kde.org/calligra/a10298726b78b5e57a4642754ab4b1e66a127ed3
Comment 5 Dmitry Kazakov 2014-12-29 12:14:35 UTC
A final summary of the bug:

1) [DONE] Tiff now can save F16/F32 images nicely
2) [WONTFIX or LATER] PSD support of F32 is not documented so we cannot implement it
3) [TODO] PNG, JPEG, BMP and other trivial formats should still save F16/F32 images with conversion into U8/U16 on the fly. Of course the user should be notices about the fact of conversion.
Comment 6 Halla Rempt 2016-01-24 15:50:01 UTC
We even has two warnings now! We should fix the warning on file format incompatibilities.
Comment 7 Halla Rempt 2016-08-17 08:28:06 UTC
*** Bug 344810 has been marked as a duplicate of this bug. ***
Comment 8 Halla Rempt 2016-10-01 09:19:38 UTC
This is now implemented in the rempt/impex refactoring branch.
Comment 9 Tyson Tan 2016-10-13 08:18:33 UTC
Thank you, Boud! Hope we can soon see this fix at work.

Additonally, today I just received another question about wrong color output because people are using sRGB-elle-v2-srgbtrc.icc with 16 bit while exporting it as PNG. They do that all the time because we tell them 16 bit is superior. Can we just automatically convert the color space to sRGB 8bit for them while exporting as JPG/PNG?

Or, do we consider collapsing those color space options by default in Document Creation dialogue?