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

[Doc]: Inappropriate example in gallery #26604

Closed
jsdodge opened this issue Aug 26, 2023 · 7 comments · Fixed by #26606 or #26785
Closed

[Doc]: Inappropriate example in gallery #26604

jsdodge opened this issue Aug 26, 2023 · 7 comments · Fixed by #26606 or #26785
Labels
Documentation Good first issue Open a pull request against these issues if there are no active ones!
Milestone

Comments

@jsdodge
Copy link
Contributor

jsdodge commented Aug 26, 2023

Documentation Link

https://matplotlib.org/stable/gallery/statistics/histogram_features.html

Problem

  • Given the sordid historical misuse of IQ, this gallery example unnecessarily reinforces a crude association between IQ and "smarts" that distracts from the main point, which is to show how to make a histogram.
  • Any attempt to contextualize the meaning of IQ would further distract from this point, so it would be better to just change the axis labels to avoid the negative association.

Suggested improvement

  • Remove the axis labels altogether, as was done here, or change the simulation example to something more neutral, such as the masses of apples from a hypothetical farm.
@jklymak
Copy link
Member

jklymak commented Aug 26, 2023

I agree that examples should avoid issues to do with humans that could be misconstrued, no matter how well meaning. This is a pretty easy fix if anyone wants to come up with a better example.

@jklymak jklymak added the Good first issue Open a pull request against these issues if there are no active ones! label Aug 26, 2023
@github-actions
Copy link

Good first issue - notes for new contributors

This issue is suited to new contributors because it does not require understanding of the Matplotlib internals. To get started, please see our contributing guide.

We do not assign issues. Check the Development section in the sidebar for linked pull requests (PRs). If there are none, feel free to start working on it. If there is an open PR, please collaborate on the work by reviewing it rather than duplicating it in a competing PR.

If something is unclear, please reach out on any of our communication channels.

@gautamsagar99
Copy link
Contributor

Hello,

I'm new to open source contribution, I want to solve this good first issue. I hope no one had created a PR for this issue can I start working on it?

Thanks

@RohanG610
Copy link

Is this issue still open?

@svgnix
Copy link

svgnix commented Aug 26, 2023

If the issue is still open, can you please assign me? Would love to work on it.

rcomer pushed a commit that referenced this issue Aug 26, 2023
* [Doc] Revise histogram features example (Addresses Issue #26604)

Remove reference to IQ scores and "Smarts" in figure; modify distribution parameters; update RNG syntax

* Update histogram_features.py

Wrap title string

* Fix trailing whitespace

* Restore RNG seed to 19680801
meeseeksmachine pushed a commit to meeseeksmachine/matplotlib that referenced this issue Aug 26, 2023
@rcomer rcomer added this to the v3.8-doc milestone Aug 26, 2023
@rcomer
Copy link
Member

rcomer commented Aug 26, 2023

@sagniksantra we do not assign issues. Please see our contributors’ guide:
https://matplotlib.org/devdocs/devel/contributing.html

timhoffm added a commit that referenced this issue Aug 26, 2023
…606-on-v3.8.x

Backport PR #26606 on branch v3.8.x ([Doc] Revise histogram features example (Closes #26604))
@story645
Copy link
Member

I agree that examples should avoid issues to do with humans that could be misconstrued, no matter how well meaning.

I think in situations like this where the concerns are about whether an example is a good fit for the community, rather than its technical suitability, then the guiding principle on whether an example should be removed should be on its alignment with our mission statement and core values.

I agree that IQ should be removed-not because it's about humans but because highlighting bad science rooted in eugenics goes against our values of facilitating good science and aiming for inclusivity. There are situations where removing an example could work against our values, and harm community engagement, in that it could signify, probably unintentionally, that we consider data about that demographic, and implicitly that demographic, inherently controversial. There are also plenty of non-human datasets that we would also reject on the basis of it being bad science or otherwise in conflict with our values.

Conversely there are situations where including human data is completely in alignment with our mission and values. In those situations it could even promote inclusion and community engagement as it's a low effort way to signify that we as a project believe in our values enough to put them on display in our gallery. Which bolsters the credibility of the mission statement and code of conduct. Which is why our criteria for evaluating examples where the issue is community, not technical, should be whether it promotes our values.

Maybe we can discuss this more in a meeting?

@QuLogic QuLogic modified the milestones: v3.8-doc, v3.8.0 Aug 28, 2023
@ksunden ksunden mentioned this issue Sep 15, 2023
5 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Documentation Good first issue Open a pull request against these issues if there are no active ones!
Projects
None yet
9 participants