Re: [Maposmatic-dev] [PATCH maposmatic] Add a rendered maps feed

[Maxime Petazzoni]

Hi,

* Thomas Petazzoni <address@hidden> [2010-02-09 22:36:31]:

> Ok for me, except one question :
> 
> > +        return (models.MapRenderingJob.objects
> > +                .filter(status=2)
> > +                .filter(resultmsg='ok')
> > +                .order_by('-endofrendering_time')[:10])
> 
> If I understand correctly, it means that at any given time, the RSS
> feed will only contains the 10 latest renderings. It means that if the
> mean rendering for the last 10 renderings is 4 minutes, they will
> disappear after 40 minutes.
> 
> If the user reloads its RSS feed only every hour, or worse everyday,
> the user will miss a lot of renderings.
> 
> So I think we should list a larger number of renderings, or list all
> renderings for the past 24/48 hours ?

That's a good point. But we need a combination of the two: all
renderings from the past 24 hours, but /at least/ ten of them, when
possible, otherwise we could end up with an empty feed (not likely, but
we never know).

Let's put this in first, I'll send a patch to fix that specific object
selection for the RSS feed later.

- Maxime

-- 
Maxime Petazzoni <http://www.bulix.org>
 ``One by one, the penguins took away my sanity.''
Linux kernel and software developer at MontaVista Software

signature.asc
Description: Digital signature