Repository navigation
Changing bar examples to tea and coffee #23590
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
Merged
timhoffm
merged 24 commits into
matplotlib:main
from
SourajitaDewasi:change_bar_examples
Aug 27, 2022
Merged
Changes from all commits
Commits
Show all changes
24 commits
Select commit
Hold shift + click to select a range
39980d4
changing bar examples to tea and coffee
SourajitaDewasi dd83fd4
changing stacked bar examples to tea and coffee
SourajitaDewasi b73e8ff
changing stacked bar examples to tea and coffee
SourajitaDewasi 96cab12
changing stacked bar examples to tea and coffee
SourajitaDewasi 397141b
changing barchart examples to tea and coffee
SourajitaDewasi c9a098d
changing stacked bar examples to tea and coffee
SourajitaDewasi ff0d0e0
changing barchart examples to tea and coffee
SourajitaDewasi 9dcfa49
changing bar unit examples from men and women to bream_fish length an…
SourajitaDewasi 1eff97f
changing bar unit examples from men and women to bream_fish length an…
SourajitaDewasi a9bcdcf
changing bar unit examples from men and women to bream_fish length an…
SourajitaDewasi edbaa83
changing bar unit examples from men and women to bream_fish length an…
SourajitaDewasi c0f5d1f
changing bar unit examples from men and women to bream_fish length an…
SourajitaDewasi 3f8cb4d
changing bar unit examples from men and women to coffee and tea cup size
SourajitaDewasi 706ccd3
changing bar unit examples from men and women to coffee and tea cup size
SourajitaDewasi 7538ebc
changing bar unit examples from men and women to coffee and tea cup size
SourajitaDewasi 1e19265
changing bar unit examples from men and women to coffee and tea cup size
SourajitaDewasi 35207a2
changing stacked bar examples to tea and coffee
SourajitaDewasi 1c7dca2
changing bar unit from men and women to coffee and tea cup size
SourajitaDewasi 6fc4e49
changing bar label examples from men and women to coffee and tea
SourajitaDewasi 13d61d0
changing bar label examples from men and women to coffee and tea
SourajitaDewasi 9e6a79d
changing bar label examples from men and women to coffee and tea
SourajitaDewasi 49f8a6a
changing bar label examples from men and women to coffee and tea
SourajitaDewasi cf3733a
changing bar unit from men and women to coffee and tea cup size
SourajitaDewasi b8bf772
Update examples/units/bar_unit_demo.py
timhoffm File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I just realized that the example has been semantically weird already before. The stacked bar suggests that the quantity is summed up from both subgroups, and the errorbar indicates that within each subgroup the quantity is averaged. I cannot come up with a reasonable quantity for "Scores" that would actually work this way.
Second, before there were groups and inside these groups we distinguished by gender. This is something everybody is familiar with. Splitting a group up by beverage choice and scoring these subgroups is possible but an even more weird experiment.
I think we have two choices:
a) Come up with something completely new
b) Or accept that the plot is not semantically meaningful, merge and do something a) later.
@story645 what do you think?
Uh oh!
There was an error while loading. Please reload this page.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ugh, I want to be okay with merging something that's not semantically meaningful 'cause @SourajitaDewasi worked so much on this, but I hear you that the semantics being off might be a distraction that will make the example harder to follow.
Keeping it simple, maybe change the underlying pivot table to
(caffeinated, tea), (decaffeinated, tea), (caffeinated, coffee), (decaffeinated, coffee) and the y axis is like #avg drink rating (1-5) so the example is basically a made up rating of what type of drink folks prefer.
ETA: cup height probably works just as well, or average amount drunk or the like.
Alternatively, palmer penguins is being used a lot to illustrate aggregated visualizations, and we can probably keep this example pretty small if what we store in the example is the pivot table made from it https://allisonhorst.github.io/palmerpenguins/
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I would be fine with merging as-is as an improvement, with the suggestion below, and have a follow up either a) removing the example, or b) reworking it to include penguins or other good ideas.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ok, let's take this as is (suggestion below committed) so that we can git this in 3.6. And let's think about how we can further improve later.