[Robotica]

This Addon really matured, thnx Xycl.
2 feature requests:
1> It would be great if orientation could be saved after rotation.(maybe this is a XBMC feature request)
2> Give rating to pictures (maybe this should be skinned)

And hopefully someone can have a look at those general XBMC Slideshow/pictures bugs for confirmation.

[Xycl]

Not perfectly happy with my translation of Match all tags - so might update it further in the future.
FFY

I don't know if "Match all tags" is a clever description.
My german translation was like "apply each tag to all pictures"

2 feature requests:
1> It would be great if orientation could be saved after rotation.(maybe this is a XBMC feature request)
2> Give rating to pictures (maybe this should be skinned)

1> & 2> The used EXIF parser has no write function.
2> Storing the rating in the database instead of in the pictures is like the rating in the music section. I don't like it to rate my pictures or music and it's not stored into the files.

[Robotica]

Thnx for your clear response; you're right about thd picture rating; it should be stored in the picture tags.

Do you know which EXIF-parser is used? And is this addon-specific or XBMC-specific?

[Xycl]

Exif parser is from http://sourceforge.net/projects/exif-py/
I don't know the version alexsolex used orginally for MyPicsDB because you always find "Version 1.1.0" in the source code of the parser.

[Robotica]

Exif parser is from http://sourceforge.net/projects/exif-py/
I don't know the version alexsolex used orginally for MyPicsDB because you always find "Version 1.1.0" in the source code of the parser.

Thnx . I see the project moved to github and only had 1 change in the last 4 years: https://github.com/ianare/exif-py/blob/m...hanges.txt
So you probably use the version before the last change. (ps. I guess it is save to update to the latest version.)

[Xycl]

Version 0.9.1
Added: Filter wizard XML files for following skins: skin.aeon.nox.svn, skin.aeonmq.2, skin.glass.svn and skin.refocus.release
Update: Performance tuning. Needless DDL commands during scanning removed.
Update: New dialog box in "Scan all Paths". Enables you to rescan all pictures even if they are not modified.
Update: Latest EXIF parser included.

[mikebzh44]

Release 0.9.1 also available on Passion-XBMC repository

[Xycl]

Release 0.9.1 also available on Passion-XBMC repository

Please update dialogaddonscan, too.

[mikebzh44]

Done :

http://passion-xbmc.org/addons/?Page=Vie...gaddonscan

[fiveisalive]

Not sure if this has already been addressed by the new EXIF parser, but I've noticed that the add-on only appears to sort by date, and then by filename, but not by actual time. For example, let's say I have 4 pictures all taken on the same date by 2 different cameras, using different image prefixes, with the following EXIF timestamps. I'd expect them to be sorted like this (i.e. the images all interleaved according to timestamp, not just date):

Code:

DSC_1001    2012-07-10 12:52
IMG_0001    2012-07-10 13:05
DSC_1002    2012-07-10 14:04
IMG_0002    2012-07-10 17:23

But instead they are sorted like this:

Code:

DSC_1001    2012-07-10 12:52
DSC_1002    2012-07-10 14:04
IMG_0001    2012-07-10 13:05
IMG_0002    2012-07-10 17:23

Is this a limit in the core XBMC pictures/image display, or does the timestamp need to be added to the sqlite somehow? This is an important use case, because I have a lot of images taken with different cameras and it's disconcerting when images taken at the same time all appear out of order.

[fiveisalive]

I did some research into the code, and by inserting debugging statements in the show_pics() method, I can see that the database *is* returning the correct order, e.g. if I print out the ordered list of filenames returned in this fragment:

Code:

elif self.args.period in ["month","date"]:
                filelist = MPDB.pics_for_period(self.args.period,self.args.value)
                print "[MyPicsDB]: period: month, date"
                for f in filelist:
                    print f[1]

the filenames are in the correct order, but somehow don't get displayed in the order returned in "filelist".

[Xycl]

The view sorts the thumbnails.
Set "Sort by: Default" in the view settings. This should give you the needed sort order .

[fiveisalive]

The view sorts the thumbnails.
Set "Sort by: Default" in the view settings. This should give you the needed sort order .

Thanks for the suggestion: that's a good workaround. However, it still really should sort by date and time in the "Date" view as well, since using the "Default" order doesn't allow a reverse sort, and it violates the principle of least surprise, since most people will expect date sorting to take into account the time and to "just work" out of the box. In addition, you lose the date in the "Label2" field which is useful to see what date the photo was taken when looking at a huge list of pictures. (The date currently selected is not shown in the default Confluence skin, and you might be looking at entire week or month).

I suspect that this will require a ticket request upstream to allow the SORT_METHOD_DATE to take into account the time and to extend the picture interface to use the time as well as the date as part of the label. How do think we should proceed?

[Xycl]

I think at first someone of the developers must tell us which date is used for sorting.
Perhaps this is the problem.

[Robotica]

XBMC sorts on file date(See code; DateTime class, which uses FILETIME as it's base.) not on EXIF-date, which is much more usefull for pictures (i.e. many file dates are changed moving pictures around, definetly by FTP). So using default XBMC views requires a patch for this (is possible since XBMC also contains an EXIF parser) or plugins should create their own views with sorting based on EXIF data.


See also: http://trac.xbmc.org/ticket/10519