Skip to content
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

Remove path_to_geos heuristics #158

Closed
rhattersley opened this issue Nov 20, 2012 · 3 comments · Fixed by #1162
Closed

Remove path_to_geos heuristics #158

rhattersley opened this issue Nov 20, 2012 · 3 comments · Fixed by #1162

Comments

@rhattersley
Copy link
Member

Ultimately it'd be nicer to remove the heuristic guesswork from path_to_geos by modifying MPL to actually use the CLOSE_POLY vertex code when a path describes a polygon. In the meantime, we can get a chunk of that benefit by removing the heuristics and making GeoAxes.contourf() ensure the resulting paths use CLOSE_POLY.

(Originally raised in #63.)

@cpelley
Copy link

cpelley commented Jun 18, 2014

@rhattersley my head is not in this space, but after a little reading around, I can't see anything obvious that suggests that this has been done yet? Can you confirm this for me.
If so, I suggest we modify the milestone for this issue to 0.12.0? Can you do the honors @rhattersley /@pelson?

@rhattersley
Copy link
Member Author

I can't see anything obvious that suggests that this has been done yet?

Me neither.

@rhattersley rhattersley modified the milestones: 0.12, 0.11 Jun 18, 2014
@cpelley
Copy link

cpelley commented Jun 18, 2014

Thank you for confirming @rhattersley

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

Successfully merging a pull request may close this issue.

4 participants