react-plotlyjs icon indicating copy to clipboard operation
react-plotlyjs copied to clipboard

Add cloneDeep for data

Open Scimonster opened this issue 8 years ago • 1 comments

I found my app rerendering too often due to the data object changing as well. Plotly was adding uid keys to the traces.

Scimonster avatar Aug 10 '17 10:08 Scimonster

Thanks for the PR, I'm a little wary of merging this as cloneDeep can be very expensive. I begrudgingly added it to layout as it tends to be limited in size - however data can be a very large object. Ideally fixing #6 properly should fix this. Putting this on hold for now.

benjeffery avatar Aug 25 '17 13:08 benjeffery