GeoTools

Creating GridCoverage2D from BufferedImage.TYPE_USHORT_555_RGB throws Exception

Details

  • Type: Bug Bug
  • Status: Closed Closed
  • Priority: Minor Minor
  • Resolution: Fixed
  • Affects Version/s: 2.7.0
  • Fix Version/s: 2.7.1, 8.0-M0
  • Component/s: coverage
  • Labels:
    None

Description

From post by Markus on the user list:

BufferedImage image = ImageIO.read(filename);
GridCoverage2D grid = factory.create("results", image, new ReferencedEnvelope(y1, y2, x1, x2, crs));

->

Exception in thread "BackgroundImageRenderer-0" java.lang.IllegalArgumentException: Illegal argument: "type=null".
at org.geotools.coverage.TypeMap.map(TypeMap.java:407)
at org.geotools.coverage.TypeMap.isFloatingPoint(TypeMap.java:362)
at org.geotools.coverage.grid.RenderedSampleDimension.create(RenderedSampleDimension.java:252)
at org.geotools.coverage.grid.RenderedSampleDimension.create(RenderedSampleDimension.java:138)
at org.geotools.coverage.grid.GridCoverage2D.<init>(GridCoverage2D.java:233)
at org.geotools.coverage.grid.GridCoverageFactory.create(GridCoverageFactory.java:590)
at org.geotools.coverage.grid.GridCoverageFactory.create(GridCoverageFactory.java:514)
at org.geotools.coverage.grid.GridCoverageFactory.create(GridCoverageFactory.java:455)

BufferedImage.TYPE_BYTE_INDEXED and BufferedImage.TYPE_INT_RGB work fine, but the native format of the picture is TYPE_USHORT_555. Conversion takes time and BYTE_INDEXED loses quality and TYPE_INT_RGB takes too much memory.

From the code of TypeMap it seems that it should support 16 bit images.

Activity

Ascending order - Click to sort in descending order
Hide
Permalink
Michael Bedward added a comment -
The problem seems to be in org.geotools.coverage.TypeMap where there is no entry for 5 bit values so null is returned. Also this class could be turned into an enum to make the code more readable.
Show
Michael Bedward added a comment - The problem seems to be in org.geotools.coverage.TypeMap where there is no entry for 5 bit values so null is returned. Also this class could be turned into an enum to make the code more readable.
Hide
Permalink
Michael Bedward added a comment -
Yuk - just tried adding a new TypeMap instance for the 5 bit case but of course, being the coverage module, nothing is that simple... everything is coupled to everything else and all tested against standard code lists etc. Really it just needs a thorough clean up.
Show
Michael Bedward added a comment - Yuk - just tried adding a new TypeMap instance for the 5 bit case but of course, being the coverage module, nothing is that simple... everything is coupled to everything else and all tested against standard code lists etc. Really it just needs a thorough clean up.
Hide
Permalink
Michael Bedward added a comment -
Sorry - I can't work this one out so passing it on.
Show
Michael Bedward added a comment - Sorry - I can't work this one out so passing it on.
Hide
Permalink
Markus Sunela added a comment -
Well, thanks for trying.
Show
Markus Sunela added a comment - Well, thanks for trying.
Hide
Permalink
Simone Giannecchini added a comment -
Tentative fix is in. With next build you should be good to go.
Show
Simone Giannecchini added a comment - Tentative fix is in. With next build you should be good to go.
Hide
Permalink
Andrea Aime added a comment -
Mass closing all issues that have been in "resolved" state for more than one month without further comments
Show
Andrea Aime added a comment - Mass closing all issues that have been in "resolved" state for more than one month without further comments

People

  • Assignee:
    Simone Giannecchini
    Reporter:
    Michael Bedward
Vote (0)
Watch (0)

Dates

  • Created:
    Updated:
    Resolved: