GV

GeezerVison

The World Wide Web, restored to 800 pixels of glory.

Home|About|Gaming cabinet|Archives|How it works|Starter sites

On this site

  • Sign in
  • ActionsCI/CD and automation
  • CodespacesCloud development environments
  • All Features
  • Documentation
  • Training
  • Certifications
  • Build with Copilot
  • Customize Copilot
  • Integrations
  • Trending repos
  • GitHub Sponsors
  • Developer articles
  • Enterprises

This view

  • Original page
  • New address

GeezerVison is showing github.com — Include test_text in default_test_modules by ivanov · Pull Request #58 · matplotlib/matplotlib

Conversation

@ivanov

@ivanov ivanov commented Mar 26, 2011

Copy link
Copy Markdown
Member

I added a pull request for this two line change just in case there was a specific reason to exclude test_text from the test modules.

For instance, right now, I get one failure in the test suite if I include it. The failure is in test_text:test_font_styles, but this has been the case for a while, it's just that these tests weren't running before.

Any developers want to chime in on this?

By the way, this was the source of discrepancy between running via nosetests and matplotlib.test() as was described [here] [1]

[1]: http://old.nabble.com/matplotlib.test%28%29-no-errors%2C-but-%24nosetest-matplotlib.tests--%3E-errors-%09and-failure--td29257062.html#a29257062 "here".

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.
All reactions
@ivanov
include test_text in default_test_modules
ac63bbb
@jkseppan

jkseppan commented Mar 26, 2011

Copy link
Copy Markdown
Member

I don't see any reason to have tests in the tree that do not get run. Please merge this.
All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.
@mdboom

mdboom commented Mar 28, 2011

Copy link
Copy Markdown
Member

I'll go ahead and merge this -- on the v1.0.x branch. However, I'm not able to reproduce the failure in test_text:test_font_styles. I'll go ahead and create an issue for that, and maybe Paul can provide more info, pictures etc. over there.
All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.
@mdboom mdboom closed this Mar 28, 2011
@richbwood richbwood mentioned this pull request Dec 19, 2012
test_pcolormesh hangs #1609
Closed
magnunor pushed a commit to magnunor/matplotlib that referenced this pull request Dec 5, 2013
@francisco-dlp
BUG-FIX: in EELSSpectrum the global elements variable was being overw…
0ab8ae0 
…ritten in the add_elements method. Bug reported by Magnus Nord in issue matplotlib#58
fariza added a commit to fariza/matplotlib that referenced this pull request Sep 22, 2016
@fariza
Merge pull request matplotlib#58 from fariza/doc-logo
653f6ac 
Adding logo image
@twmacro twmacro mentioned this pull request Sep 27, 2020
Getting internal "MatplotlibDeprecationWarning: shading='flat' ..." #18595
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Uh oh!

There was an error while loading. Please reload this page.

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Uh oh!

There was an error while loading. Please reload this page.

3 participants

@ivanov @jkseppan @mdboom
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.
You are visitor 01402520 — GeezerVison does not store the pages you look up.
Best viewed with Netscape Navigator 4.7 or Internet Explorer 6 at 800×600. No cookies required.