Opened 11 years ago

Closed 11 years ago

Last modified 10 years ago

#226 closed enhancement (fixed)

[wktraster] Motion to remove PT_16BF from specification

Reported by: mloskot Owned by: mloskot
Priority: high Milestone: WKTRaster 0.1.6
Component: raster Version: master
Keywords: wktraster, type, float-point Cc:

Description (last modified by mloskot)

Recently, Frank and Jorge have pointed that there is no known good reason of supporting PT_16BF (16-bit float-point numeric type) defined in the original specification documents (see RFC1-SerializedFormat and RFC2-WellKnownBinaryFormat. Also, it is unclear of what's the purpose of this type.

I have never seen such a thing before and I'm not sure why it is part of the WKTRaster specification. I would suggest removing it. - Frank's post

Martin Daly and I, we agree that there is no point to support 16-bit float-point type.

Also, current version (r4326) of WKT Raster declares PT_16BF type as unsupported.

This ticket is to propose (official) removal of PT_16BF type from the WKT Raster specification as well as from the implementation.

Any objections?

Change History (10)

comment:1 Changed 11 years ago by mloskot

Description: modified (diff)

comment:2 Changed 11 years ago by mloskot

Description: modified (diff)

comment:3 Changed 11 years ago by mloskot

Description: modified (diff)

comment:4 Changed 11 years ago by mloskot

Owner: changed from pracine to mloskot
Status: newassigned

comment:5 Changed 11 years ago by mloskot

Description: modified (diff)

comment:6 Changed 11 years ago by pracine

I have no problem removing support for PT_16BF. I included it in the original specs without really verifying if there were existing images with this pixel type.

comment:7 Changed 11 years ago by mloskot

Resolution: fixed
Status: assignedclosed

Fixed (r4476)

I've rebuilt and run tests, but please reopen the ticket if this changeset causes any problems.

comment:8 Changed 11 years ago by mloskot

Summary: Motion to remove PT_16BF from specification[wktraster] Motion to remove PT_16BF from specification

comment:9 Changed 11 years ago by pracine

Milestone: WKTRaster 0.1.6

comment:10 Changed 10 years ago by mloskot

More update to docs (r5861)

Note: See TracTickets for help on using tickets.