@anonymous-matomo-user opened this Issue on August 15th 2011

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

@anonymous-matomo-user commented on August 15th 2011

Attachment: Patch implementing this feature

@anonymous-matomo-user commented on September 22nd 2011

Attachment: Defines new period=hour and subperiod=hour

@anonymous-matomo-user commented on August 15th 2011

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

@robocoder commented on August 15th 2011 Contributor

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') ;
@anonymous-matomo-user commented on August 15th 2011

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?

@mattab commented on August 21st 2011 Member

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?

@robocoder commented on August 22nd 2011 Contributor

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

@anonymous-matomo-user commented on September 22nd 2011

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.

This Issue was closed on September 22nd 2011
Powered by GitHub Issue Mirror