How to update a signal plot with a new data array

Hi

In fact, I use below method to achieve growing data display.

// initial
ScottPlot.PlottableSignal my_plt;
System.Random my_rand = new Random();
List<double> data_growing = new List<double>();
private void formsPlot1_Load(object sender, EventArgs e)
{
    this.data_growing = ScottPlot.DataGen.Random(my_rand, 10).ToList();
    this.my_plt = this.formsPlot1.plt.PlotSignal(data_growing.ToArray());
    this.formsPlot1.Render();
}

// growing
private void button1_Click(object sender, EventArgs e)
{
    data_growing.AddRange(ScottPlot.DataGen.Random(my_rand, 5).ToList());
    this.my_plt.ys = this.data_growing.ToArray();
    this.my_plt.maxRenderIndex = this.my_plt.ys.Length - 1;
    this.formsPlot1.plt.AxisAuto();
    this.formsPlot1.Render();
}

But I encountered a problem when I updated from 4.0.39 to 4.0.40. It will report the error IndexOutOfRangeException.

Because the data array my_plt.ys is not synchronized with the array my_plt.minmaxSearchStrategy.SourceArray

Here’s an example of what I’d like to do: Maybe can modify the PlottableSignalBase.updateData program like this

public void updateData(T[] newData)
{
    //updateData(0, newData.Length, newData);
    this.ys = newData;
    this.maxRenderIndex = this.ys.Length - 1;
    this.minmaxSearchStrategy.SourceArray = newData;
}

Then I can update ys and SourceArray by updateData function at the same time.

private void button1_Click(object sender, EventArgs e)
{
    data_growing.AddRange(ScottPlot.DataGen.Random(my_rand, 5).ToList());
    //this.my_plt.ys = this.data_growing.ToArray();
    //this.my_plt.maxRenderIndex = this.my_plt.ys.Length - 1;
    this.my_plt.updateData(this.data_growing.ToArray()); // Replace the old method
    this.formsPlot1.plt.AxisAuto();
    this.formsPlot1.Render();
}

[HowardWhile]

Hi @swharden @StendProg

Thanks for your suggestions. I will follow Solution 2 and make some adjustments to my program to adapt to the new version.

I look forward to the next version and glad to see the outcome.

[swharden]

@HowardWhile I’m glad we arrived at a solution that works for you!

Good discussion remains about how to improve plottable properties (capitalizing them, making them read-only, etc.), but that’s outside the scope of this issue so I’ll go ahead and close it