Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Already on GitHub? Sign in to your account

Log scale pickle #1989

Merged
merged 2 commits into from May 10, 2013

Conversation

Projects
None yet
2 participants
Member

pelson commented May 9, 2013

Fixes #1539.

@mdboom mdboom merged commit 87bd8b8 into matplotlib:master May 10, 2013

1 check passed

default The Travis CI build passed
Details
Owner

mdboom commented May 10, 2013

Manually merged as f4d3a1b

@mdboom mdboom referenced this pull request May 10, 2013

Closed

Pickling of log axes #1539

Member

pelson commented May 13, 2013

Thanks @mdboom. Do we want to head down the path of deprecating the use of the nested classes (via the aliases)?

Owner

mdboom commented May 13, 2013

I think as an API, the nested classes are nice -- i.e. one would look for transforms related to log in the log scale's namespace, and the only reason we're doing this is to make pickling work... So I'm sort of 👎 on deprecating them. However, I could be convinced otherwise with the right argument.

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