philogb / jit

The JavaScript InfoVis Toolkit provides tools for creating Interactive Data Visualizations for the Web
http://thejit.org
Other
1.51k stars 297 forks source link

Bug in Class Sunburst (incl. solution) #162

Open kanohi opened 11 years ago

kanohi commented 11 years ago

In NodeTypes 'multipie' (and maybe other places) the line var polar = new Polar(polarNode.rho, begin); must be changed to var polar = new Polar(begin, polarNode.rho);

as Polar is defined as follows: var Polar = function(theta, rho)

The bug has currently little effect, because the fill() is still working properly. But if one wants to draw lines instead of fill the bug becomes visible