-
-
Notifications
You must be signed in to change notification settings - Fork 7.8k
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 our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
I can't reproduce this here, but it may be something unique about your system. I believe that commit you reference fixes 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? |
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: but later on was overwritten with the commit refered in the first comment. |
I see. Thanks for the additional info. I'll look at the 1.1.x branch and see what I can discover there. |
The fix in b5ce842 was no longer needed following an update to PyCXX. Strangely, I'm still unable to reproduce on 1.1.1 without -NDEBUG:
This sometimes happens when things don't get fully rebuilt after switching branches. |
CXXFLAGS need to be set as well |
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. |
What platform are you on? |
i am on gentoo x86_64, but i did check on ubuntu 12.04 x86_64 to make sure it was not distro specific. CXXFLAGS=-UNDEBUG CFLAGS=-UNDEBUG python setup.py build |
stix_fonts_demo.py fails with bad refcount
Right you are. The missing piece was the matplotlibrc from the With debugging turned on, it looks like those |
1 similar comment
Right you are. The missing piece was the matplotlibrc from the With debugging turned on, it looks like those |
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 commit 85af0c0#src/ft2font.cpp is the culprit