srggrs opened a new pull request #5928: added hack to make time pivot chart 
working
URL: https://github.com/apache/incubator-superset/pull/5928
 
 
   guys
   
   I noticed there is a bug in the chart time periodicity pivot. For example if 
you use the example birth data as the datasource/table ("birth_names), the 
chart return and error
   
   `'NoneType' object is not iterable`
   
   so I went down and dig in the code and realised there is some issue in 
getting the metrics values from metric. So I added that line below that make 
the chart working (try to use another data source with data time resolution of 
weeks or day to see the overlapping of all the lines as all the example data 
are with year minimum resolution hence can overlap them as there is no frequecy 
option higher than year).
   
   I think the problem is that  in the class `NVD3TimePivotViz`, the method 
`query_obj` can't override the default method that is in the base class 
`BaseViz`.
   
   Am I right?

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
[email protected]


With regards,
Apache Git Services

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to