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

max_points in plot_trajectories should be None by default #272

Open
mauruscz opened this issue Oct 18, 2023 · 0 comments
Open

max_points in plot_trajectories should be None by default #272

mauruscz opened this issue Oct 18, 2023 · 0 comments

Comments

@mauruscz
Copy link
Collaborator

mauruscz commented Oct 18, 2023

The default value of the parameters max_pointsin plot_trajectory should be None.

By default, I expect to see the whole trajectory plotted, then if I have computational/visualization problem I could downgrade to a fixed number of points (for example 1000 points, as the current default value).

Having a fixed numerical threshold of points can yield to inconsistent results. For example:

I have a trajectory of 50K points and I plot it with default options
I do the same for the compressed version

--> I obtain a visualization of only 1k points of the first trajectory that is not necessarily consistent with the visualization of 1k points of the compressed one.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant