Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Implement x axis manipulation in plotting tab #715

Merged
merged 1 commit into from
Jun 20, 2024
Merged

Conversation

gemenerik
Copy link
Member

The last N seconds can now be plotted.
A range can now be set in seconds (low, high limits). The starting range is automatically adapted to match the latest view in samples / seconds mode. This makes the range mode useful for zooming in on certain parts of the plot.

Changed x axis behavior such that the range configuration (samples, seconds, range) dictates x limits of the plot even if there is a (partial) lack of data for the range. Rather than stretching the plot as it fills up, now the entire plot limits are shown immediately. This immediate feedback helps with choosing a desirable x axis setting.

The last N seconds can now be plotted.
A range can now be set in seconds (low, high limits). The range is automatically adapted to match the latest view in samples / seconds mode. This makes the range mode useful for zooming in on certain parts of the view.

Changed x axis behavior such that the range configuration (samples, seconds, range) dictates x limits of the plot even if there is a (partial) lack of data for the range. Rather than stretching the plot as it fills up, now the entire plot limits are shown immediately. This immediate feedback helps with choosing a desirable x axis setting.
@gemenerik gemenerik marked this pull request as ready for review June 17, 2024 08:32
Copy link
Contributor

@ToveRumar ToveRumar left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good! Nice feature!

Great commit message, makes it easy to review.

Image would help to see the change visually.

@gemenerik
Copy link
Member Author

Looks good! Nice feature!

Great commit message, makes it easy to review.

Image would help to see the change visually.

Thanks for the review! Adding an image to a GUI related pull request is a good idea!

@gemenerik gemenerik merged commit 254f997 into master Jun 20, 2024
1 check passed
@gemenerik gemenerik deleted the rik/plotting branch June 20, 2024 08:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants