Slow performance of new version of d3-tip about d3-tip HOT 6 CLOSED

Wow, that's an order of magnitude slower. Quickly looking at your code, it looks like you're initializing a new instance every time you mouseover a node here and in similar areas. Try moving the invocation to the parent SVG element. You only need to call tip.show and tip.hide in the mouseover and mouseout events.

from d3-tip.

a5b1216 should also help. It performed pretty well with 5k nodes and was just acceptable at around 10k. I haven't really dug into performance tuning yet, but I have a few ideas that I think could speed things up:

• Only perform matrix transforms on the direction you're interested in.
• Remove expensive operations from tip.show.
• Possibly use direct DOM APIs in tip.show instead of d3.

from d3-tip.

That doesn't seem to be the problem. svg.call(tip) happens just once for each of the four types of tips I've made, and tip.show and tip.hide are just in the mouseover/mouseout events.

The line you pointed to is in the original (faster) version of the script, using the old version of d3-tip. Here's the corresponding line in the revised version.

I initialize at lines 300-303

from d3-tip.

You're right, I was looking at the wrong file. I just did some performance tuning. Optimizing this loop cut the time in half, but I don't think it's going to be as fast as the original version because the original version was inherently broken for all but the most simple cases.

What's happening here is a bunch of mouseover events are being triggered, one right after the other, causing the operations to run on nodes that are likely not the node the user's mouse stops on. One option is to delay tip.show so it doesn't get called every time the user moves their mouse.

Unfortunately, I wasn't able to get your example running locally. Here's the updated version: https://raw.github.com/Caged/d3-tip/performance-tuning/src/d3.tip.js

from d3-tip.

Understood; thanks for spending so much time looking into it. Adding a delay does seem like the best option.

from d3-tip.

No problem!

from d3-tip.