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

Update readme for create_bar, create_typical, and SpaceTypeAndConstrutionSetWizard measures #59

Open
DavidGoldwasser opened this issue Feb 11, 2021 · 1 comment
Assignees

Comments

@DavidGoldwasser
Copy link
Collaborator

specifically the DOE versions but maybe others as well. There are new template choices and new building types. Here are links to some of the measures. Use code from new template measure to automatically get valid choices from extension gem so they are not hard coded in this measure. I also want to add a sentence or two explaining the new ComStock choices when using DOE templates. I'm going to hard code these in the measures for now but I'm also going to add to method in extension gem. Then when we update these measures again after 3.2 we can increase the min version and pull this from extension gem.

https://github.com/NREL/openstudio-model-articulation-gem/blob/develop/lib/measures/create_typical_doe_building_from_model/README.md.erb#L37

https://github.com/NREL/openstudio-model-articulation-gem/blob/develop/lib/measures/create_bar_from_doe_building_type_ratios/README.md.erb#L41

https://github.com/NREL/openstudio-model-articulation-gem/tree/develop/lib/measures/SpaceTypeAndConstructionSetWizard

@DavidGoldwasser DavidGoldwasser self-assigned this Feb 11, 2021
@Ski90Moo
Copy link
Contributor

Ski90Moo commented Apr 28, 2023

I think the readme and modeler description for create_typical_doe_building_from_model leaves something to be desired. In some instances I cannot get it to run and I do not know how to troubleshoot because there is no error information.
2023-04-27 (9)

One example I can think of; the measure requires the user to select a climate zone for the field "!- Climate Zone Value 1". I think there is a lot more to preparing the model for using this measure, but it seems the modeler is left to grope in the dark.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants