[Insight-developers] JPEG2000ImageIO moved to Review : Input images too large...

[Bradley Lowekamp]

Luis,

This patch is for the jpeg2000 library. So it may need to work its way into that library' svn too.  But getting this intrinsic stuff portable is challenging!

Here is the failure I am getting:
http://www.cdash.org/CDash/viewBuildError.php?buildid=702338

On a related issue the sun system is getting the following errors about not having the xmmintrin.h file:
http://www.cdash.org/CDash/viewBuildError.php?buildid=701665

I would suspect that this is a case for try compile, check for header, make a config file, but openjpeg does not have the generated configuration file yet.
I have gone a head and staged and merged these changes.

Brad

On Aug 23, 2010, at 5:17 PM, Luis Ibanez wrote:

> 
> Hi Brad,
> 
> Thanks for giving it a try.
> 
> I haven't seen the Experimental build from "victoria"
> in the Dashboard yet.
> 
> Should I apply the patch anyways  ?
> 
> 
>    Please advise,
> 
> 
>         Thanks
> 
>  
>               Luis
>  
> 
> 
> --------------------------------
> On Mon, Aug 23, 2010 at 9:52 AM, Bradley Lowekamp <blowekamp at mail.nih.gov> wrote:
> Hello,
> 
> I am experiencing compilation in the openjpeg2000 library with regards to sse intrinsics:
> (my workstation victoria should be finishing the erroneous experimental build soon, so just check the cdash )
> 
> I have published changes onto my github account in the "openjpeg2000_sse" branch. I have also attached a patch which can be applied with the "git am" command.
> 
> Using SSE intrinsic portably is tricky. And it's tricky to test because it's not always clear which compilers have them enabled on the dashboard. But briefly reading the Intel, GCC, and MSDN documentation I believe the intrinsic functions are more portable then the overloaded operators.
> 
> 
> 
> On Aug 21, 2010, at 8:48 PM, Luis Ibanez wrote:
> 
>> 
>> Hi Kishore,
>> 
>> Now that we have updated openjpeg  to Monday's 
>> version of openjpeg-v2, we were also able to move 
>> the JPEG2000ImageIO classes from the 
>> Insight Journal paper:
>> 
>> http://www.insight-journal.org/browse/publication/741
>> http://hdl.handle.net/10380/3187
>> 
>> 
>> BTW: Thanks for submitting the paper     :-)
>> 
>> 
>> We are however, struggling with the input images,
>> since they are rather large.
>> 
>> So far we have committed only the images 
>> 
>> Bretagne1.j2k ---- 90Kb
>> Cevennes2.jp2 ---- 18Kb
>> 
>> The others are larger than 100Kb.... 
>> 
>> 
>> The conundrum is that, we actually should test 
>> with very large images, given that this is one 
>> of the purposes of introducing JPEG2000.
>> 
>> 
>> Any ideas are welcome....
>> 
>> In the meantime, if you have a chance, please
>> give it a try at the new code in ITK/Code/Review.
>> 
>> 
>>    Thanks
>> 
>> 
>>         Luis
>> 
>> 
>> <ATT00001..txt>
> 
> 
> 

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://www.itk.org/mailman/private/insight-developers/attachments/20100824/b6972916/attachment.htm>