Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Added colorblind compatibility cluster examples #5555

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

Closed

Conversation

johannah
Copy link
Contributor

Colorblind compatibility as discussed in #5435.

plot_adjusted_for_chance_measures.py
figure_1
figure_2

plot_affinity_propagation.py
figure_1

plot_agglomerative_clustering_metrics.py
figure_1
figure_5
figure_6
figure_7

plot_dbscan.py
figure_1

plot_kmeans_stability_low_dim_dense.py
figure_1

plot_mean_shift.py
figure_1

@johannah johannah changed the title Added colorblind compatibility Added colorblind compatibility cluster examples Oct 23, 2015
@amueller
Copy link
Member

Hm, I don't think I know how to review this. It seems to me that for example the mean-shift one here http://scikit-learn.org/dev/auto_examples/cluster/plot_mean_shift.html had more contrast before for Deuternanopia but less for Achromatopsia...

Base automatically changed from master to main January 22, 2021 10:48
@cmarmo cmarmo added Needs Decision - Close Requires decision for closing and removed Waiting for Reviewer labels Feb 14, 2022
@cmarmo
Copy link
Contributor

cmarmo commented Feb 14, 2022

The matplotlib default are now colorblind friendly, if I understand correctly (see #16808).

@cmarmo
Copy link
Contributor

cmarmo commented Oct 30, 2022

I am closing this pull request as all the examples have been modified (some of them using suggestions from this PR).
Thanks @johannah for working on this.

@cmarmo cmarmo closed this Oct 30, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Superseded PR has been replace by a newer PR
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants