-
-
Notifications
You must be signed in to change notification settings - Fork 653
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
Deterministic line color sequence #196
Comments
Good point @alexswerner. Thanks @facontidavide. I am using this great plotter on a daily basis now. |
I have been waiting people to ask this for a long time ;) Actually, once you drag a new timeseries into a plot, a color is assigned to it and is reused if the same timeseries is added to another plot. What I do is to give the color blue to timeseries @JimmyDaSilva, I improved a bit the palette in 2.3.0, but I need to create different palettes for dark and light theme, because the default one will not have sufficient contrast for both themes. If anyone wants to suggest a palette, feel free to propose a PR modifying this file: https://github.com/facontidavide/PlotJuggler/blob/master/include/PlotJuggler/random_color.h For reference: https://seaborn.pydata.org/tutorial/color_palettes.html https://ux.stackexchange.com/questions/85088/ideal-color-palette-for-24-colors http://repec.sowi.unibe.ch/stata/palettes/getting-started.html |
I improved the palette of colors and it should be less annoying. Since I believe the current behavior is the "right one", I will close the issue. But thanks for the feedback 👍 |
Please test this branch https://github.com/facontidavide/PlotJuggler/tree/curve_colors |
Hi,
it would be nice to have deterministic colors when adding lines to a plot. Whenever i create a new tab with a single plot and add multiple lines, the order of colors seems to be different. I suggest that the order in which lines are added to the plot determines the colors. Maybe the Light/Dark mode should predefine a set of colors.
Cheers,
Alex
The text was updated successfully, but these errors were encountered: