-
Notifications
You must be signed in to change notification settings - Fork 195
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Support all Shading Control Types in ShadingControl Object #4074
Comments
I found this problem independently while working on #4066, I came to open the issue only to realize you beat me to it, and the issue is well detailed. What I wrote on slack earlier is about the same, see below: The default OpenStudio/src/model/ShadingControl.cpp Lines 130 to 140 in 2aeb103
yet the FT for some obscure reason has a block (which will never be entered) that uses 100 W/m2 OpenStudio/src/energyplus/ForwardTranslator/ForwardTranslateShadingControl.cpp Lines 146 to 153 in 2aeb103
Much Worse, there are like 10 Shading Control Type values that expect a setpoint (or two), and yet you can't set the setpoint for any of them but the OpenStudio/src/model/ShadingControl.cpp Lines 194 to 202 in 2aeb103
example of a ShadingControlType you cannot use:
|
Additionally: all of these fields are problematic:
Shading Control Is Scheduled is set if you assign a Schedule to it so it's redundant, see OpenStudio/src/model/ShadingControl.cpp Lines 189 to 197 in 2aeb103
The rest of them aren't even being used. Edit:: Setpoint2 was implemented already in #4066
|
Fully support all
Shading Control Types
in ShadingControl ObjectThe OpenStudio IDD for the
OS:ShadingControl
object nominally supports a bunch ofShading Control Types
(see IDD excerpt below). These were added in a PR in 2019. This limited implementation does not actually work for all of the control types because of the model and forward translator code for this object, as described below.First, the OpenStudio C++ code for the
.setSetpoint()
method currently only sets theSetpoint
if theShading Control Type = OnIfHighSolarOnWindow
. Similarly, the OpenStudio C++ code for the.setShadingControlType()
method actually resets theSetpoint
field unless theShading Control Type = OnIfHighSolarOnWindow
. Additionally, at least four of the possibleShading Control Type
options require use of theSetpoint 2
field, which is not available in the API and is not forward translated.Because the forward translator code only translates the
Setpoint
field to the IDF if theShading Control Type = OnIfHighSolarOnWindow
, even using the workspaceObject.setString()
workaround can't be applied to this objecct.Possible Implementation
Read through the IDD options and revise the API accordingly.
The text was updated successfully, but these errors were encountered: