Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Example that fail unicode #1

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

Merged
merged 2 commits into from
Mar 15, 2016
Merged

Conversation

Titan-C
Copy link

@Titan-C Titan-C commented Mar 15, 2016

@Eric89GXL This is my very particular use case that still fails after your great work on unicode(sphinx-gallery#106) if you want to give it a try, my other cases seem to work now on py2, thanks. The example runs without problems in python 2 and 3, is thus valid code. And builds in Sphinx-Gallery python 3 master and this branch but fails on python 2 in both cases. Travis is green because the gallery continues to build despite errors, it will stop to be so once sphinx-gallery#97 is merged.

The issue in this example is very particular to my use case, I have latex in the python code. very particular an \uparrow and for lazyness I use raw strings not to need to escape the backslash. But \u means to python that an unicode character is to be interpreted, that's where the problem comes.

Second I have put a patch to make travis pass the test on the mayavi build, the one that has the issue with pygments. It is a problem of the conda virtual environment and the version of pyface(4.4.0) and some of its dependencies. Previously I forced the update of pyface(4.5.0) within conda, but nowt the versions are checked more strickly within conda and pyface does not update, so the update is forced in pip to pyface(5.0.0) It works well enough for our examples.

Titan-C added 2 commits March 15, 2016 00:43
The conda virtualenv for Mayavi forces a version of pyface that clashes
with sphinx. The manual update of pyface within conda is no longer
enough to update to a new version that does not clash with sphinx. Thus
the update is forced through pip.

Mayavi is an experimentally supported use case of Sphinx-Gallery
@lesteve
Copy link

lesteve commented Mar 15, 2016

cc @Eric89GXL (I am never sure whether opening a PR on someone's fork trigger a notification ...)

@larsoner
Copy link
Owner

Yeah I was notified don't worry, just a bit busy today

larsoner added a commit that referenced this pull request Mar 15, 2016
@larsoner larsoner merged commit 96a2cc7 into larsoner:unicode Mar 15, 2016
@lesteve
Copy link

lesteve commented Mar 15, 2016

Yeah I was notified don't worry, just a bit busy today

No worries, I know I managed to miss a few PR on my fork in the past. Probably due to my github notification settings then ...

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.

3 participants