-
Notifications
You must be signed in to change notification settings - Fork 12
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
[Micro task] Update a Bokeh example plot #7
Comments
Hello @pavithraes |
Hi @pavithraes |
Changed colorbar_log example from viridis to iridescent |
Updated Color pallete for examples\plotting\histogram.py to be more visible and less conflicting |
Updated colour palette |
hey , @pavithraes , @bryevdv i have changed the color of the jitter graph taken from the gallery. Can you review my work at https://github.com/KesharwaniArpita/outreach-programs/tree/Gallery_bokeh |
Hi @KesharwaniArpita Thanks for the links. I am not sure that the new colors are an improvement according to the criteria of accessibility. One of the most common color deficiences is an inability to distinguish red from green, but these examples seem to be updated us use both those hues together. I think the jitter example may also have issues with low background/foreground contrast. Here are some links that may be helpful:
|
|
Hello mentors @bryevdv , @pavithraes. I have a question. I noticed that some colors in the example plots were generated using loops. For instance |
Hi @Faith-Nchifor those are some very old examples that have also been used widely in various public project materials over the years. I think perhaps they could be improved, but I think it's not so clear cut and needs a more involved discussion. I'd advise looking at some of the other examples before those. |
okay @bryevdv. |
I worked on the slope plot bokeh/bokeh#12959 |
Saw some of those when I was trying out the examples. They all still work on the current version, while palettes like tol will output errors on older versions like bokeh-2.4.3 (which is the default in a Google Colab notebook)
|
Hello! I created a PR to make a small change to the span example in the gallery, changing from solid to dashed lines. TBH this is my first time making a PR to something other than a project between me and friends, so I’m very open to any feedback about the process as well as code. |
Thanks @fractaldatalearning jus FYI very many of the example plots have been updated in the repo but to not yet appear in the published gallery, because there has not yet been a new release since those changes. At this point, in order to see the true current state of the gallery, you would need to build the docs locally as described in the contributor guide. |
Thanks for the clarification! It makes sense that there would actually be many more plots that are updated by now. To find any additional visualizations to work on, I’ll look through the repo. |
Select a Bokeh plot from the examples gallery, review it for accessibility (primarily, check the colors used in the plot), and update it to use more accessible colours if needed.
Share the plot you are working on a comment on this issue, so that we don't have multiple participants working on the same example.
You can make a PR against bokeh/bokeh#11481 for this micro task to update plots. If you find plots that don't need updating, share it on this issue as a comment.
The text was updated successfully, but these errors were encountered: