Skip to content

Bugfix for path collection rendering on agg backends [backport to 1.4.x] #3761

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

Closed
wants to merge 1 commit into from

Conversation

cimarronm
Copy link
Contributor

Attempt to fix issue with drawing collections artist which include vertices that are extremely far apart horizontally while being rendered through the agg backend. This seems to resolve issue #3626 but may not be the proper fix.

has line segments with very large horizontal differences between
vertices
@tacaswell tacaswell modified the milestones: v1.5.x, v1.4.3 Nov 6, 2014
@tacaswell tacaswell changed the title Bugfix for path collection rendering on agg backends Bugfix for path collection rendering on agg backends [backport to 1.4.x] Nov 6, 2014
@mdboom
Copy link
Member

mdboom commented Nov 18, 2014

@efiring: Would you mind having a look at this one? I think you implemented the "chunking" feature that is being removed here.

@efiring
Copy link
Member

efiring commented Nov 20, 2014

@mdboom, It looks to me like this code comes from @pelson, or was last modified by him. It looks like it might be an optimization, going straight to draw_markers instead of drawing a collection when there is only one element. This seems odd, though. I hope the wizard of the rainy isles can illuminate it.

@WeatherGod
Copy link
Member

"wizard of the rainy isles" @efiring, doesn't rain in Hawaii as well? :-P

On Thu, Nov 20, 2014 at 2:38 PM, Eric Firing notifications@github.com
wrote:

@mdboom https://github.com/mdboom, It looks to me like this code comes
from @pelson https://github.com/pelson, or was last modified by him. It
looks like it might be an optimization, going straight to draw_markers
instead of drawing a collection when there is only one element. This seems
odd, though. I hope the wizard of the rainy isles can illuminate it.


Reply to this email directly or view it on GitHub
#3761 (comment)
.

@efiring
Copy link
Member

efiring commented Nov 21, 2014

On 2014/11/20, 3:40 PM, Benjamin Root wrote:

"wizard of the rainy isles" @efiring, doesn't rain in Hawaii as well? :-P

It's not rain, it's liquid sunshine.

@mdboom
Copy link
Member

mdboom commented Nov 21, 2014

@efiring: Sorry -- I thought I was looking at something else. I'll dig further...

@mdboom
Copy link
Member

mdboom commented Nov 21, 2014

@cimarronm: Thanks for the investigating this so well -- it really helped to pinpoint a solution. I've created #3826 which, rather than turning the optimization off altogether, ensures that the path is small enough before performing the optimization.

@mdboom mdboom closed this Nov 21, 2014
@pelson
Copy link
Member

pelson commented Nov 21, 2014

I hope the wizard of the rainy isles can illuminate it.

It's not rain, it's liquid sunshine.

😆 - we don't get rain here - its mostly aerified ocean. And sometimes we can even stay dry.

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.

7 participants