-
Notifications
You must be signed in to change notification settings - Fork 58
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 Weather module #1650
Merged
Merged
Refactor Weather module #1650
Changes from 1 commit
Commits
Show all changes
59 commits
Select commit
Hold shift + click to select a range
fe46765
add epw class
eringold 59a46de
add StatFile class
eringold 2706dfb
extend and refactor StatFile
eringold 752540f
harmonize Epw.to_json
eringold feb7f5a
Harmonize StatFile.load
eringold 1f7cd1c
add get epw path and representative cz weather file to information
eringold a6a1a2f
refactor and replicate changebuildinglocation in Weather.modify
eringold c8e4431
initial replace model_get_weather_file
eringold d37ec2f
test stat
eringold f9501fd
misc weather module corrections
eringold 1953c67
test weather information
eringold 25cd12a
test weather modify
eringold a107442
clean up test
eringold b6df36c
Merge remote-tracking branch 'origin/master' into refactor/weather_mo…
eringold 3d4b7af
yard and rubocop fixes
eringold 1763454
correct weather file path call
eringold 6592121
Merge branch 'master' of https://github.com/NREL/openstudio-standards…
eringold 33f9102
Merge branch 'master' of https://github.com/NREL/openstudio-standards…
eringold fa62912
Merge branch 'master' into refactor/weather_module
eringold 338f4c7
Merge branch 'refactor/weather_module' of https://github.com/NREL/ope…
eringold 8006491
delete Weather.stat_file.rb
mdahlhausen e0572db
remove redundant methods from Weather.Model.rb
mdahlhausen f067729
weather method replacements
mdahlhausen 6a54fd6
Merge branch 'master' into refactor/weather_module
mdahlhausen dfb06d5
replace EnergyPlus::StateFile.new
mdahlhausen 6d956f9
add set undisturbed ground temperature methods
mdahlhausen d7bab9b
rubocop edits
mdahlhausen d68fe17
extend weather module methods
mdahlhausen cf84aa8
delete California weather files
mdahlhausen 888e469
weather module yard doc edits
mdahlhausen 241d8e6
Change stat_file logs to warn instead of error
mdahlhausen 50967e7
replace model_add_ground_temperatures
mdahlhausen 0fa3814
Merge branch 'refactor/weather_module' of https://github.com/NREL/ope…
eringold 74ca35f
weather modify changes, design day defaults
mdahlhausen 1b5311a
split and add weather tests
mdahlhausen 7db5211
replace model_add_design_days_and_weather_file
mdahlhausen 0405af9
remove remaining instances of model_add_design_days_and_weather_file
mdahlhausen 7493411
refactor weather method calls in necb tests
mdahlhausen 6f967ff
update regression tests
mdahlhausen 6422c1e
remove Weather.Model.rb
mdahlhausen b0f8523
update nil ddy_list test result
eringold 12935ac
calculate design day average global irradiance
mdahlhausen a433fcd
improve ddy regex lookup
eringold 2d29716
update all heating+cooling ddy test
eringold dbb4d36
remove Weather.Model.rb
mdahlhausen 3905cf9
fix to hdd lookup
mdahlhausen 11b745b
add annual cooling wetbulb design day to necb regression models
mdahlhausen e994cc3
Merge branch 'refactor/weather_module' of https://github.com/NREL/ope…
eringold 374d26e
ddy docstrings
eringold 7cfcbf6
fix undefined error
mdahlhausen 3b89ed2
update performance results
mdahlhausen 1f612c4
Fixing typo when using weather_file_path in btap_data.rb.
ckirney 96b1ebb
fix additional typos
mdahlhausen 47012b8
update module weather tests
mdahlhausen 70a51d8
update NECB building regression tests
mdahlhausen 22e971a
update DOE regression models
mdahlhausen 7989168
Merge branch 'master' into refactor/weather_module
mdahlhausen c818dc1
update warehouse regression models
mdahlhausen 57a4290
fix param name missing for yard doc
mdahlhausen File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file was deleted.
Oops, something went wrong.
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I tried to find any sources for it; the only thing I found is that Graham Wright says in his paper that “DDD was calculated by thresholding the monthly humidity data at a humidity ratio of 0.010”. To me it sounded like they considered monthly average, so I used it as daily humidity ratio…
I tested for Montreal, the two methods give 0.76 using your method and 0.15 using my method… I’m not sure which one would make more sense for what PHIUS is asking for… we can use the same method as heating/cooling degree days as you suggested
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
If Graham is using monthly humidity averages, then both of our methods are wrong. It also wouldn't be "degree-days" in the normal meaning of the term. In the standard calculation of heating degree days for example, it's often the case that the sum of monthly heating degree days at 18C are non-zero when the monthly average temperature is above 18C.
I think the best way to resolve this is to write up and implement a proposal to PHIUS to implement a better and clearer regression. But I don't have time for that. Until then I suggest we settle with the methods we have that at least are clear and somewhat defensible.