bifacialvf
bifacialvf copied to clipboard
Change "facing west" criterion to allow small uncertainty in `sazm`
Closes #61
Using the same reproducer as #61 with this branch, the swapping is gone:
Disclaimer: I'm not very familiar at all with bifacialvf and have not evaluated the suitability of this fix beyond making the above plot.
Test failure seems unrelated.
See my comment - I think checking the sign of sine is a more robust solution.