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

Refactor auc:TimeSeries for more concise modeling #236

Closed
macintoshpie opened this issue Sep 9, 2020 · 3 comments
Closed

Refactor auc:TimeSeries for more concise modeling #236

macintoshpie opened this issue Sep 9, 2020 · 3 comments
Assignees
Labels
Breaking Change ignore Use this label to ignore this item in the changelog.

Comments

@macintoshpie
Copy link
Contributor

As brought up in the 2020 proposal Add Associated Cost to TimeSeries Option 2, we can more concisely model time series data by grouping reading types within a single auc:TimeSeries element.

This issue is a suggestion to revisit this design when creating version 3 of the schema.

For example (taken from the proposal)

<auc:Scenario>
    <auc:TimeSeriesData>
        <auc:TimeSeries>
            <!-- Add this element -->
            <auc:IntervalReading>
                <auc:Total>123</auc:Total>
                <auc:Peak>123</auc:Peak>
                <auc:Cost>1000</auc:Cost>
            </auc:IntervalReading>
            <auc:StartTimestamp>2019-01-01T00:00:00</auc:StartTimestamp>
            <auc:EndTimestamp>2019-02-01T00:00:00</auc:EndTimestamp>
            <auc:IntervalFrequency>Month</auc:IntervalFrequency>
            <auc:ResourceUseID IDref="ResourceUse-Electricity" />
        </auc:TimeSeries>
    </auc:TimeSeriesData>
</auc:Scenario>
@macintoshpie macintoshpie added this to To do in BuildingSync v3.0 via automation Sep 9, 2020
@nllong nllong removed this from To do in BuildingSync v3.0 Sep 11, 2020
@nllong nllong moved this from To Do to To Do (Schema) in BuildingSync 3.0 and Related Feb 2, 2021
@nllong nllong moved this from To Do (Schema) to FY21-Q3-BSyncV3 in BuildingSync 3.0 and Related Mar 25, 2021
@macintoshpie
Copy link
Contributor Author

@nllong Is this something we still want to add to v3? Things to consider:

  • AT import (is their support for v3 required?)
  • SEED import (not too heavy of a lift for us)
  • Existing schematron which reference meter readings (211, SEED, ...)
  • How do other folks model TS data for multiple data types (reading, cost, etc) -- e.g. PM associates all readings to a single interval

@nllong
Copy link
Member

nllong commented Jul 6, 2021

We are going to table this until there is a more specific need.

@nllong nllong closed this as completed Jul 6, 2021
BuildingSync 3.0 and Related automation moved this from FY21-Q3-BSyncV3 to Done Jul 6, 2021
@nllong nllong added the ignore Use this label to ignore this item in the changelog. label Jul 6, 2021
@macintoshpie
Copy link
Contributor Author

I think Audit Template's method for representing multiple time series data for a resource use could benefit from this change. E.g. here's a row representing meter data in AT which maps well:
Screen Shot 2021-08-25 at 10 59 36 AM

We'd have to update the proposal to handle units for each reading type

@markborkum thoughts in the feasibility of this change for audit template?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Breaking Change ignore Use this label to ignore this item in the changelog.
Development

No branches or pull requests

3 participants