Skip to content

xkcd plots rendered as regular plots on Mac OS X #5442

Closed

Description

@mario-grgic

This is with Python 2.7.10 and matplotlib 1.5.0 on Mac OS X 10.11.1. All xkcd plots render as regular plots.

Activity

  1. DanHickstein commented on Nov 10, 2015

    @DanHickstein
    Contributor

    Does switching backends help?
    #2269

  2. mario-grgic commented on Nov 10, 2015

    @mario-grgic
    Author

    Yes, if I save the plot as pdf or use CocoaAgg backend, it looks right. It looks like the default MacOSX backend has a problem.

  3. tacaswell commented on Nov 11, 2015

    @tacaswell
    Member

    @mario-grgic Is regression from 1.4.3?

    attn @mdehoon

  4. modified the milestones: , on Nov 11, 2015
  5. jenshnielsen commented on Nov 11, 2015

    @jenshnielsen
    Member

    It's not a regression. It doesn't work with the OSX backend in 1.4.3 either.

  6. mario-grgic commented on Nov 11, 2015

    @mario-grgic
    Author

    In 1.4.3 xkcd plots failed with error described in Issue #5049.

  7. mdehoon commented on Nov 12, 2015

    @mdehoon
    Contributor

    With matplotlib 1.5.0 using the gtkcairo backend, the xkcd plots also render as regular plots. @mario-grgic can you identify exactly where in the xkcd code the difference between agg/pdf and macosx/gtkcairo occurs?

  8. mdboom commented on Nov 12, 2015

    @mdboom
    Member

    The "sketched line" functionality happens in the path iterator code (src/path_converters.h). The macosx backend interacts with that code through a C wrapper around the C++ (src/path_cleanup.cpp) that hasn't been updated to expose the sketch functionality. We'd need to both update that wrapper and then update the macosx backend to pass along the sketch parameters from the gc to that. I can probably hack that together when I have time.

    The Cairo backend is different -- since it's currently 100% Python (outside of the third-party Cairo wrapper), it doesn't use the path iterator code. Doing so would require a lot more overhead, but it's doable, and we do actually already expose the path iterators to Python through _path.cleanup_path.

  9. mdehoon commented on Nov 13, 2015

    @mdehoon
    Contributor

    @mdboom OK, when you're done, I can have have a look at the macosx backend side of things.

  10. added a commit that references this issue on Nov 23, 2015
    2ae1520
  11. assigned and unassigned on Nov 23, 2015
  12. mdboom commented on Mar 21, 2016

    @mdboom
    Member

    Fixed by #6178.

  13. modified the milestones: , on Mar 28, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

Type

No type

Projects

No projects

    Milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions