[KPhotoAlbum] gps search: a patch

[Reimar Imhof]

Hello Johannes,

after Tobias put his version of the patch to the gps git branch I had a look 
at that one.

To Your question about caching.
I've just recompiled kphotoalbum without caching.
gps searching took me about 15 to 20 sec (no cache).
With caching first search run took about 2 sec, any following searches I just 
couldn't measure - the answer came immediately.
I also go the impression that opening a lot images (about 1500 in my largest 
folder) in the annotation dialog is much faster with caching.

Some numbers: I've got about 16800 pictures, 2500 of them have gps position 
info. So searching for gps info has to look at all the pictures.
Hardware is an about 5 year old i5 with hdd, no ssd, 4gb ram.

Cheers
Reimar

Am Sonntag, 31. Mai 2015, 23:14:11 schrieb Johannes Zarl-Zierl:
> Hello again,
> 
> I've cleaned the patch up a little for fellow reviewers. I did not yet have
> time for any real testing, though.
> 
> You added caching of the coordinates in DB::ImageInfo. I'm curious: did you
> get significant performance differences with/without caching for searching
> or for "regular" MapWidget use?
> I'm asking because we initially did the same caching there, but there was no
> discernible performance difference...
> 
> Cheers,
>   Johannes
> 
> On Sunday 31 May 2015 22:17:21 you wrote:
> > Hello Reimar,
> > 
> > Thanks for the patch! Searching for GPS locations is certainly a nice
> > feature.
> > 
> > I'll look into it as soon as possible (which should be some time this
> > week)
> > and give you some feedback. Maybe Tobias can also give some feedback...
> > 
> > Cheers,
> > 
> >   Johannes
> > 
> > On Sunday 31 May 2015 21:59:55 Reimar Imhof wrote:
> > > Hallo,
> > > 
> > > here you'll find a patch for a simple gps search.
> > > 
> > > It's included in the search dialog. The map comes up in standard pan
> > > mode
> > > -
> > > you can change that to region selection.
> > > 
> > > If you select a region you'll get images with gps coordinates in that
> > > region. Images without gps info will not be found.
> > > 
> > > If you want to select images without gps info or don't care about gps
> > > you
> > > just don't need to select a region. Or you could press the new "Remove
> > > Search Region" button.
> > > 
> > > There is one little problem in
> > > KGeoMap::GeoCoordinates DB::ImageInfo::coordinates() const
> > > This runs an exifdb update if it's not version 2.
> > > This update just takes a little time. That's why the first search
> > > including
> > > gps info might take some time. And there is no search progress bar...
> > > 
> > > 
> > > Enhancements are welcome.
> > > 
> > > Reimar