Re: [JPP-Devel] Important bug on class RasterImageLayer on OpenJUMP 6669

Mon, 08 Feb 2021 14:54:36 -0800

Oh yes, this an flt file ! Did not think about it, sollry.

I will set bpp to 16b for flt, asc and txt, as this seems to be how it is read.

Michaël

envoyé : 8 février 2021 à 15:07
de : Giuseppe Aruta <[email protected]>
à : Michaud Michael <[email protected]>, OpenJump develop and use <[email protected]>
objet : Re: [JPP-Devel] Important bug on class RasterImageLayer on OpenJUMP 6669


Hi Michael
here you can find the test image flt that I used (both .flt and hdr files)
Best regards
Peppe

Il giorno dom 7 feb 2021 alle ore 11:55 Michaud Michael <[email protected]> ha scritto:

Hi,

Peppe, could you provide me an image throwing an exception. I tried with a floating point image from R. Rossi but it worked well. I'd like to be able to reproduce the error before fixing. Proposition from Ede makes sense but I'll first check if image/io offers a more robust method to evaluate the number of bpp.

Michaël

 

envoyé : 4 février 2021 à 12:22
de : [email protected]
à : [email protected]
objet : Re: [JPP-Devel] Important bug on class RasterImageLayer on OpenJUMP 6669


looks like a test like
http://commons.apache.org/proper/commons-imaging/apidocs/org/apache/commons/imaging/Imaging.html#getFormatCompliance-byte:A-

before running Imaging.getImageInfo() might make sense. but it'll likely just throw an Exception as well. didn't test it.

..ede

On 2/4/2021 11:57, [email protected] wrote:

the commit by Mike seems to have simply added the call Imaging.getImageInfo()
https://sourceforge.net/p/jump-pilot/code/6643/

try to wrap it in try/catch and it should work again. or leave it up to Mike who fixed the memory routine ;).. ede

ps. All - are we set on the github branch now? if yes i suggest to name it "devel" or "trunk" as "master" seems to be out of fashion and "main" is to generic for my taste.

On 2/4/2021 6:53, Giuseppe Aruta wrote:

>> Note that GridFloat (FLT) and GridAscii (ASC) classes have their own methods to read cell size, GridFloat(string filename).readHdr and than getCellSize().>>
>> Il giorno gio 4 feb 2021 alle ore 06:48 Giuseppe Aruta <[email protected] <mailto:[email protected]>> ha scritto:
>>
>> Hi all,
>> it seems that we lost the ability to read and display FLT and ASC raster files using (Sextante) raster image layer framework. The bug is located into class RasterImageLayer,
>> line 352:
>>  bitsPerPixel = Imaging.getImageInfo(new File(imageFileName)).getBitsPerPixel();
>>
>> The reason is that classorg.apache.commons.imaging.Imaging
>> cannot read ASC and FLT files
>>
>> Best regards
>> Peppe
>>
>>
>>
>> _______________________________________________
>> Jump-pilot-devel mailing list
>> [email protected]
>> https://lists.sourceforge.net/lists/listinfo/jump-pilot-devel
>>

_______________________________________________
Jump-pilot-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/jump-pilot-devel



_______________________________________________
Jump-pilot-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/jump-pilot-devel

_______________________________________________
Jump-pilot-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/jump-pilot-devel
_______________________________________________
Jump-pilot-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/jump-pilot-devel






















					Reply via email to