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

Chdir import path fixes by mdboom · Pull Request #50 · matplotlib/matplotlib · GitHub

Repository navigation

Chdir import path fixes - #50

Closed
mdboom wants to merge 1 commit into
matplotlib:masterfrom
mdboom:chdir_import_path_fixes
Closed

mdboom wants to merge 1 commit into
matplotlib:masterfrom
mdboom:chdir_import_path_fixes

Conversation

mdboom commented Mar 22, 2011

Copy link
Copy Markdown
Member

In some situations, the matplotlib package seems to be imported for a relative module path. Since much of the backend wrangling involves importing submodules in functions, this means that a chdir() before calling one of these functions can raise confusing ImportErrors. The test framework performs such a chdir() in the image_comparison() decorator that many of the tests use, which causes spurious test errors.

Copy link
Copy Markdown
Member

Fine, I suppose. What happens if matplotlib is run from an egg file?

mdboom commented Mar 28, 2011

Copy link
Copy Markdown
Member Author

No idea. I never use setuptools. Can anyone else comment?

mdboom commented Jun 15, 2011

Copy link
Copy Markdown
Member Author

Someone who was able to reproduce this error (CTPUG folks) should comment. I've never reproduced the problem myself.

mdboom closed this Jun 1, 2012

mdboom commented Jun 1, 2012

Copy link
Copy Markdown
Member Author

Closing -- I was never certain about the need for this, and the OP has been silent for over a year.

richbwood mentioned this pull request Dec 19, 2012
magnunor pushed a commit to magnunor/matplotlib that referenced this pull request Dec 5, 2013
fariza added a commit to fariza/matplotlib that referenced this pull request Sep 22, 2016
…ab_camera

It makes more sense to have radiance as parameter for grab than a camera attribute
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.

2 participants


Back | FazBrowse Home | New Git URL