Movatterモバイル変換


[0]ホーム

URL:


Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

Merge "Scatter Symbol" and "Scatter Custom Symbol" examples#22276

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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Merged
timhoffm merged 1 commit intomatplotlib:mainfromtimhoffm:doc-scatter-symbol
Mar 17, 2022

Conversation

timhoffm
Copy link
Member

@timhoffmtimhoffm commentedJan 20, 2022
edited
Loading

Merge "Scatter Symbol" and "Scatter Custom Symbol" examples.

Remove "Scatter plot with pie chart markers". This is only a variant of
path-as-custom-symbol, which is already covered here. Also, all pie
markers in the scatter plot have the same data distribution, so the
pie is more a style thing and cannot convey additional
datapoint-specific information.

Alsocloses#22266 by usingsizes instead ofs and spelling out the
color to make their meaning clearer.

@timhoffmtimhoffm added this to thev3.6.0 milestoneJan 20, 2022
@timhoffmtimhoffmforce-pushed thedoc-scatter-symbol branch 2 times, most recently from1733ce1 tod345624CompareJanuary 20, 2022 23:45
Remove "Scatter plot with pie chart markers". This is only a variant ofpath-as-custom-symbol, which is already covered here. Also, all piemarkers in the scatter plot have the same data distribution, so thepie is more a style thing and cannot convey additionaldatapoint-specific information.Alsoclosesmatplotlib#22266 by using *sizes* instead of *s* and spelling out thecolor to make their meaning clearer.
@anntzer
Copy link
Contributor

@timhoffm
Copy link
MemberAuthor

Yes, but I don‘t have the time to work on this for now. Therefore I prefer to merge as is. Feel free to amend or open an issue so that we don‘t forget.

@timhoffm
Copy link
MemberAuthor

Self-merging based on the positive review as an incremental improvement.@anntzer I've added an issue based on your suggestion:#22663

tacaswell reacted with thumbs up emoji

@timhoffmtimhoffm merged commita2ddfc0 intomatplotlib:mainMar 17, 2022
@timhoffmtimhoffm deleted the doc-scatter-symbol branchMarch 18, 2022 22:38
Copy link

@dixon-kundixon-kun left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others.Learn more.

Ok now 👍

Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment
Reviewers

@oscargusoscargusoscargus approved these changes

@dixon-kundixon-kundixon-kun approved these changes

Assignees
No one assigned
Projects
None yet
Milestone
v3.6.0
Development

Successfully merging this pull request may close these issues.

[Doc]: Improve examples in documentation
4 participants
@timhoffm@anntzer@oscargus@dixon-kun

[8]ページ先頭

©2009-2025 Movatter.jp