Skip to content

Address Polygon Rasterizer Issues - #1963

Merged
lossyrob merged 5 commits into
locationtech:masterfrom
jamesmcclain:fix/polygon-rasterizer
Jan 26, 2017
Merged

lossyrob merged 5 commits into
locationtech:masterfrom
jamesmcclain:fix/polygon-rasterizer

Conversation

@jamesmcclain

@jamesmcclain jamesmcclain commented Jan 15, 2017 •

Copy link
Copy Markdown
Member

Still Needs

  • Proper handling of horizontal segments for point-pixels
  • More test cases
  • Solidification
  • Feedback from gentleman who reported bug?

Fixes #1962

@jamesmcclain
jamesmcclain force-pushed the fix/polygon-rasterizer branch from 5654af1 to 22e3936 Compare January 15, 2017 21:43
@jamesmcclain
jamesmcclain force-pushed the fix/polygon-rasterizer branch from 22e3936 to 0550238 Compare January 17, 2017 18:27
@jamesmcclain

jamesmcclain commented Jan 17, 2017 •

Copy link
Copy Markdown
Member Author

@quertenmont This is not completely finished, but it may be complete enough to consider testing. The polygons that you sent should work now. This should be complete.

@jamesmcclain jamesmcclain changed the title [WIP] Address Polygon Rasterizer Issues Address Polygon Rasterizer Issues Jan 18, 2017
@quertenmont

Copy link
Copy Markdown

I'll try to give it a try in the coming days,
Thanks for your fast fix!

@jamesmcclain

Copy link
Copy Markdown
Member Author

You are welcome, sir. Thank you for the bug report and test cases!

@lossyrob
lossyrob merged commit ca31aa0 into locationtech:master Jan 26, 2017
@jamesmcclain
jamesmcclain deleted the fix/polygon-rasterizer branch January 26, 2017 16:55
@lossyrob lossyrob added this to the 1.1 milestone Mar 12, 2017
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.

3 participants