FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

backport to 2.x fixes for sphinx 1.8? · Issue #13258 · matplotlib/matplotlib · GitHub

Repository navigation

backport to 2.x fixes for sphinx 1.8? #13258

Closed

Description

Hello,
in Debian we just got this bug report, http://bugs.debian.org/918896:

matplotlib2 fails to build with Sphinx 1.8, currently available in
experimental:

  # build the doc
  [...] /usr/bin/make -C doc html
  make[1]: Entering directory '/<<PKGBUILDDIR>>/doc'
  Running Sphinx v1.8.3

  Warning, treated as error:
  while setting up extension matplotlib.sphinxext.mathmpl: directive u'math' is already registered, it will be overridden
  make[1]: *** [Makefile:20: html] Error 2

This was fixed in matplotlib 3.x in this commit, please backport it:

https://github.com/matplotlib/matplotlib/commit/f87e9f46daf42ad7

That commit switches from generating PNG images for math formulas to MathJax
library, which renders the math formulas on client side. To package it for
Debian, please add this line to conf.py:

  mathjax_path = 'file:///usr/share/javascript/mathjax/MathJax.js?config=TeX-AMS-MML_HTMLorMML'

and make your documentation package depend on libjs-mathjax.

Also upstream has noticed that some math renders incorrectly with MathJax,
there is a follow-up commit which fixes it:

https://github.com/matplotlib/matplotlib/commit/a101de79650e077a

i see an issue was already reported and fixed at #12108 but it looks like the fixes where never backported to 2.x - would it be possible to do that?

thanks!

Activity

  1. NelleV commented on Jan 22, 2019

    Member

    Sure thing!

  2. NelleV commented on Jan 22, 2019

    Member

    I believe it already has been. This patch is fully in our branch already.

  3. dstansby commented on Jan 24, 2019

    Member

    I'm going to close this since it looks like it's fixed - feel free to comment or re-open if I'm wrong!

  4. added this to the v2.2.4 milestone on Jan 24, 2019
  5. LocutusOfBorg commented on Jan 30, 2019

    Hello, can we please have a backport against 2.2.x branch?

  6. tacaswell commented on Feb 1, 2019

    Member

    It looks like this got pulled back to 3.0.x and 3.0.0-doc, but not the 2.2.x branches.

    I think we need to backport both #12149 and #11928

  7. tacaswell commented on Feb 1, 2019

    Member

    Tried to do this naively, did not work, do not have time to dig deeper now.

  8. tacaswell commented on Feb 12, 2019

    Member

    Closed by #13420

  9. LocutusOfBorg commented on Feb 12, 2019

    thanks! I will test it in a few minutes

  10. sandrotosi commented on Feb 12, 2019

    ContributorAuthor

    @LocutusOfBorg i'm already working on it, please leave the package maintainer job to the package maintainer

  11. LocutusOfBorg commented on Feb 12, 2019

    @sandrotosi I'll leave happily to you the Debian part :)
    I want to take care of the Ubuntu part, and Ubuntu is going to freeze really soon and matplotlib2 never reached any stable Ubuntu release.

  12. LocutusOfBorg commented on Feb 12, 2019

    (btw without manual intervention the package won't probably be ever autosyncd, because it has been removed from Ubuntu due to sphinx 1.8)

  13. sandrotosi commented on Feb 12, 2019

    ContributorAuthor

    the PR patch (applied directly on top of 2.2.3) works, thanks!

  14. tacaswell commented on Feb 13, 2019

    Member

    There should be a 2.2.4 in the next few days as well.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

      Milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions


      Back | FazBrowse Home | New Git URL