NatLabRockies / NatLabRockies/EnergyPlus
SQL 'Time' table: 'Interval' is empty when reporting frequency is 'Annual'
Nobody has claimed this yet.
- Dominant language
- C++
- Stars
- 1.6k
- Forks
- 490
- Avg merge
- 6d 21h
- Merged PRs (30d)
- 22
Description
Issue overview
Time: "Interval" is empty when reporting frequency is "Annual"
This is 1ZoneUncontrolled, with slight modifications to add the sqlite output and a bunch of output variables (cf https://github.com/NREL/OpenStudio/tree/develop/resources/utilities/SqlFile)
SELECT ReportDataIndex, TimeIndex, ReportDataDictionaryIndex, Value, Month, Day, Hour, Minute, Interval, Name, ReportingFrequency From ReportVariableWithTime
WHERE ReportingFrequency = "Annual" OR ReportingFrequency = "Run Period"
AND Name="Zone Mean Air Temperature"

This isn't new, but still it the case up to 9.3.0 official. It goes back to 8.9.0 (where Annual was added/differentiated from Run Period or something like that)
Details
Some additional details for this issue (if relevant):
- Platform (Operating system, version): All
- Version of EnergyPlus (if using an intermediate build, include SHA): 9.3.0
Checklist
Add to this list or remove from it as applicable. This is a simple templated set of guidelines.
- Defect file added (list location of defect file here): https://github.com/NREL/EnergyPlusDevSupport/blob/3a70b68e1b09708d67b30a65b3d6112092020a6d/DefectFiles/7000s/7939/1ZoneEvapCooler-V9-3-0.idf
- Ticket added to Pivotal for defect (development team task)
- Pull request created (the pull request will have additional tasks related to reviewing changes that fix this defect)
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Start with the defect file DefectFiles/7000s/7939/1ZoneEvapCooler-V9-3-0.idf and reproduce the provided ReportVariableWithTime query against the SQLite output. Trace how Annual and Run Period records populate the Time table's Interval field, using the 1ZoneUncontrolled modifications and SqlFile resources as context. Done means Annual records report the expected Interval value without regressing Run Period output.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- cpp, sqlite
- Domain
- backend, databases
- Issue type
- Bug
- Difficulty
- 4/5
- Estimated time
- 3-5 days
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 35/100