FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

Marker/scatter reconciliation by mdboom · Pull Request #400 · matplotlib/matplotlib · GitHub

Repository navigation

Marker/scatter reconciliation - #400

Merged
mdboom merged 9 commits into
matplotlib:masterfrom
mdboom:marker_scatter_reconcile
Jul 25, 2011
Merged

mdboom merged 9 commits into
matplotlib:masterfrom
mdboom:marker_scatter_reconcile

Conversation

mdboom commented Jul 15, 2011

Copy link
Copy Markdown
Member

The set of available Line2D markers and scatter markers has always been slightly different. Some features (such as mathtext markers) were only available in one place and not the other. This refactors everything into a new markers.py module which is used from everywhere else to keep the functionality consistent and equally capable everywhere.

This seems to pass all regression tests and doesn't change any relevant examples, but it's pretty core functionality, so care should be taken it doesn't break anything.

mdboom commented Jul 15, 2011

Copy link
Copy Markdown
Member Author

Addresses bugs #350, #125

Comment thread CHANGELOG Outdated

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality

2011

mdboom added a commit that referenced this pull request Jul 25, 2011
mdboom merged commit fafc0cf into matplotlib:master Jul 25, 2011
mdboom deleted the marker_scatter_reconcile branch March 3, 2015 18:44
QuLogic added this to the v1.1.0 milestone Nov 4, 2015
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
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants


Back | FazBrowse Home | New Git URL