jmhdz / currentcostgui

Automatically exported from code.google.com/p/currentcostgui
GNU General Public License v3.0
0 stars 0 forks source link

Average Weekday Graph Not Correct #16

Open GoogleCodeExporter opened 9 years ago

GoogleCodeExporter commented 9 years ago
Hi Dale

Great app, just a small issue the Average Day graph has a title of Average
Weekday.  Actually the code isn't using weekday it's actually using weekends

       for k, v in hourData.iteritems():
            if v > 0:
                if k.weekday() >= 5:
                    hoursCount[k.hour] += 1
                    hoursSum[k.hour] += v

While not being wrong, a week day and weekend version would be useful (or
quick title change of course).

Regards

Rob

Original issue reported on code.google.com by ro...@btopenworld.com on 19 Dec 2009 at 1:14

GoogleCodeExporter commented 9 years ago
Good spot! :-)

Thanks very much for letting me know - I agree that a separate weekend and 
weekday 
graph would be a good fix. Will do it after Christmas

Original comment by dale.l...@gmail.com on 23 Dec 2009 at 10:38