Hi Michael,<br><br><div class="gmail_quote">On Sun, May 26, 2013 at 5:34 PM, Michael G. Hansen <span dir="ltr"><<a href="mailto:mike@mghansen.de" target="_blank">mike@mghansen.de</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Hi,<br>
<br>
two questions about video metadata handling:<br>
<br>
Current schema of VideoMetadata:<br>
CREATE TABLE VideoMetadata<br>
                            (imageid INTEGER PRIMARY KEY,<br>
                            aspectRatio TEXT,<br>
                            audioBitRate TEXT,<br>
                            audioChannelType TEXT,<br>
                            audioCompressor TEXT,<br>
                            duration TEXT,<br>
                            frameRate TEXT,<br>
                            exposureProgram INTEGER,<br>
                            videoCodec TEXT);<br>
<br>
1. Video metadata is stored as text in the database, even things like audioBitRate, duration and frameRate. Is this intentional? Can we change it to REAL to avoid string-to-real conversions needed for locale-aware display of the data and for sorting?<br>
</blockquote><div><br>When I created the database it was primarily for the task of displaying information.<br>Sure you should change it to REAL or INTEGER, if that is how it would work best.<br> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">

<br>
2. The aspect ratio of videos is also stored in the database, although the width and height are also available, which means that the aspect ratio could simply be calculated. This information is also displayed twice in digikam, once for image properties and once for video properties. Is there a special reason to store the aspect ratio like this or can we remove it?<br>
</blockquote><div><br>I think it should be removed. It can be computed from width and height. I added this as we were picking up the data directly from Exiv2 xmp metadata. Previously it was not being computed within digiKam.<br>
I think it should be removed from Video Properties, but maintained in the Image Properties.<br><br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
Best regards,<br>
<br>
Michael<br>
______________________________<u></u>_________________<br>
Digikam-devel mailing list<br>
<a href="mailto:Digikam-devel@kde.org" target="_blank">Digikam-devel@kde.org</a><br>
<a href="https://mail.kde.org/mailman/listinfo/digikam-devel" target="_blank">https://mail.kde.org/mailman/<u></u>listinfo/digikam-devel</a><br>
</blockquote></div><br><br clear="all"><div><br></div>-- <br>Regards,<br>Abhinav Badola<br>