Line time series axis rerender loop

[LoicUV]

Render a line chart with time series on X axis. Rerender the parent component (eg useState). The X axis component enters a render loop.
I tried it with both native & string dates. Doesn't occur with a point scale.

Codesandbox

Looks like each defaultTickRenderer components of Axis is getting a new key every second.

[jamesdh]

I think I'm also experiencing this with Bar charts at the moment, but still digging into it.

[jamesdh]

I pushed a branch into my fork that contains a story which I believe demonstrates this issue.

Feel free to take a look: https://github.com/jamesdh/nivo/tree/bar_time_scale

[jamesdh]

Ok, please ignore my above comments. My issue is because Bar charts don't even support time scales as their "index" values.

[stephent]

I'm seeing something very similar with ResponsiveLine. It appears possibly to be a regression since version 64. If I rollback to 63.1, I don't see it.

With version 67, the line chart seems to end up in a render loop. This fires continuously sometimes from page load, sometimes after a user-initiated change that forces a chart re-render (after which it never stops):

These are the warnings in Chrome that caught my attention:

When I revert to 63.1, and profile using React Tools, the Axes only re-render when I expect (the yellow spikes here):

Let me know if there's more data I can provide or profiling that would be helpful.

[clarkd]

We are seeing the same thing, very slow performance and the axis re-rendering constantly.