ladybug-blender icon indicating copy to clipboard operation
ladybug-blender copied to clipboard

LB tris/quads/ngons?

Open ppaawweeuu opened this issue 5 years ago • 4 comments

Is there limitation to tris and quads? No ngons?

vertices

ppaawweeuu avatar Oct 04 '20 20:10 ppaawweeuu

I see that ngons can cast shadows but can not be receiver/sensor.

tris

ppaawweeuu avatar Oct 04 '20 21:10 ppaawweeuu

Hehehehe I guess that helps answer my question here: https://github.com/ladybug-tools/ladybug-blender/blob/master/ladybug_tools/togeometry.py#L197

If @mostaphaRoudsari / @chriswmackey can clarify when ngons are / aren't allowed, I can update the translator to enforce tessellation at certain points.

Moult avatar Oct 05 '20 01:10 Moult

We only support faces with 3 or 4 vertices in ladybug_geometry. This should not be a major limitation since every ngon mesh face can be broken down into faces with 3 or 4 vertices.

Also, the reason for meshing is to generate the location and direction of the sensors. We use the center of each mesh face (and not the vertices) for the calculation. A good approach is to try to keep the size of the cells close to each other.

mostaphaRoudsari avatar Oct 05 '20 08:10 mostaphaRoudsari

Re-opening, since this is still a bug that can be fixed. Users should feel confident that they can just drop in any object even if it includes ngons, and ideally the node should detect it and tessellate when necessary.

Moult avatar Oct 06 '20 20:10 Moult