Skip to content

EventPlot: Don't use a variable for the data

Javi Merino requested to merge github/fork/JaviMerino/two_eventplots_fail_copy into master

When we generate the data for EventPlot, we store it in a variable called "data". If there are two plots in a notebook and the notebook is copied (or the page reloaded), the second "data" variable clobbers the first one, so when requirejs gets round to execute EventPlot.generate(), all EventPlots use the same data.

We don't really need to store the data in a variable, we can just pass it straight to EventPlot.generate()

Merge request reports