ladybug-tools / ladybug-geometry / 446

93%

master: 93%

push

travis-ci

fix(face): Fix bug in Face3D automatic normal calculation

Man, this one was really deep in there but this should hopefully be the last time that I need to fix this function. There were some rare cases where ladybug_geometry would calculate the face normal in the opposite direction from what it actually was. This was the result of me computing the cross product of non-unit vectors. With the extra unit-ization added by this commit, the normal should be computed correctly for any planar Face3D.

14 of 14 new or added lines in 1 file covered. (100.0%)

4142 of 4452 relevant lines covered (93.04%)

1.86 hits per line