3D plot redraw() inside a listener cause a recursive loop

I have the following code to be executed:

this.$.plot3d.on('plotly_click', function(d){
  self.data[1].x.push(parseFloat(d.points[0].x))
  self.data[1].y.push(parseFloat(d.points[0].y))
  self.data[1].z.push(parseFloat(d.points[0].z))
  redrawing = true
  Plotly.redraw(self.$.plot3d); 
})

It just adds a 3D point (x,y,z) to a trace and then it redraws the plot. If I do like this, as far as I seen, the redraw method somewhere trigger again the plotly_click event and this cause a recursive loop.

I used an ugly workaround to fix this, which is the following:

this.$.plot3d.on('plotly_click', function(d){
  if(!redrawing){        
    self.data[1].x.push(parseFloat(d.points[0].x))
    self.data[1].y.push(parseFloat(d.points[0].y))
    self.data[1].z.push(parseFloat(d.points[0].z))
    redrawing = true
    Plotly.redraw(self.$.plot3d);
  } else {
    redrawing = false
  }
})

But it’s not hte best way. Am I do correctly redrawing the graph in a listener? Is there any other way (maybe the right one) to redraw a graph after a point selection? Is this allowed?

Thank you

[mikemey]

I’ve found an awful workaround for this issue: place the call to Plotly.redraw/Plotly.restyle in a timeout with a small delay:

setTimeout(() => Plotly.restyle( ... ), 200)

Sometimes I still get multiple calls, but it stops the infinite event loop.

I’ll take a shower, I feel dirty…

[kelegorm]

Ok, how I see problem. in 1.6.0 version was presented fix for issue #240. It added click events for 3D scattergl. But it was done in strange way: clicks are firing on each redrawing while mouse button is down. For example, in same case 2D event is fired once mouse is up. But this is only part of issue.

Somewhere between 1.9.0 and 1.10.0 some change was done that makes things go recursively: now, if mouse was down and in it’s callback was redrawing event fires and fires again even after mouse was released, so selection.buttons == true all the time.

May be it was force redrawing, may be - some other change.

I hope, that info will help somebody to fix that problem, I still can’t get root of problem, what need to change. And while we are using both 2d and 3d scenes it’s hard to make easy way to overcome that problem.