{"action":"created","issue":{"url":"https://api.github.com/repos/UnderwaterApps/overlap2d/issues/399","repository_url":"https://api.github.com/repos/UnderwaterApps/overlap2d","labels_url":"https://api.github.com/repos/UnderwaterApps/overlap2d/issues/399/labels{/name}","comments_url":"https://api.github.com/repos/UnderwaterApps/overlap2d/issues/399/comments","events_url":"https://api.github.com/repos/UnderwaterApps/overlap2d/issues/399/events","html_url":"https://github.com/UnderwaterApps/overlap2d/pull/399","id":153601966,"number":399,"title":"Feedback requested for adding choosable polygonizer","user":{"login":"brycecovert","id":506896,"avatar_url":"https://avatars.githubusercontent.com/u/506896?v=3","gravatar_id":"","url":"https://api.github.com/users/brycecovert","html_url":"https://github.com/brycecovert","followers_url":"https://api.github.com/users/brycecovert/followers","following_url":"https://api.github.com/users/brycecovert/following{/other_user}","gists_url":"https://api.github.com/users/brycecovert/gists{/gist_id}","starred_url":"https://api.github.com/users/brycecovert/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/brycecovert/subscriptions","organizations_url":"https://api.github.com/users/brycecovert/orgs","repos_url":"https://api.github.com/users/brycecovert/repos","events_url":"https://api.github.com/users/brycecovert/events{/privacy}","received_events_url":"https://api.github.com/users/brycecovert/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":4,"created_at":"2016-05-07T16:01:40Z","updated_at":"2016-05-16T21:22:58Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/UnderwaterApps/overlap2d/pulls/399","html_url":"https://github.com/UnderwaterApps/overlap2d/pull/399","diff_url":"https://github.com/UnderwaterApps/overlap2d/pull/399.diff","patch_url":"https://github.com/UnderwaterApps/overlap2d/pull/399.patch"},"body":"I'm working on making it possible to choose which polygonizer algorithm to use. I do this because I'd like to use one of my polygons for navmesh pathfinding, and bayazit gives a much better result than ewjordan.\r\n\r\nRight now, I've got the view component correctly updating the geometry, but I'm not certain about whether or not the PolygonComponent would need to store the polygonizer algorithm as well, in order for it to be persisted when saved and used when moving points/updating the polygon using the PolygonTool.\r\n\r\nIs the correct approach to have a polygonizer field on the PolygonComponent?\r\n\r\nThanks,\r\nBryce"},"comment":{"url":"https://api.github.com/repos/UnderwaterApps/overlap2d/issues/comments/219552281","html_url":"https://github.com/UnderwaterApps/overlap2d/pull/399#issuecomment-219552281","issue_url":"https://api.github.com/repos/UnderwaterApps/overlap2d/issues/399","id":219552281,"user":{"login":"brycecovert","id":506896,"avatar_url":"https://avatars.githubusercontent.com/u/506896?v=3","gravatar_id":"","url":"https://api.github.com/users/brycecovert","html_url":"https://github.com/brycecovert","followers_url":"https://api.github.com/users/brycecovert/followers","following_url":"https://api.github.com/users/brycecovert/following{/other_user}","gists_url":"https://api.github.com/users/brycecovert/gists{/gist_id}","starred_url":"https://api.github.com/users/brycecovert/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/brycecovert/subscriptions","organizations_url":"https://api.github.com/users/brycecovert/orgs","repos_url":"https://api.github.com/users/brycecovert/repos","events_url":"https://api.github.com/users/brycecovert/events{/privacy}","received_events_url":"https://api.github.com/users/brycecovert/received_events","type":"User","site_admin":false},"created_at":"2016-05-16T21:22:58Z","updated_at":"2016-05-16T21:22:58Z","body":"Makes sense. My primary use case for using overlap2d would be the ability to draw polygons for pathfinding, and I'd rather not have to write the convex-polygonization algorithm myself. I can see both sides of that -- on the one hand, maybe the editor shouldn't care how to split up the polygon. On the other, as a developer, having a tool that allows me to see the polygons allows me to visualize the best way to set up pathfinding.\r\n\r\nWould it be crazy to add polygonization as an optional action on a polygon? That is, instead of polygonizing by default, have it be a boolean flag whether to polygonize, and then allow the user to select the method.\r\n\r\n"}}