TIP 529: Add metadata dict property to tk photo image

Login
Bounty program for improvements to Tcl and certain Tcl packages.
Tcl 2019 Conference, Houston/TX, US, Nov 4-8
Send your abstracts to tclconference@googlegroups.com
or submit via the online form by Sep 9.
Author:         Harald Oehlmann <oehhar@users.sourceforge.net>
State:          Draft
Type:           Project
Vote:           Pending
Created:        07-Dec-2018
Keywords:       Tk, image
Tcl-Version:    8.7
Tk-Branch:     tip-529-image-metadata

Abstract

An additional property is proposed for photo images to hold image metadata:

myimage cget -metadata
myimage configure -metadata [dict create dpi 300.0]

Rationale

Image files may contain a lot of metadata like resolution, comments, GPS location etc. This metadata should be accessable and setable for the following aims:

This TIP specially targets the resolution (DPI) value of the image.

The image resolution included in an image file is crucial for its usage, as many applications (word & co.) use this field to calculate a default size. One may enjoy, that image files used in pdf4tcl are automatically scaled at the right resolution (e.g. the resolution saved in the image file).

This information is included in png files (supported by tk core) and many other image formats included in the Img patch.

I authored an extension to the Img patch to specify the dpi field of a bmp file on file writing. The syntax was accorded with Jeff Hobbs:

myimage write file.bmp -format [list bmp -resolution 300 i]

This may be expressed (when all packages are adopted) by:

myimage configure -metadata [dict create dpi 300.0]
myimage write file.bmp

Specification

Metadata dict

The propery "-metadata" is added to each image file. It contains a dict, where the keys are specified for each image format.

The following default keys are used, if the corresponding data is present:

key description example image formats
dpi Image resolution png
comment Text comment gif
xmp xmp image data png

Commands

The following commands are directly concerned:

image create photo myimage -metadata $metadict
myimage cget -metadata
myimage configure -metadata $metadict

It is also ok to store application data related to the image within the property dict. A widget may store special properties.

Any load and save command may use the content of the metadata dict. This may be an ongoing process, specially within the Img patch.

The commands are:

image create photo myimage -file myfile.bmp
myimage configure -file myfile.bmp
myimage write file.bmp

The image create and file load functions recreate the metadata dict. Any current contents is lost.

The write file commands uses any dict keys it knows. Any unknown dict keys are ignored.

C interface

The current C interface for drivers and other programs is not sufficient to support the metadata dict.

A new extensible interface is envisaged with:

There are currently functions to access the raster data (TkPhotoGetImage, TkPhotoPutBlock). The other properties (-file, -data, -format, -gamma) are not exposed.

A new interface may also be used for other image formats like svg where the image data consists of the rastered image, the image data and metadata. In this case, an image save operation might even not use the raster data but only metadata and image data.

image property get and set functions

Two new stub table functions are added:

Tcl_Obj * Tk_PhotoGetOption(Tk_PhotoHandle handle, char *OptionName)

void Tk_PhotoSetOption(Tk_PhotoHandle handle, char *OptionName, Tcl_Obj *optionValue)

Revised image driver commands

The new command interface only passes the image handle. Any required data may be accquired by the get/set functions. Also, access to the raster data is done by the raster access functions.

The current registration function is Tk_CreatePhotoImageFormat()

...

Implementation

The current implementation is only a sketch in my brain. I hope, that an image handler can set the options after reading and a binary extension of the image handlers is not necessary. Nevertheless, I don't know jet.

Any help and comments are appreciated. I am realy a novice here.

Implementation may start soon with the tag tip-529-image-metadata.

Rejected alternatives

C interface: only use metadata dict get and set functions

Two new stub table functions are added:

Tcl_Obj * Tk_PhotoGetMetadata(Tk_PhotoHandle handle)

void Tk_PhotoSetMetadata(Tk_PhotoHandle handle, Tcl_Obj *metadata)

This works well for image read. The function "Tk_PhotoSetMetadata" is called within the image read function.

But it is not suitable for image write, as the required photo handle is not passed into the write functions:

static int CommonWriteGIF(Tcl_Interp *interp,
        const char *fileName,
        WriteBytesFunc *writeProc,
        Tcl_Obj *format,
        Tk_PhotoImageBlock *blockPtr);

In consequence, the write function should be extended by the metadata object pointer.

Copyright

This document has been placed in the public domain.