Setting weekends to false causes burndown line to disappear

[anonymous]

I am attempting to display a burndown chart as follows.

[[BurndownChart(milestone=TBR 2.0, startdate=2009-07-08)]]

This works correctly, but if I try setting weekends to false the burndown line disappears.

[[BurndownChart(milestone=TBR 2.0, startdate=2009-07-08, weekends=False)]]

I looked through the code and am uncertain what the problem is.

[evan]

I am having this same issue. I thought it might be a timezone issue, but the server is correctly set to Australian Easter Standard Time.

[evan]

The chart URL with weekends is

​http://chart.apis.google.com/chart?chs=750x400&chf=c,s,ffffff00|bg,s,00000000&chd=t:0.00,3.13,6.25,9.38,12.50,15.63,18.75,21.88,25.00,28.13,31.25,34.38,37.50,40.63,43.75,46.88,50.00,53.13,56.25,59.38,62.50,65.63,68.75,71.88,75.00,78.13,81.25,84.38,87.50,90.63,93.75,96.88,100.00|91.48,94.54,94.54,100.00,74.89,77.07,71.18,68.78,66.38,66.38,66.59,66.59,-1,-1,-1,-1,-1,-1,-1,-1,-1,-1,-1,-1,-1,-1,-1,-1,-1,-1,-1,-1,-1|0,100|157.21,0&cht=lxy&chxt=x,x,y&chxl=0:|12|13|14|15|16|17|18|19|20|21|22|23|24|25|26|27|28|29|30|1|2|3|4|5|6|7|8|9|10|11|12|13|14|1:|11/2009|12/2009&chxr=2,0,458.0&chm=R,ccccccaa,0,0.04,0.11|R,ccccccaa,0,0.26,0.33|R,ccccccaa,0,0.48,0.55|R,ccccccaa,0,0.70,0.77|R,ccccccaa,0,0.92,0.99&chg=3.13,10.917&chco=ff9900,ffddaa&chtt=GIST%20v02.13

and the (broken) chart URL without weekends is

​http://chart.apis.google.com/chart?chs=750x400&chf=c,s,ffffff00|bg,s,00000000&chd=t:0.00,4.55,9.09,13.64,18.18,22.73,27.27,31.82,36.36,40.91,45.45,50.00,54.55,59.09,63.64,68.18,72.73,77.27,81.82,86.36,90.91,95.45,100.00,104.55,109.09,113.64,118.18,122.73,127.27,131.82,136.36,140.91,145.45|96.77,100.00,-1,-1,-1,-1,-1,-1,-1,-1,-1,-1,-1,-1,-1,-1,-1,-1,-1,-1,-1,-1,-1|0,100|166.28,0&cht=lxy&chxt=x,x,y&chxl=0:|12|13|16|17|18|19|20|23|24|25|26|27|30|1|2|3|4|7|8|9|10|11|14|1:|11/2009|12/2009&chxr=2,0,433.0&chm=&chg=4.55,11.5473&chco=ff9900,ffddaa&chtt=GIST%20v02.13

[François Boissonneault]

Yes, we've seen this too. I believe the calculation of "days" removed from the burndown based on the number of weekends (or whatsoever) is wrong. It would be cool to get a fix for this too (but also cool to be able to manually set what days we're off).

[osimons]

Replying to FakePlasticFrank:

It would be cool to get a fix for this too (but also cool to be able to manually set what days we're off).

Agreed. I'd like to see a more flexible handling of this as well. I'm thinking of something like nowork=weekends|2010-08-17|2010-08-24:2010-08-25 where weekends is just a keyword that gets expanded to appropriate days (like now) but where also single days or ranges are accepted.

With something like this the weekends argument should be removed, and the toggle of display could also be done as part of the nowork argument by supporting an optional hide word.

A default of nowork=weekends mimics the current defaults. Using nowork=weekeds|hide would remove weekeds just like weekeds=false.

Anyway, the idea is good but the syntax may need verification by others. I'll help out with a patch if you think this is a good idea.

[osimons]

I noticed this bug earlier, and intended to look at it again after I got [8758] committed. But now I can't recreate the bug... Have I somehow unintentionally improved this, or can someone still offer me a non-working example using latest trunk version?

[mnsh]

possible fix

[mnsh]

I ran into this problem with latest trunk, and I think I see why it happens: _scale_data() assumes that enddate - startdate is the number of dates we're dealing with, which is no longer true after the weekends have been removed.

See if attachment:estimationtools.patch​ solves this for you.

[osimons]

(In [9947]) EstimationToolsPlugin: Fix for burndown chart line disappearing with weekends=False set.

Solution provided by mnsh (thanks!), and I've added some tests that verifies what should be the correct scaling. Closes #5545.