Opened 5 years ago

Closed 3 months ago

Last modified 3 months ago

#4745 closed defect (fixed)

[Patch] KML Superoverlay hybrid jpg and png output

Reported by: ryanl Owned by: Robert Coup
Priority: normal Milestone:
Component: GDAL_Raster Version: svn-trunk
Severity: normal Keywords: KML Superoverlay kmlsuperoverlay
Cc:

Description

The current KML Superoverlay driver cannot output transparent tiles if the output format is jpg.

This patch allows for this hybrid output where a tile is only created in png format if it contains nodata or transparent pixels.

The patch does not include any changes to the docs.

Attachments (9)

auto_format.diff (4.4 KB) - added by ryanl 5 years ago.
auto_format_no_empty_tiles.diff (14.8 KB) - added by cdestigter 5 years ago.
prevents kmlsuperoverlay from generating thousands of empty tiles from sparse raster sources
auto_format_no_empty_tiles.2.patch (14.7 KB) - added by cdestigter 4 years ago.
updated patch to apply on 1.10 branch
auto_format_no_empty_tiles.3.patch (15.2 KB) - added by cdestigter 4 years ago.
fixes build errors
4745.4.diff (14.5 KB) - added by cdestigter 4 years ago.
4745.5.diff (14.5 KB) - added by cdestigter 4 years ago.
fixes bogus dupe NetworkLinks?
4745.6.diff (14.5 KB) - added by cdestigter 4 years ago.
fix bad <north> coordinates due to copy/paste error
4745.7.patch (14.5 KB) - added by cdestigter 2 years ago.
fixes compile errors on trunk (RasterIO signature changed)
4745.8.patch (20.9 KB) - added by cdestigter 3 months ago.
Fixes bugs in sparse raster output (and adds a test) and fixes broken NetworkLinks? to missing KML files.

Download all attachments as: .zip

Change History (13)

Changed 5 years ago by ryanl

Attachment: auto_format.diff added

Changed 5 years ago by cdestigter

prevents kmlsuperoverlay from generating thousands of empty tiles from sparse raster sources

Changed 4 years ago by cdestigter

updated patch to apply on 1.10 branch

Changed 4 years ago by cdestigter

fixes build errors

Changed 4 years ago by cdestigter

Attachment: 4745.4.diff added

comment:1 Changed 4 years ago by cdestigter

4745.4.diff fixes missing .kml files at the most detailed level of the superoverlay.

Changed 4 years ago by cdestigter

Attachment: 4745.5.diff added

fixes bogus dupe NetworkLinks?

Changed 4 years ago by cdestigter

Attachment: 4745.6.diff added

fix bad <north> coordinates due to copy/paste error

Changed 2 years ago by cdestigter

Attachment: 4745.7.patch added

fixes compile errors on trunk (RasterIO signature changed)

Changed 3 months ago by cdestigter

Attachment: 4745.8.patch added

Fixes bugs in sparse raster output (and adds a test) and fixes broken NetworkLinks? to missing KML files.

comment:2 Changed 3 months ago by Even Rouault

Resolution: fixed
Status: newclosed

In 37396:

KMLSuperOverlay: Add FORMAT=AUTO option (patch by Craig de Stigter, fixes #4745)

Uses PNG for semi-transparent areas, else JPG.
Includes a test for sparse raster output (child KML/JPEG don't get created
if tile is entirely transparent)

comment:3 Changed 3 months ago by Even Rouault

I've done a few changes, mostly suppressing a duplicated VSIMkdir() call that caused warnings to be emitted in kmlsuperoverlay_4 test, and fixing some compiler warnings.

comment:4 Changed 3 months ago by Even Rouault

In 37397:

KMLSuperOverlay: fix MSVC warning (refs #4745)

Note: See TracTickets for help on using tickets.