Skip to content
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

Add EIA ID to Utility #244

Merged
merged 1 commit into from
Sep 25, 2020
Merged

Add EIA ID to Utility #244

merged 1 commit into from
Sep 25, 2020

Conversation

macintoshpie
Copy link
Contributor

@macintoshpie macintoshpie commented Sep 24, 2020

Motivation

It would be nice to have these IDs for identifying utility types: https://openei.org/wiki/List_of_United_States_Utility_Companies_and_Aliases#cite_note-1

@macintoshpie
Copy link
Contributor Author

@corymosiman12 Can you improve the description if needed?

Copy link
Contributor

@markborkum markborkum left a comment

Choose a reason for hiding this comment

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

In the linked webpage, the IDs appear to be xs:nonNegativeInteger, not xs:string, e.g.,

3 Phases Energy Services (ID # 21093)

BuildingSync.xsd Outdated
@@ -3736,6 +3736,11 @@
</xs:restriction>
</xs:simpleType>
</xs:element>
<xs:element name="EIAUtilityID" type="xs:nonNegativeInteger" minOccurs="0">
<xs:annotation>
<xs:documentation>ID of the OpenEI EIA Utility.</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.

"EIA Utility ID as found in EIA-861 and as available in OpenEI"

Copy link
Contributor

@markborkum markborkum left a comment

Choose a reason for hiding this comment

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

Changes look good, thanks.

refactor(schema): change EIAUtilityID type to nonnegative int

doc(schema): update EIAUtilityID docs
@macintoshpie macintoshpie merged commit ee04520 into develop Sep 25, 2020
@macintoshpie macintoshpie deleted the feat/eia-utility-id branch September 25, 2020 16:18
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
Non-breaking Change Schema: General General update to BuildingSync
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants