The icon of the PSD file is not displayed. /* need to confirm in non OSX */ #37023

New Issue

perfection cat · 2013-10-10T07:50:42+02:00

perfection cat commented

2013-10-10 07:50:42 +02:00

%%%--- Operating System, Graphics card ---
Windows XP/SP3 32bits
Nvidia GeForce 9500GT 512MB

- Blender version with error, and version that worked ---

version 2.69 r60652 Daily Build (blender-2.69-r60652-win32.zip)

- Short description of error ---

The icon of the PSD file is not displayed.

- Steps for others to reproduce the error (preferably based on attached .blend file) ---

The icon of the PSD file is not displayed with a file opening screen.
%%%

%%%--- Operating System, Graphics card --- Windows XP/SP3 32bits Nvidia GeForce 9500GT 512MB - Blender version with error, and version that worked --- version 2.69 r60652 Daily Build (blender-2.69-r60652-win32.zip) - Short description of error --- The icon of the PSD file is not displayed. - Steps for others to reproduce the error (preferably based on attached .blend file) --- The icon of the PSD file is not displayed with a file opening screen. %%%

perfection cat commented

2013-10-10 07:50:42 +02:00

Changed status to: 'Open'

Shinsuke Irie commented

2013-10-10 08:10:04 +02:00

%%%Confirmed the PSD support is not working in SCons build on Ubuntu 13.10, while it's working correctly in CMake build. The SCons build seems still incomplete.

Assigning to Dalai.
%%%

%%%Confirmed the PSD support is not working in SCons build on Ubuntu 13.10, while it's working correctly in CMake build. The SCons build seems still incomplete. Assigning to Dalai. %%%

Dalai Felinto commented

2013-10-10 18:15:03 +02:00

%%%scons is not building here (OSX) at the moment, and is not the original reported issue.
As for the report: as you can see from the attached image the icon is shown here. If it doesn't show can you please send a screenshot of the problem?%%%

%%%scons is not building here (OSX) at the moment, and is not the original reported issue. As for the report: as you can see from the attached image the icon is shown here. If it doesn't show can you please send a screenshot of the problem?%%%

Shinsuke Irie commented

2013-10-10 23:29:11 +02:00

%%%Attached a patch (fix_psd_scons.patch). Please test it.
Note that the PSD support defaults to OFF, so you have to enable it explicitly as follows:

Windows:

python scons\scons.py WITH_BF_PSD=True

Mac:

python scons/scons.py WITH_BF_PSD=True

Linux:

python scons/scons.py WITH_BF_OIIO=True WITH_BF_PSD=True
%%%

%%%Attached a patch (fix_psd_scons.patch). Please test it. Note that the PSD support defaults to OFF, so you have to enable it explicitly as follows: Windows: python scons\scons.py WITH_BF_PSD=True Mac: python scons/scons.py WITH_BF_PSD=True Linux: python scons/scons.py WITH_BF_OIIO=True WITH_BF_PSD=True %%%

perfection cat commented

2013-10-11 12:02:25 +02:00

%%%Because I came to work normally, I attach an image and report it.%%%

Dalai Felinto commented

2013-10-11 15:51:34 +02:00

%%%Can you attach a PSD file that doesnt work? Like the fish.psd%%%

Dalai Felinto commented

2013-10-11 18:39:45 +02:00

%%%scons patch adapted and committed.
As for your file I suspect it was not saved with maximum compatibility. I can't tell without looking at them.%%%

%%%scons patch adapted and committed. As for your file I suspect it was not saved with maximum compatibility. I can't tell without looking at them.%%%

Dalai Felinto commented

2013-10-11 23:59:47 +02:00

%%%scons building was fixed on rev. 60690. closing the report now.
If you still have problems make sure the PSD was saved with compatibility option. And if the problem still persist please attach the problematic file and I can reopen the report %%%

%%%scons building was fixed on rev. 60690. closing the report now. If you still have problems make sure the PSD was saved with compatibility option. And if the problem still persist please attach the problematic file and I can reopen the report %%%

perfection cat commented

2013-10-12 01:29:12 +02:00

%%%I seemed to do the report that let you misunderstand it.
I do not use attached Patch.%%%

%%%I seemed to do the report that let you misunderstand it. I do not use attached Patch.%%%

perfection cat commented

2013-10-12 01:32:16 +02:00

%%%I just watched attached Patch, but this is the contents itself which I reported.%%%

Shinsuke Irie commented

2013-10-12 01:38:10 +02:00

%%%Both CMake and Scons builds were still broken, so I fixed them in r60692.

Perfection cat, could you please try the latest svn? (You don't need additional build option like "WITH_BF_PSD=True" anymore.)
%%%

%%%Both CMake and Scons builds were still broken, so I fixed them in r60692. Perfection cat, could you please try the latest svn? (You don't need additional build option like "WITH_BF_PSD=True" anymore.) %%%

perfection cat commented

2013-10-12 03:29:39 +02:00

%%%I tried it in the latest edition that you revised.
It can read the file which I output in Psd form in Blender once.
And the icon is shown a thumbnail.
However, it cannot read original psd file as ever.
And the original thumbnail indication is not possible.
%%%

%%%I tried it in the latest edition that you revised. It can read the file which I output in Psd form in Blender once. And the icon is shown a thumbnail. However, it cannot read original psd file as ever. And the original thumbnail indication is not possible. %%%

perfection cat commented

2013-10-12 04:41:18 +02:00

%%%The image which I attached is an opened image in Gimp2.
I use the image of a sample enclosed by PhotoShop.
I read an original image in Blender and saved it in psd form without processing anything.
There is considerably a difference in luminosity to two images, but what is this cause?%%%

%%%The image which I attached is an opened image in Gimp2. I use the image of a sample enclosed by PhotoShop. I read an original image in Blender and saved it in psd form without processing anything. There is considerably a difference in luminosity to two images, but what is this cause?%%%

Dalai Felinto commented

2013-10-12 22:53:46 +02:00

%%%perfection cat Please Attach a Sample .PSD file where I can see the problem.
The difference you see is most likely due to color management issues%%%

%%%perfection cat *Please Attach a Sample .PSD file* where I can see the problem. The difference you see is most likely due to color management issues%%%

Dalai Felinto commented

2013-10-13 03:56:11 +02:00

%%%from your comment on the mailing list the bug was fixed. I'm closing it now, thanks for the report.
Next time please prioritize using the tracker over the mailing list. The noise ratio in the list is already too high.%%%

%%%from your comment on the mailing list the bug was fixed. I'm closing it now, thanks for the report. Next time please prioritize using the tracker over the mailing list. The noise ratio in the list is already too high.%%%

perfection cat commented

2013-10-16 04:12:56 +02:00

%%%I think this bug report to open again.
Even if the icon of the psd file that the contents are jpeg is displayed, it does not have a meaning.
The icon of the psd file is not displayed definitely now either.

I attach psd file.
%%%

%%%I think this bug report to open again. Even if the icon of the psd file that the contents are jpeg is displayed, it does not have a meaning. The icon of the psd file is not displayed definitely now either. I attach psd file. %%%

Dalai Felinto commented

2013-10-16 04:21:35 +02:00

%%%you need to save the file with max-compatibility
see psd.png attached%%%

%%%you need to save the file with max-compatibility see psd.png attached%%%

perfection cat commented

2013-10-16 04:39:42 +02:00

%%%I do not know whether I may attach a sample file enclosed by Photoshop.
Do you support only max-compatibility?
There is not so a lot a person having Photoshop in individuals.
And it is not announced when it supports only max-compatibility.
You tell me to attach a file, but this problem does not occur with a specific file.
If you test it with the sample file included in it if you have Photoshop, the result appears.%%%

%%%I do not know whether I may attach a sample file enclosed by Photoshop. Do you support only max-compatibility? There is not so a lot a person having Photoshop in individuals. And it is not announced when it supports only max-compatibility. You tell me to attach a file, but this problem does not occur with a specific file. If you test it with the sample file included in it if you have Photoshop, the result appears.%%%

perfection cat commented

2013-10-16 05:25:09 +02:00

%%%I attach blender-2.69-r60739-win32.JPG.
I deleted all the thumbnail that Blender made from a PC and ran it.
I use Blender of daily build distributed in an official site.
Fish.psd, NewFish.psd and PerfectionCat.psd are psd files of max-compatibility.
nPerfectionCat.psd, GimpFish.psd are the psd files which they output in Gimp.%%%

%%%I attach blender-2.69-r60739-win32.JPG. I deleted all the thumbnail that Blender made from a PC and ran it. I use Blender of daily build distributed in an official site. Fish.psd, NewFish.psd and PerfectionCat.psd are psd files of max-compatibility. nPerfectionCat.psd, GimpFish.psd are the psd files which they output in Gimp.%%%

Dalai Felinto commented

2013-10-16 05:34:23 +02:00

%%%attached (fish.png) how I see fish.psd in OSX.
re-opening so someone can report how it looks in linux or again in windows.

does the file open? or not at all. (in other words, is this a thumbnail only problem or blender can't open it either).

The max-compatibility is mentioned here: http://wiki.blender.org/index.php/Dev:Ref/Release_Notes/2.70#PSD_files
There is not much we can do for the thumbnails (overkill to report an error in their case), but I can look at adding a report error when Blender fails to open the PSD file
(right now you get a message on console only - IMB_ibImageFromFile: unknown fileformat )%%%

%%%attached (fish.png) how I see fish.psd in OSX. re-opening so someone can report how it looks in linux or again in windows. does the file open? or not at all. (in other words, is this a thumbnail only problem or blender can't open it either). The max-compatibility is mentioned here: http://wiki.blender.org/index.php/Dev:Ref/Release_Notes/2.70#PSD_files There is not much we can do for the thumbnails (overkill to report an error in their case), but I can look at adding a report error when Blender fails to open the PSD file (right now you get a message on console only - IMB_ibImageFromFile: unknown fileformat )%%%

perfection cat commented

2013-10-16 05:47:07 +02:00

%%%I can never open psd with the file which you revised since I reported the problem of the psd file first.
In addition, I can never display the icon of the psd file.
There is the thing that the icon of the psd file is displayed in the image which I attached.
When one uses Blender which linked a library of OpenImageIO which I revised.
Extension is psd one more, but is the file that the contents were stored in jpeg form.
Because it was not made by a correction of svn of a while ago, the latter does not exist anymore.%%%

%%%I can never open psd with the file which you revised since I reported the problem of the psd file first. In addition, I can never display the icon of the psd file. There is the thing that the icon of the psd file is displayed in the image which I attached. When one uses Blender which linked a library of OpenImageIO which I revised. Extension is psd one more, but is the file that the contents were stored in jpeg form. Because it was not made by a correction of svn of a while ago, the latter does not exist anymore.%%%

perfection cat commented

2013-10-16 06:02:55 +02:00

%%%I'm sorry.
I did not read a message of r60621 till the last.
It was written clearly that it was necessary to be max-compatibility.
I downloaded a sample file of psd distributed in github and tried it, but the result was the same.%%%

%%%I'm sorry. I did not read a message of r60621 till the last. It was written clearly that it was necessary to be max-compatibility. I downloaded a sample file of psd distributed in github and tried it, but the result was the same.%%%

perfection cat commented

2013-10-16 06:26:10 +02:00

%%%Because there was a part expressing giving misunderstanding, I correct it.
I remodel OpenImageIO of version 1.1.11 and use it.
I do not make modifications ordered in r60621.%%%

%%%Because there was a part expressing giving misunderstanding, I correct it. I remodel OpenImageIO of version 1.1.11 and use it. I do not make modifications ordered in r60621.%%%

Campbell Barton commented

2013-10-16 06:34:15 +02:00

%%%fish.psd thumbnail generates OK here, 64bit Linux.%%%

perfection cat commented

2013-10-16 06:50:11 +02:00

%%%Probably I think that it depends on the compiler.
It is a cause that fail by an initialization function called by a constructer of class ImageInput.
It calls the close function of the stream to close a stream by an initialization function.
However, naturally the stream does not open because it is the generation of the constructer.
close is not to handle it by an initialization function%%%

%%%Probably I think that it depends on the compiler. It is a cause that fail by an initialization function called by a constructer of class ImageInput. It calls the close function of the stream to close a stream by an initialization function. However, naturally the stream does not open because it is the generation of the constructer. close is not to handle it by an initialization function%%%

perfection cat commented

2013-10-16 07:02:03 +02:00

%%%I had nothing to do with this problem, but this sometimes sometimes misread the version of the psd file.
The position of the stream seemed to slip off when I checked it.
It came to recognize a version to add the handling of to correct at the time of a start with a position in the lead read_header function definitely.%%%

%%%I had nothing to do with this problem, but this sometimes sometimes misread the version of the psd file. The position of the stream seemed to slip off when I checked it. It came to recognize a version to add the handling of to correct at the time of a start with a position in the lead read_header function definitely.%%%

Shinsuke Irie commented

2013-10-16 07:03:43 +02:00

%%%Fish.psd is thumbnailed here, but nPerfectionCat.psd is not.
Also tested them with the oiio's image vewer (iv). The iv crashes when loading nPerfectionCat.psd.

r60795
Ubuntu 13.10 amd64
OpenImageIO 1.2.2
%%%

%%%Fish.psd is thumbnailed here, but nPerfectionCat.psd is not. Also tested them with the oiio's image vewer (iv). The iv crashes when loading nPerfectionCat.psd. r60795 Ubuntu 13.10 amd64 OpenImageIO 1.2.2 %%%

perfection cat commented

2013-10-16 07:05:45 +02:00

%%%nPerfectionCat.psd is not max-compatibility.
%%%

%%%nPerfectionCat.psd is not max-compatibility. %%%

perfection cat commented

2013-10-16 07:25:16 +02:00

%%%An icon is displayed like the image which I attached when I use Blender which changed the OiiO library.
I attached My OiiO Lib.JPG.

GimpFish.psd and nPerfectionCat.psd are not max-compatibility.%%%

%%%An icon is displayed like the image which I attached when I use Blender which changed the OiiO library. I attached My OiiO Lib.JPG. GimpFish.psd and nPerfectionCat.psd are not max-compatibility.%%%

Dalai Felinto commented

2013-10-16 07:33:07 +02:00

%%%@thomas-40, can you test if Fish.psd produces a correct thumbnail (and open) in Windows?%%%

Dalai Felinto commented

2013-10-16 07:34:01 +02:00

%%%@perfectioncat it may be that windows is using a different version of OIIO than osx and linux%%%

perfection cat commented

2013-10-16 07:46:08 +02:00

%%%They seem to use 1.2.2 so that irie writes it.
We use 1.1.11.
I already checked 1.2.2, but the point that I pointed out thinks that the problem is not settled because it was the same.

I was scolded to you, but I reported many times in mailing list.
An icon is displayed in the library which I remodeled definitely.
And I can open the file.
I have only environment of windows.%%%

%%%They seem to use 1.2.2 so that irie writes it. We use 1.1.11. I already checked 1.2.2, but the point that I pointed out thinks that the problem is not settled because it was the same. I was scolded to you, but I reported many times in mailing list. An icon is displayed in the library which I remodeled definitely. And I can open the file. I have only environment of windows.%%%

perfection cat commented

2013-10-16 08:05:49 +02:00

%%%I thought that you knew it because it was ordered as a construction method of the oiio library of windows.
I downgrade it in 1.1.11 and it is built by it after applying patch.%%%

%%%I thought that you knew it because it was ordered as a construction method of the oiio library of windows. I downgrade it in 1.1.11 and it is built by it after applying patch.%%%

Thomas Dinges commented

2013-10-16 08:21:35 +02:00

%%%I checked with Fish.psd on SVN 60789 (Windows 7 x64):

Image Editor -> Open -> Thumbnail View: No thumbnail is generated for Fish.psd
Also after opening nothing. I get "IMB_ibImageFromFile: unknown fileformat (C:\Users\Thomas\Desktop\Fish.psd)" in console.

I compile a release build config, so OIIO is enabled. Any special flag to set? %%%

%%%I checked with Fish.psd on SVN 60789 (Windows 7 x64): * Image Editor -> Open -> Thumbnail View: No thumbnail is generated for Fish.psd Also after opening nothing. I get "IMB_ibImageFromFile: unknown fileformat (C:\Users\Thomas\Desktop\Fish.psd)" in console. I compile a release build config, so OIIO is enabled. Any special flag to set? %%%

perfection cat commented

2013-10-16 08:41:49 +02:00

%%%@thomas-40 I think that you should run daily build even if you do not build.%%%

Thomas Dinges commented

2013-10-16 08:56:26 +02:00

%%%My own builds and buildbot should be the same, I do Windows release builds. %%%

Thomas Dinges commented

2013-10-16 08:59:28 +02:00

%%%Yes, buildbot Win x64 behaves the same.%%%

Dalai Felinto commented

2013-10-16 20:20:48 +02:00

%%%@thomas-40, are you building with scons or cmake? I tested only with CMake%%%

Dalai Felinto commented

2013-10-16 20:25:39 +02:00

%%%@PerfectCat I remember similar issues with OpenEXR on Windows. We will eventually use OIIO 1.3 with Blender, so we are not updating the libraries in the meantime.
It's important to know beforehand if the libraries will work.

Now, I didn't mean to 'scold' you in the list, but you need to understand that the mailing list is not the place to discuss bugs, as I repeatedly mentioned. Apart from that it's been really difficult to understand what you are trying to say. I understand that English is not your first English (nor is fine), so there is an extra effort required by all parts.

In general if you are proposing a code change it's easier to attach (or point to a link on pasteall.org) with a patch.
If you are referring to unexpected behaviour, an image can speak by itself. And a sample file is a most anytime.%%%

%%%@PerfectCat I remember similar issues with OpenEXR on Windows. We will eventually use OIIO 1.3 with Blender, so we are not updating the libraries in the meantime. It's important to know beforehand if the libraries will work. Now, I didn't mean to 'scold' you in the list, but you need to understand that the mailing list is not the place to discuss bugs, as I repeatedly mentioned. Apart from that it's been really difficult to understand what you are trying to say. I understand that English is not your first English (nor is fine), so there is an extra effort required by all parts. In general if you are proposing a code change it's easier to attach (or point to a link on pasteall.org) with a patch. If you are referring to unexpected behaviour, an image can speak by itself. And a sample file is a most anytime.%%%

Dalai Felinto commented

2013-10-16 20:30:21 +02:00

%%%edit: "I understand that English is not your first language (nor is mine), so there is an extra effort required by all parts."%%%

Dalai Felinto commented

2013-10-16 23:47:37 +02:00

%%%So, it works in OSX with Scons and CMake, and on Windows it works with CMake only...%%%

perfection cat commented

2013-10-17 01:35:28 +02:00

%%%It was understood that you determined not to support it when it was built in SCONS.
Then I want you to consider it to exclude the function in SCONS.
It becomes with a build switch of OiiO and cannot remove it individually now.
I can do it personally, but compatibility with svn lowers.
If you do not exclude it, the failed thumbnail is recorded.
There are not many users understanding a method to update failed thumbnail.
Because I establish the build switch that I control the generation of the thumbnail personally, I do not come under influence.%%%

%%%It was understood that you determined not to support it when it was built in SCONS. Then I want you to consider it to exclude the function in SCONS. It becomes with a build switch of OiiO and cannot remove it individually now. I can do it personally, but compatibility with svn lowers. If you do not exclude it, the failed thumbnail is recorded. There are not many users understanding a method to update failed thumbnail. Because I establish the build switch that I control the generation of the thumbnail personally, I do not come under influence.%%%

Shinsuke Irie commented

2013-10-17 08:50:37 +02:00

%%%I tried VS2012 + SCons on Win64, and confirmed the PSD support works properly.
However I couldn't try VC++ Express 2008 because Microsoft seems not to distribute it anymore.
%%%

%%%I tried VS2012 + SCons on Win64, and confirmed the PSD support works properly. However I couldn't try VC++ Express 2008 because Microsoft seems not to distribute it anymore. %%%

Dalai Felinto commented

2013-10-17 08:54:56 +02:00

%%%@Shinsuke Irie: Can you try the builds from the build-bot? (builder.blender.org). I believe they are also using scons.%%%

Dalai Felinto commented

2013-10-17 09:04:58 +02:00

%%%Actually did you build a 64 build or a 32? (I know your windows is 64, but the build doesn't need to be necessarily)
Same for @thomas-40, the one that failed was 32 or 64?

My CMake build in Windows was VS2012, Debug I don't remember which bit. (my windows machine runs under virtual machine, so it's a clumsy test box)%%%

%%%Actually did you build a 64 build or a 32? (I know your windows is 64, but the build doesn't need to be necessarily) Same for @thomas-40, the one that failed was 32 or 64? My CMake build in Windows was VS2012, Debug I don't remember which bit. (my windows machine runs under virtual machine, so it's a clumsy test box)%%%

Shinsuke Irie commented

2013-10-17 09:31:50 +02:00

%%%Tested the buildbot r60807 64bit.
Icon preview is OK, but loading PSD file fails. I got an error "IMB_ibImageFromFile: unknown fileformat".

The own build is actually 64bit, using vcvarsall.bat with an argument "x86_amd64".
%%%

%%%Tested the buildbot r60807 64bit. Icon preview is OK, but loading PSD file fails. I got an error "IMB_ibImageFromFile: unknown fileformat". The own build is actually 64bit, using vcvarsall.bat with an argument "x86_amd64". %%%

Dalai Felinto commented

2013-10-17 09:46:02 +02:00

%%%The thumbnails are cached (~/.thumbnails/fail/blender and ~/.thumbnail/normal). If you remove the files from those folders they will probably fail as well in the buildbot you tested.
Thomas just reported that he built with msvc9, scons 64 bit

We can wait for a few days until the buildbot experimental vs2012 machine builds an updated blender. That should help to decide if the error is indeed msvc2008 vs 2012.
(we are looking for a build after your scons fixes on 60698)%%%

%%%The thumbnails are cached (~/.thumbnails/fail/blender and ~/.thumbnail/normal). If you remove the files from those folders they will probably fail as well in the buildbot you tested. Thomas just reported that he built with msvc9, scons 64 bit We can wait for a few days until the buildbot experimental vs2012 machine builds an updated blender. That should help to decide if the error is indeed msvc2008 vs 2012. (we are looking for a build after your scons fixes on 60698)%%%

Shinsuke Irie commented

2013-10-17 09:57:28 +02:00

%%%Ah sorry, the icon preview is indeed not working with the buildbot build...%%%

Dalai Felinto commented

2013-10-17 10:09:51 +02:00

%%%There is a version mismatch going on.
http://lists.blender.org/pipermail/bf-committers/2013-October/041956.html%%%

%%%There is a version mismatch going on. http://lists.blender.org/pipermail/bf-committers/2013-October/041956.html%%%

Shinsuke Irie commented

2013-10-18 12:56:50 +02:00

%%%I committed debug prints for error in the PSD loader. r60841.
We can now obtain more verbose error message in console.
%%%

%%%I committed debug prints for error in the PSD loader. r60841. We can now obtain more verbose error message in console. %%%

Shinsuke Irie commented

2013-10-19 06:17:25 +02:00

%%%Tested the latest buildbot builds r60854. VS2012 build also is available.

2012 (vc11):

Works as we expect. Everything is OK.

2008 (vc9):

Still doesn't work. I got the following error messages in console:

imb_load_photoshop: ImageInput::create() failed:
OpenImageIO could not open "C:\Users\irie\Downloads\Fish.psd" as psd: [Header] invalid signature
[Header] invalid signature
imb_load_photoshop: ImageInput::create() failed:
OpenImageIO could not open "C:\Users\irie\Downloads\Fish.psd" as psd: [Header] invalid signature
[Header] invalid signature
IMB_ibImageFromFile: unknown fileformat (C:\Users\irie\Downloads\Fish.psd)

File signature recognition in the OIIO library seems to go wrong. I think there is nothing to do further in Blender side.
%%%

%%%Tested the latest buildbot builds r60854. VS2012 build also is available. 2012 (vc11): Works as we expect. Everything is OK. 2008 (vc9): Still doesn't work. I got the following error messages in console: imb_load_photoshop: ImageInput::create() failed: OpenImageIO could not open "C:\Users\irie\Downloads\Fish.psd" as psd: [Header] invalid signature [Header] invalid signature imb_load_photoshop: ImageInput::create() failed: OpenImageIO could not open "C:\Users\irie\Downloads\Fish.psd" as psd: [Header] invalid signature [Header] invalid signature IMB_ibImageFromFile: unknown fileformat (C:\Users\irie\Downloads\Fish.psd) File signature recognition in the OIIO library seems to go wrong. I think there is nothing to do further in Blender side. %%%

Dalai Felinto commented

2013-10-19 19:58:37 +02:00

%%%> I think there is nothing to do further in Blender side.
It would help to know if the version we are using with vc11 (1.2.2) would work for the vc9.
Or even the other way around, if the version we are using with vc9 (1.1.11) works with the vc11.

This would reveal if the problem is the compiler or the library version.
It's a lot of work to test that though, so I wouldn't mind if we get back to that only after the new library is actually required.%%%

%%%> I think there is nothing to do further in Blender side. It would help to know if the version we are using with vc11 (1.2.2) would work for the vc9. Or even the other way around, if the version we are using with vc9 (1.1.11) works with the vc11. This would reveal if the problem is the compiler or the library version. It's a lot of work to test that though, so I wouldn't mind if we get back to that only after the new library is actually required.%%%

perfection cat commented

2013-10-30 02:46:45 +01:00

%%%I show the point that I changed if you intend to change a library.

void
PSDInput::init ()
{

  m_filename.clear ();

// m_file.close ();

  m_subimage = -1;
  m_subimage_count = 0;
  m_specs.clear ();
  m_WantRaw = false;
  m_layers.clear ();
  m_image_data.channel_info.clear ();
  m_image_data.transparency = false;
  m_channels.clear ();
  m_alpha_names.clear ();
  m_channel_buffers.clear ();
  m_rle_buffer.clear ();
  m_transparency_index = -1;
  m_keep_unassociated_alpha = false;
  m_background_color[0] = 1.0;
  m_background_color[1] = 1.0;
  m_background_color[2] = 1.0;
  m_background_color[3] = 1.0;

}

bool
PSDInput::close ()
{

  m_file.close ();
  init();
  return true;

}

%%%

%%%I show the point that I changed if you intend to change a library. void PSDInput::init () { ``` m_filename.clear (); ``` // m_file.close (); ``` m_subimage = -1; m_subimage_count = 0; m_specs.clear (); m_WantRaw = false; m_layers.clear (); m_image_data.channel_info.clear (); m_image_data.transparency = false; m_channels.clear (); m_alpha_names.clear (); m_channel_buffers.clear (); m_rle_buffer.clear (); m_transparency_index = -1; m_keep_unassociated_alpha = false; m_background_color[0] = 1.0; m_background_color[1] = 1.0; m_background_color[2] = 1.0; m_background_color[3] = 1.0; ``` } bool PSDInput::close () { ``` m_file.close (); init(); return true; ``` } %%%

perfection cat commented

2013-10-30 02:54:10 +01:00

%%%bool
PSDInput::read_header ()
{
m_file.seekg( 0, std::ios::beg );

  m_file.read (m_header.signature, 4);
  read_bige<uint16_t> (m_header.version);
  m_file.seekg(6, std::ios::cur);
  read_bige<uint16_t> (m_header.channel_count);
  read_bige<uint32_t> (m_header.height);
  read_bige<uint32_t> (m_header.width);
  read_bige<uint16_t> (m_header.depth);
  read_bige<uint16_t> (m_header.color_mode);
  return check_io ();

}
%%%

%%%bool PSDInput::read_header () { m_file.seekg( 0, std::ios::beg ); ``` m_file.read (m_header.signature, 4); read_bige<uint16_t> (m_header.version); m_file.seekg(6, std::ios::cur); read_bige<uint16_t> (m_header.channel_count); read_bige<uint32_t> (m_header.height); read_bige<uint32_t> (m_header.width); read_bige<uint16_t> (m_header.depth); read_bige<uint16_t> (m_header.color_mode); return check_io (); ``` } %%%

perfection cat commented

2013-10-30 03:00:23 +01:00

%%%I changed read_header for the following reasons.

It cannot process it by the reading of the header after the case that the psd file which does not have Maximum-compatible was included in definitely.%%%

%%%I changed read_header for the following reasons. It cannot process it by the reading of the header after the case that the psd file which does not have Maximum-compatible was included in definitely.%%%

perfection cat commented

2013-10-31 23:36:16 +01:00

%%%I tried it in CMake+vs2008 of Windows, but the thumbnail is not displayed.%%%

Dalai Felinto commented

2013-10-31 23:53:02 +01:00

%%%@perfection_cat:

Forget about thumbnail not displaying. The real issue is whether or not the image loads fine (once that's working, then we all clean our thumbnail caches and see if it's working, this way the tests are more reliable).
As suggested some time ago, please share a patch of the changes you did to OIIO. We are getting lost in translation here, but code is 'universal language' ;)
Does it work with your patch?
Does it work with 'master' of openimageio?
Thanks%%%

%%%@perfection_cat: 1) Forget about thumbnail not displaying. The real issue is whether or not the image loads fine (once that's working, then we all clean our thumbnail caches and see if it's working, this way the tests are more reliable). 2) As suggested some time ago, please share a patch of the changes you did to OIIO. We are getting lost in translation here, but code is 'universal language' ;) 3) Does it work with your patch? 4) Does it work with 'master' of openimageio? Thanks%%%

perfection cat commented

2013-11-01 00:52:54 +01:00

%%%I thought what said to work in exe which you built in CMake in a question.
Therefore I built in CMake+vs2008.
And it was proved not to work when I tried it with the library of the master.
I confirmed that error message was output by a console.
I have already known that a thumbnail is performed cash of.%%%

%%%I thought what said to work in exe which you built in CMake in a question. Therefore I built in CMake+vs2008. And it was proved not to work when I tried it with the library of the master. I confirmed that error message was output by a console. I have already known that a thumbnail is performed cash of.%%%

perfection cat commented

2013-11-01 01:54:27 +01:00

%%%I attach the file which applied a change point to an original file of 1.1.11.%%%

perfection cat commented

2013-11-01 02:32:58 +01:00

%%%Because it is not translated as I intended by machine translation, I rewrite it.

I show the reason that changed the read_header function.
When the psd file which had low compatibility was mixed, it might not acquire signature of the file with maximum compatibility definitely.
I changed read_header function to improve the phenomenon.%%%

%%%Because it is not translated as I intended by machine translation, I rewrite it. I show the reason that changed the read_header function. When the psd file which had low compatibility was mixed, it might not acquire signature of the file with maximum compatibility definitely. I changed read_header function to improve the phenomenon.%%%

perfection cat commented

2013-12-04 03:43:07 +01:00

Because I revised a file collected in master of OpenimageIO, I attach it.
It functions in my environment.

https://github.com/OpenImageIO/oiio

Because I revised a file collected in master of OpenimageIO, I attach it. It functions in my environment. https://github.com/OpenImageIO/oiio

Thomas Dinges commented

2013-12-06 17:37:12 +01:00

Changed status from 'Open' to: 'Resolved'

Thomas Dinges closed this issue

2013-12-06 17:37:12 +01:00

Thomas Dinges commented

2013-12-06 17:37:12 +01:00

The issue has been fixed on Windows, please update the libraries from SVN or use a new build.

(x32 libs are being commited atm, should be up in a few minutes).

The issue has been fixed on Windows, please update the libraries from SVN or use a new build. (x32 libs are being commited atm, should be up in a few minutes).

Sign in to join this conversation.

No Label

Download

What's New

Blender Studio

Manual

Developers Blog

Documentation

Benchmark

Blender Conference

Development Fund

One-time Donations

The icon of the PSD file is not displayed. /* need to confirm in non OSX */ #37023