Movatterモバイル変換


[0]ホーム

URL:


Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

stix_fonts_demo.py fails with bad refcount#1054

New issue

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

By clicking “Sign up for GitHub”, you agree to ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Merged
mdboom merged 1 commit intomatplotlib:v1.1.xfrommdboom:doc_build_fail
Aug 3, 2012

Conversation

mdboom
Copy link
Member

with the Agg backend, trying to build docs, I get:

python stix_fonts_demo.py

python2.7: CXX/Python2/cxx_extensions.cxx:1320: virtual Py::PythonExtensionBase::~PythonExtensionBase(): Assertion `ob_refcnt == 0' failed.
Aborted (core dumped)

looks like commit85af0c0#src/ft2font.cpp is the culprit

@mdboom
Copy link
Member

I can't reproduce this here, but it may be something unique about your system.

I believe that commit you referencefixes this bug (which I have seen before) rather than creates it. Are you sure you're running master and have rebuilt from scratch (by deleting the "build" directory before rebuilding)? If you're certain of that, have you tried building either of the parent commits of that commit to confirm that it works before and not after?

@sfabbro
Copy link
Author

i can reproduce with matplotlib-1.1.1 and without -DNDEBUG. the git master actually seems to works.

it is confusing, because a fix was committed back in 2010 with:

b5ce842

but later on was overwritten with the commit refered in the first comment.

@mdboom
Copy link
Member

I see. Thanks for the additional info. I'll look at the 1.1.x branch and see what I can discover there.

@mdboom
Copy link
Member

The fix inb5ce842 was no longer needed following an update to PyCXX.

Strangely, I'm still unable to reproduce on 1.1.1 without -NDEBUG:

CFLAGS=-UNDEBUG python setup.py install

This sometimes happens when things don't get fully rebuilt after switching branches.

@sfabbro
Copy link
Author

CXXFLAGS need to be set as well

@mdboom
Copy link
Member

Thanks -- just tried that. Due to a bug in how distutils handles CXXFLAGS, it doesn't actually matter, but for good measure it makes sense. Unfortunately, I'm still not able to reproduce.

@mdboom
Copy link
Member

What platform are you on?

@sfabbro
Copy link
Author

i am on gentoo x86_64, but i did check on ubuntu 12.04 x86_64 to make sure it was not distro specific.
to reproduce from a vanilla matplotlib-1.1.1 tar ball, i do:

CXXFLAGS=-UNDEBUG CFLAGS=-UNDEBUG python setup.py build
cp doc/matplotlibrc .
PYTHONPATH=build/lib.linux-x86_64-2.7 python examples/pylab_examples/stix_fonts_demo.py

mdboom added a commit that referenced this pull requestAug 3, 2012
 stix_fonts_demo.py fails with bad refcount
@mdboommdboom merged commit8089e04 intomatplotlib:v1.1.xAug 3, 2012
@mdboom
Copy link
Member

Right you are. The missing piece was the matplotlibrc from thedoc directory.

With debugging turned on, it looks like thoseob_refcnt-- operations are required on both 1.1.x and master. Thanks for finding this. Without debugging turned on, these are probably silently leaking.

1 similar comment
@mdboom
Copy link
Member

Right you are. The missing piece was the matplotlibrc from thedoc directory.

With debugging turned on, it looks like thoseob_refcnt-- operations are required on both 1.1.x and master. Thanks for finding this. Without debugging turned on, these are probably silently leaking.

Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

2 participants
@mdboom@sfabbro

[8]ページ先頭

©2009-2025 Movatter.jp