-
Notifications
You must be signed in to change notification settings - Fork 25
New issue
Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? # to your account
Geometry Shading:Building Bug #242
Comments
Thanks @mldichter , would you be able to check if this issue is fixed in this release preview: https://github.com/openstudiocoalition/OpenStudioApplication/releases/tag/v1.0.1 We fixed #214 which I think was the same issue as this |
Thanks for checking. I looked into it further and agree that there could be a misunderstanding of these objects in the OpenStudio translation from IDF to OSM here: https://github.com/NREL/OpenStudio/blob/e42fde8aa21cf78bc8d961dcbc8480448f50e102/src/energyplus/GeometryTranslator.cpp#L414 |
Looks like the GlobalGeometryRules doesn't translate from IDF to OSM. I imported the IDF into OpenStudio Application v1.0.1 and found these. These are the World coordinates case.
I didn't find an equivalent OSM object for the GlobalGeometryRules IDF object. With Relative coordinates, those four OSM objects are the same. The difference is in the building surfaces. World
Relative
Looks like the building surfaces are being pre-rotated, then the OS:Building North Axis {deg} field is being used to rotate axes and the
control which shading objects are being rotated based on the Shading Surface Type, maybe?... It's getting a little confusing. The OpenStudio Geometry tab doesn't have enough information in the OSM on whether or not to rotate the Shading:Building objects. You could fix the problem by having the IDF import pre-rotate the Shading object vertices based on the GlobalGeometryRules IDF object, but then OpenStudio further ingrains this hybrid of rotating some objects based on the OS:Building fields and relying on pre-rotating other objects on IDF import. |
There is also
which was 0 in the World and Relative cases since the building surface vertices were pre-rotated. I changed the value in the OSM with a text editor and the OSM field does indeed rotate the zone. |
Seems like this is an OpenStudio SDK issue no? @macumber |
OpenStudio purposefully has only one way to define coordinate systems. Allowing multiple definitions in IDF was a choice made to support users when they were entering coordinates one by one in text, it's not as useful anymore. I agree this is an OpenStudio SDK bug in the import of Shading:Building and Shading:Site objects from IDF. Filed here: NREL/OpenStudio#4111 |
…nates are used for Rectangular Surfaces ref openstudiocoalition/OpenStudioApplication#242
Will be fixed in NREL/OpenStudio#4235 |
Issue overview
OpenStudio Application v1.0.0 Geometry tab does not render Shading:Building objects in the correct location. The Shading:Building object has been rotated while the building geometry has not. As stated in the Input Output Reference, this should never be the case.

Current Behavior
Here is a screenshot of the Geometry tab for the below linked IDF.

Expected Behavior
Here is a screenshot of the DXF generated by EnergyPlus 9.3 for the below linked IDF.

Steps to Reproduce
Link to modified Example File
Possible Solution
Work with EnergyPlus developers to verify consistent agreement between source code and documentation across all IDF geometry viewers. Link to EnergyPlus github issue.
Details
Environment
Context
Shading:Building objects displayed in the OpenStudio Geometry tab are inconsistent with the modeled position of Shading:Building objects in EnergyPlus, which has drastically changed results of PV systems referencing the Shading:Building objects.
The text was updated successfully, but these errors were encountered: