Skip to content

Add TotalCommonConditionedAboveGradeWallArea #306

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

Merged
merged 5 commits into from
Mar 23, 2021

Conversation

JieXiong9119
Copy link
Contributor

See proposal
Implementation in schema added

@@ -796,6 +796,18 @@
</xs:simpleContent>
</xs:complexType>
</xs:element>
<xs:element name="DemisingAboveGradeWallArea" minOccurs="0">
<xs:annotation>
<xs:documentation>Above grade demising wall area. (ft2)</xs:documentation>
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We might want to reverence the ASHRAE standard here. If you search in the schema there are a few places where we do this

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We might want to reverence the ASHRAE standard here. If you search in the schema there are a few places where we do this

Could you point me to those places?

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Here's an example:

<xs:documentation>The annual amount of all the energy the premises consumes onsite, as reported on the utility bills. Calculated as imported energy (Eimp) - exported energy (Eexp) - net increase in stored imported energy (Es) (per ASHRAE 105-2014 Figure 5.6). (kBtu)</xs:documentation>

BuildingSync.xsd Outdated
@@ -796,6 +796,18 @@
</xs:simpleContent>
</xs:complexType>
</xs:element>
<xs:element name="DemisingAboveGradeWallArea" minOccurs="0">
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I propose changing the element name to: TotalCommonConditionedAboveGradeWallArea and the documentation to: The total area of the above grade wall common with other conditioned buildings

</xs:annotation>
<xs:complexType>
<xs:simpleContent>
<xs:extension base="xs:decimal">
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Jie let's also add this as a non negative decimal.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

excellent!

```

## References
https://www.pnnl.gov/main/publications/external/technical_reports/PNNL-25130.pdf
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

For reference, you can use:

Copy link
Contributor

@corymosiman12 corymosiman12 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi Jie - minor requested changes - thanks for putting together!

1. Changed `DemisingAboveGradeWallArea` to `TotalCommonConditionedAboveGradeWallArea`.
2. Clarified definition and description.
<Site>
<Buildings>
<Building ID="G1">
<DemisingAboveGradeWallArea>1000</DemisingAboveGradeWallArea>
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Maybe we should use 1000.0 instead of 1000?
Our XML has such format.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The value is set to nonNegativeDecimal. So this is just a visual thing.

@nllong nllong merged commit 1180792 into develop Mar 23, 2021
@nllong nllong deleted the proposals/demising-above-grade-wall-area branch March 23, 2021 19:45
@JieXiong9119 JieXiong9119 changed the title Proposals/demising above grade wall area Add TotalCommonConditionedAboveGradeWallArea Sep 27, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature Adding new functionality to BuildingSync Non-breaking Change UDFs
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants