arviz-devs / arviz-plots

ArviZ modular plotting
https://arviz-plots.readthedocs.io
Apache License 2.0
2 stars 1 forks source link

Set default color in plot_dist #56

Open OriolAbril opened 2 months ago

OriolAbril commented 2 months ago

Right after https://github.com/arviz-devs/arviz-plots/blob/6f8b7507888cacfc6fcab488411d179da0b853ab/src/arviz_plots/plots/distplot.py#L229 we should check if color has an aesthetic mapped to it and if not srt a default color. Otherwise due to matplotlib default property cycle we get lines with different colors without any aesthetic mapping.

We already do something similar for plot_forest: https://github.com/arviz-devs/arviz-plots/blob/main/src/arviz_plots/plots/forestplot.py#L489-L493