Skip to content

Scatter min/max render indexes#763

Merged
swharden merged 2 commits intoScottPlot:masterfrom
StendProg:ScatterMinMaxRenderIndexes
Feb 10, 2021
Merged

Scatter min/max render indexes#763
swharden merged 2 commits intoScottPlot:masterfrom
StendProg:ScatterMinMaxRenderIndexes

Conversation

@StendProg
Copy link
Contributor

@StendProg StendProg commented Feb 8, 2021 •
edited
Loading

Purpose:
Implement MinRenderIndex and MaxRenderIndex for ScatterPlot. #737

API implemented based on the existing stub. Feel free to change.
The unresolved question was what to do when XS and YS arrays were changed, whether to change Indexes automatically or leave it to the user's discretion. Current second option.
A demo is also required for this mode...

New Functionality:

double[] xs = DataGen.Consecutive(51);
double[] sin = DataGen.Sin(51);
double[] cos = DataGen.Cos(51);

var scatter = plt.AddScatter(xs, sin);
scatter.MinRenderIndex = 12;
scatter.MaxRenderIndex = 42;

image

@swharden
Copy link
Member

swharden commented Feb 10, 2021

This is wonderful @StendProg, thank you so much! 😄

@swharden
Copy link
Member

swharden commented Feb 10, 2021

PS: This feature in combination with #743 will really help https://github.com/swharden/AP-Phase-Analysis

@swharden swharden merged commit 4367d77 into ScottPlot:master Feb 10, 2021
This was linked to issues Feb 10, 2021
Scatter Plot Min/Max Render Index #737
Closed
Scatter plots should support max/min render index #566
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Scatter Plot Min/Max Render Index Scatter plots should support max/min render index

2 participants

@StendProg @swharden