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

Jump to bottom

E+ 9.6.0: Add autosizedOccupantDiversity for SizingSystem #4446

Closed
joseph-robertson opened this issue Sep 27, 2021 · 1 comment · Fixed by #4450
Closed

E+ 9.6.0: Add autosizedOccupantDiversity for SizingSystem #4446

joseph-robertson opened this issue Sep 27, 2021 · 1 comment · Fixed by #4450
Assignees
@jmarrec
Labels
component - Model Enhancement Request

Comments

@joseph-robertson
Copy link
Collaborator

Enhancement Request

Get autosized occupant diversity value from somewhere in the sql file.

Detailed Description

Possible Implementation
@joseph-robertson joseph-robertson added Enhancement Request Triage Issue needs to be assessed and labeled, further information on reported might be needed labels Sep 27, 2021
@joseph-robertson joseph-robertson mentioned this issue Sep 27, 2021
Merged
19 tasks
@jmarrec jmarrec removed the Triage Issue needs to be assessed and labeled, further information on reported might be needed label Sep 29, 2021
@jmarrec jmarrec self-assigned this Sep 29, 2021
@jmarrec
Copy link
Collaborator

jmarrec commented Sep 29, 2021

I found it in the 62.1Summary report, and it's only there.

I'm slightly confused by the fact that the tabular reports for both heating and cooling, when the E+ IDD only has a single "Occupant Diversity" field. I reached out to the implementer of the E+ PR.

image

jmarrec added a commit that referenced this issue Sep 29, 2021
@jmarrec
Fix #4446 - add SizingSystem::autosizedOccupantDiversity. TODO: why d…
e9b8b68 
…oes it report two values,for heating **and** cooling?
@jmarrec jmarrec mentioned this issue Sep 29, 2021
Merged
19 tasks
tijcolem added a commit that referenced this issue Sep 30, 2021
@tijcolem
Merge pull request #4450 from NREL/4446_autosizedOccupantDiversity
9279637 
Fix #4446 - add SizingSystem::autosizedOccupantDiversity.
# for free to join this conversation on GitHub. Already have an account? # to comment
Assignees

@jmarrec jmarrec

Labels
component - Model Enhancement Request
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Fix #4446 - add SizingSystem::autosizedOccupantDiversity. NREL/OpenStudio
2 participants
@jmarrec @joseph-robertson