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

mplot3d, addition to losing color on movement of canvas. · Issue #761 · matplotlib/matplotlib · GitHub

Repository navigation

mplot3d, addition to losing color on movement of canvas. #761

Closed

Description

As a already known bug that the plot loses color when you move the axis.
It also tends to lose color when you do a print_figure.

So you can never create a png of the colored plot.

Activity

  1. WeatherGod commented on Mar 13, 2012

    Member

    Technically, this is the same bug. The specific known issue is that the colors are lost upon a redraw. If you want those colors for your saved images as a work-around, don't do a show and then save. Save first through savefig().

  2. sb2nov commented on Mar 13, 2012

    Author

    Thanks , I got it work. One thing which I observed was that the color map is lost only if you put an array to colormap. What is mean is that the color does not disappear when you specify a color like 'r' or something. Even if you have a lot of them.

  3. WeatherGod commented on Mar 13, 2012

    Member

    Correct. The issue is that the collection object tries to recalculate the alpha information to get the depth effects. However, it can't always retain the original colors specified and the colors become gray. Perhaps I can find time today to take another look at this bug.

  4. added a commit that references this issue on Mar 13, 2012
    aa5bdbe
  5. WeatherGod commented on Mar 13, 2012

    Member

    PR #762 should fix this bug. Turns out it was very simple to fix, I just didn't understand ScalarMappables back when I originally investigated this.

  6. sb2nov commented on Mar 13, 2012

    Author

    Thanks a lot, so is the fix been updated or I need to wait for the PR to be confirmed ? I'm new to git so don't know exactly how it functions.

  7. WeatherGod commented on Mar 13, 2012

    Member

    Ideally, you could try out the branch yourself and confirm that it works for your needs (and importantly, doesn't break anything else), and let us know in the PR whether it works or not. The PR acts as a staging area for users and developers to comment and test out any possible changes. When accepted (that's a more ambiguous process for us...), it gets "pulled" into the target branch (in this case v1.1.x) and the issues are automatically closed.

  8. sb2nov commented on Mar 13, 2012

    Author

    I tested out the plot with the changes in the art3d file. It works well but the somehow the image quality doesn't look exactly same between the plots. I am not sure behind the reason for the difference.

  9. WeatherGod commented on Mar 13, 2012

    Member

    It would work best to comment at the Pull Request. Also, I need a more detailed description regarding the change in quality (resolution? aliasing? colors wrong?). Preferrably an image for before and after that I could compare.

  10. sb2nov commented on Mar 13, 2012

    Author

    Images showing the change in the image after fixing up.

    On Tue, Mar 13, 2012 at 4:48 PM, Benjamin Root <
    reply@reply.github.com

    wrote:

    It would work best to comment at the Pull Request. Also, I need a more
    detailed description regarding the change in quality (resolution? aliasing?
    colors wrong?). Preferrably an image for before and after that I could
    compare.


    Reply to this email directly or view it on GitHub:
    #761 (comment)

  11. WeatherGod commented on Sep 20, 2012

    Member

    Finally got some time to look at this again. I originally didn't see the difference, but now I do. What you are seeing is the application of the alpha-indicated depth. In other words, when a marker is farther away from the viewer, it is made more transparent to give it a sense of depth. Before the fix, the alpha value was getting set incorrectly. This is correct behavior.

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

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions


    Back | FazBrowse Home | New Git URL