Feedback requested for adding choosable polygonizer

[brycecovert]

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.

Right 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.

Is the correct approach to have a polygonizer field on the PolygonComponent?

Thanks, Bryce

[coveralls]

Coverage decreased (-0.008%) to 6.221% when pulling d4baca7c6f5e5b2d5433d309219effb593df9c8c on brycecovert:master into aaf76b6e0563bf7a218473e8b25a2a823313eef2 on UnderwaterApps:master.

[brycecovert]

@azakhary @Sasun any thoughts on this?

[azakhary]

I have mixed thoughts about this.

a) I was thinking that when you make yo polygon, you just make the "outline" (the actual polygon) and it's probably best to store it as is. Because it can be used for many different use cases. Sometimes you want to split it into many polygons, sometimes you want to split it to triangles for rendering, and so on, so it's up to end-user code to process it. And thus, editor should not poligonize anything.

b) But in contrary, currently editor is processing and poligonyzing, just using one algorithm, I was hoping to remove that completely (because see point a.) But on the other hand at this moment if we have one algorythm why not add the other.

I am not sure, I am gravitating that point "a" makes more sense, what you guys think?

[brycecovert]

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.

Would 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.