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 — Mathtext font tests by mdboom · Pull Request #19 · matplotlib/matplotlib

Conversation

@mdboom

@mdboom mdboom commented Feb 23, 2011

Copy link
Copy Markdown
Member

Some of the tables that map characters to STIX font glyphs were out-of-date with the version of the STIX fonts we have. This fixes those tables, and adds a new set of regression tests to check for this kind of thing.

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.
All reactions
ddale
ddale reviewed Feb 23, 2011
View reviewed changes
Comment thread lib/matplotlib/tests/test_mathtext.py Outdated

@ddale ddale Feb 23, 2011

Copy link
Copy Markdown
Contributor

There was a problem hiding this comment.

Choose a reason for hiding this comment

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

Just a nit: would you mind wrapping this line?

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.
All reactions
@ddale

ddale commented Feb 23, 2011

Copy link
Copy Markdown
Contributor

Should this patch be applied to v1.0.x first? If so, I think you can just modify the pull request to point to the right branch.
All reactions

Sorry, something went wrong.

Uh oh!

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

mdboom commented Feb 24, 2011

Copy link
Copy Markdown
Member Author

Yes -- it should be applied on 1.0.x. Reading the github docs, I can't find a way to modify the destination (base) of the pull request after the fact. I will try to create a new one correctly, and assuming that works, close this one.
All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.
@astropy-buildbot
Fix some bugs in the STIX font mappings introduced a long time ago wh…
8768d70 
…en the STIX fonts were updated. Add a number of regression tests to check for this kind of thing.
@richbwood richbwood mentioned this pull request Dec 19, 2012
test_pcolormesh hangs #1609
Closed
fariza added a commit to fariza/matplotlib that referenced this pull request Sep 22, 2016
@fariza
Merge pull request matplotlib#19 from fariza/adding-to-pypi
4b0fc13 
Adding to pypi
swfiua added a commit to swfiua/matplotlib that referenced this pull request Mar 17, 2017
@swfiua
Merge pull request matplotlib#19 from Tigercardell/master
020d8ec 
fix README and remove curio.sleep() in sonogram.run()
tacaswell added a commit that referenced this pull request Mar 20, 2017
@tacaswell
Merge pull request #19 from NelleV/tacaswell_fix_nonascii_afm
9278668 
TST afm._to_str is now tested against utf8-encoded bytes
@wexi wexi mentioned this pull request Oct 25, 2017
matplotlib 2.1.0 "stable" #9578
Closed
@WeatherGod WeatherGod mentioned this pull request Mar 2, 2018
pyplot hangs at plot() #10648
Closed
This pull request was 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

@mdboom @ddale @astropy-buildbot
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 01402503 — 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.