make sunburst arc ordering deterministic

[allenday]

currently when it refreshes the segments can move around.
I suggest ordering by either:

• lexicographic on segment name
• proportional size of segment

[allenday]

@Firedrops want to take this one?

[Firedrops]

Same as other visualizer issue, it seems other examples use code like this to render the arcs, which also orders them.

var arc = d3.svg.arc()
    .startAngle(function(d) { return Math.max(0, Math.min(2 * Math.PI, x(d.x))); })
    .endAngle(function(d) { return Math.max(0, Math.min(2 * Math.PI, x(d.x + d.dx))); })
    .innerRadius(function(d) { return Math.max(0, y(d.y)); })
    .outerRadius(function(d) { return Math.max(0, y(d.y + d.dy)); });

but I can't find anything superficially similar in our source. Might be able to spot it with some learning, though.

[allenday]

It will be something farther up the stack. That's rendering logic, you are looking for sorting logic.

…

On Mon, Feb 18, 2019, 08:05 Firedrops ***@***.***> wrote: Same as other visualizer issue, it seems other examples use code like this to render the arcs var arc = d3.svg.arc() .startAngle(function(d) { return Math.max(0, Math.min(2 * Math.PI, x(d.x))); }) .endAngle(function(d) { return Math.max(0, Math.min(2 * Math.PI, x(d.x + d.dx))); }) .innerRadius(function(d) { return Math.max(0, y(d.y)); }) .outerRadius(function(d) { return Math.max(0, y(d.y + d.dy)); }); but I can't find anything superficially similar in our source. Might be able to spot it with some learning, though. — You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub <#67 (comment)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/AAanP-7Ac41wcFUqiQphWTeSgVsVzWzxks5vOe5HgaJpZM4a_goi> .