primaryDaylightingControl->phiRotationAroundZAxis() may have wrong sign

[jmarrec]

Issue overview

@eringold reported a model where E+ throws (can you post it here maybe?).
He's got two spaces, each with a daylight control with phiRotation = 0.0.

After combine spaces he ends up with a very very tiny value, but one that is negative: -3.1805546814635199E-15

I tracked it down to #3858

primaryDaylightingControl->phiRotationAroundZAxis() may have the wrong sign

See #3858 (comment)

Current:

OpenStudio/src/energyplus/ForwardTranslator/ForwardTranslateThermalZone.cpp

Line 466 in 057a86b

double glareAngle = primaryDaylightingControl->phiRotationAroundZAxis();

Prior to #3858,

OpenStudio/openstudiocore/src/energyplus/ForwardTranslator/ForwardTranslateThermalZone.cpp

Line 478 in 58ed43d

double glareAngle = -openstudio::radToDeg(primaryDaylightingControl->thetaRotationAroundYAxis());

While I think the wrong angle was used and the radToDeg apparently shouldn't be used, the - sign may be needed.

Current Behavior

Expected Behavior

Steps to Reproduce

Possible Solution

Details

Environment

Some additional details about your environment for this issue (if relevant):

• Platform (Operating system, version):
• Version of OpenStudio (if using an intermediate build, include SHA):

Context