[Tickets #6418] Re: Face recognition

[bugs at horde.org]

DO NOT REPLY TO THIS MESSAGE. THIS EMAIL ADDRESS IS NOT MONITORED.

Ticket URL: http://bugs.horde.org/ticket/6418
-----------------------------------------------------------------------
 Ticket             | 6418
 Updated By         | Duck <duck at obala.net>
 Summary            | Face recognition
 Queue              | Ansel
 Type               | Enhancement
 State              | Assigned
 Priority           | 1. Low
 Milestone          |
 Patch              |
 Owners             | Michael Rubinsky
+New Attachment     | getRecentImages.diff
-----------------------------------------------------------------------


Duck <duck at obala.net> (2008-06-05 03:36) wrote:

>  Just a note: if I import my large data set in
>> current Ansel code, PHP dies with memory exhausted fatal error as
>> Ansel will eat more than 128 MB of ram.
>
> Do you know where it is exhausting the memory? During what operation?

The main problem is in the listGalleries() usage. With a really lot of
galleries the list grows up to thousands of entries wherever we don't limit
the results. This is done in block galleries selection and
getRecentImages() method. We should limit gallery selection in the blocks
parameters. For example if we count more then 50 galleries we should skip
galleries selections or allow to jut enter the gallery id. For
getRecentImages() we must allow the galleries selection in a direct SQL
query. Of course is impossible to run a SQL query with thousands of
gallery_id in the IN(...) clause. I attaching a patch, maybe a little hacky
but is all I can do for now.