Enable the use of "hour" as a period #2629

There should be an option to show data for any graph by the hour.
Keywords: period hour

Attachment: Patch implementing this feature
hour_period.patch

Attachment: Defines new period=hour and subperiod=hour
hour_period.2.patch

Hello, can anyone review this patch? Thanks a lot!

I'll defer to Matt.

It doesn't look like a patch against trunk. The advisory lock would need to be updated (segment is missing too).

There needs to be a config option to enable this feature, given the increased data storage reqts.

In class Piwik_Period_Hour, I'm not familiar with the H\H time format.

            $out = $this->getDateStart()->toString('Y-m-d H\H') ;

This patch is against r5111 of http://dev.piwik.org/svn/trunk
The "H\H" time format is there so labels show as "2011-08-15 16H".

I was thinking about the config option but I couldn't make it work without actually saving to the database the data. Maybe having some cron process delete old records from the database would be good enough?

cesarizu, Thank you for the proposal and patch, but unfortunately providing hourly stats is not on the roadmap for now, since it adds performance overhead for benefit to only a small set of users.

it would be great if this could be done as a plugin but it sounds like maybe too big of a change to work as a plugin?

I don't think it would work as a plugin since the patch shows changes to core.

Just in case anyone needs, this patch also includes an option to define subperiod=hour so period can be 'range' or 'day' or 'week' and subperiod hour. I haven't test this very thoroughly.