UnownHash / Fletchling

10 stars 3 forks source link

Option to Merge/Ignore Polygons Encased Entirely by Another #11

Closed TurtIeSocks closed 6 months ago

TurtIeSocks commented 7 months ago
Screenshot 2024-02-09 at 7 34 47 AM

Most of these inner Polygons are not "real" Nests and don't need to be included.

lenisko commented 7 months ago

Yeah, I think it would be nice to perform a cleanup on fences by excluding a smaller ones with center point inside other fence. Maybe at the end, to ensure previous cleanup (spawnpoints, area) was run.

2024-02-10_00-45-19-firefox

comstud commented 7 months ago

Yup this is on my list and forgot file issue. Thnx! I think this is maybe the only thing missing for feature parity.

comstud commented 7 months ago

Yeah, I think it would be nice to perform a cleanup on fences by excluding a smaller ones with center point inside other fence. Maybe at the end, to ensure previous cleanup (spawnpoints, area) was run.

I have spawnpoint query (if needed) skipped if other things (area) fail first. A super large area... you don't really want to query spawnpoints for. So I'll prob keep the logic I have: first fail skips rest of checks.