Modify

Opened 3 years ago

Closed 3 years ago

#9728 closed defect (invalid)

burndown chart broken with recent gviz libraries

Reported by: anonymous Owned by: robguttman
Priority: normal Component: SumStatsPlugin
Severity: normal Keywords:
Cc: Trac Release: 0.12

Description

The burndown chart was not displaying and I found the following javascript error:

"Bars series with value domain axis is not supported"

It appears the google vis library has changed, details here:

http://code.google.com/p/google-visualization-api-issues/issues/detail?id=795

I was able to resolve it by converting the dates to strings.

$ diff -u sumstatsplugin/0.12/sumstats/web_ui.py.orig sumstatsplugin/0.12/sumstats/web_ui.py
--- sumstatsplugin/0.12/sumstats/web_ui.py.orig 2012-01-25 12:49:58.000000000 -0500
+++ sumstatsplugin/0.12/sumstats/web_ui.py      2012-01-25 12:46:46.000000000 -0500
@@ -345,12 +345,12 @@
             else:
                 total,done,todo = self._get_burndown_day(milestone, day)
             # add to rows
-            rows.append({'day':day, 'total':total, 'todo':todo, 'done':done})
+            rows.append({'day':day.date(), 'total':total, 'todo':todo, 'done':done})
             day += timedelta(days=1)

         # package the data for the google visualization query response
         import gviz_api
-        schema = {"day": ("date", "Day"),
+        schema = {"day": ("string", "Day"),
                   "total": ("number", "Total"),
                   "todo": ("number", "To Do"),
                   "done": ("number", "Done")}

Attachments (0)

Change History (4)

comment:1 Changed 3 years ago by robguttman

Yes, a bit frustrating that Google doesn't let you stick with prior versions! But this is actually a problem on the chart rendering/visualization side, not on this data provider side.

Another interim workaround for the visualization side is to use their newly documented strictFirstColumnType option. If using the VisualizationPlugin, for example:

[viz]
options = strictFirstColumnType:false

This should make the chart work with still using a continuous value for the (date) x-axis. But this is just an interim solution until I get around to a more robust, permanent solution for the VisualizationPlugin.

comment:2 Changed 3 years ago by robguttman

  • Resolution set to wontfix
  • Status changed from new to closed

As there is a workaround for this issue, I'm going to close as wontfix. If/when Google removes this deprecated functionality, I'll address at that time.

comment:3 Changed 3 years ago by robguttman

  • Resolution wontfix deleted
  • Status changed from closed to reopened

comment:4 Changed 3 years ago by robguttman

  • Resolution set to invalid
  • Status changed from reopened to closed

Actually, this ticket should have been against the VisualizationPlugin, so marking as invalid for the SumStatsPlugin.

Add Comment

Modify Ticket

Action
as closed The owner will remain robguttman.
The resolution will be deleted. Next status will be 'reopened'.
Author


E-mail address and user name can be saved in the Preferences.

 
Note: See TracTickets for help on using tickets.