-
Notifications
You must be signed in to change notification settings - Fork 20
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
Align heating and cooling load profiles with electric load year #472
Merged
Merged
Changes from all commits
Commits
Show all changes
28 commits
Select commit
Hold shift + click to select a range
5511ac2
Improve test description, avoid using Dates.jl in runtests.jl
Bill-Becker 3a620f5
Add leap year test input file
Bill-Becker fdcdd6a
Fix input file path formatting
Bill-Becker fc9208b
Include 2023 and 2024 years for all new tests
Bill-Becker dd13bed
Update year input for HeatingLoad
Bill-Becker 60376db
Add year input for CoolingLoad
Bill-Becker f472f9a
Remove year=2017 requirement for ElectricLoad
Bill-Becker 160ed67
Avoid passing year twice to constructors, make load consistency requi…
Bill-Becker 93fac74
Shift CRB load profiles to match the starting day of the input year, …
Bill-Becker a6c42ca
Fix leap year handling for helper function get_monthly_energy for sim…
Bill-Becker b244565
Update simulated_load() for year input and alignment across loads
Bill-Becker d3b5c60
Add tests for load alignment based on year input
Bill-Becker 64b54a0
Merge branch 'develop' into load-year-align
Bill-Becker 5384b5f
Update changelog for load align fix
Bill-Becker be8197c
small text updates
adfarth 7af1601
Require year input, in particular with ElectricLoad.loads_kw input
Bill-Becker c100eee
Add year input to fix tests with loads_kw
Bill-Becker 9d2ea82
small update to normalize explanation
adfarth 2b10ede
Change CRB profile year alignment strategy
Bill-Becker e5a55e3
Add more updates to Changelog.
Bill-Becker c91743c
Merge branch 'load-year-align' of https://github.com/NREL/REopt.jl in…
Bill-Becker cdcd928
Fix test input for doe_reference_name
Bill-Becker 5f46a78
Update test with get_monthly_energy() now that it's consistent with l…
Bill-Becker d62ef0e
updating CoolingLoad help text
adfarth 5d700be
Update changelog for required year for heating and cooling load profi…
Bill-Becker 74d362d
Make ElectricTariff.year = nothing by default, passed from ElectricLo…
Bill-Becker 375289e
Include industrial_reference_names for process heat default year of 2017
Bill-Becker f67b06c
Merge branch 'load-year-align' of https://github.com/NREL/REopt.jl in…
Bill-Becker 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 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
Oops, something went wrong.
Oops, something went wrong.
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.
Adding a note here to require the year if a custom load profile is uploaded (here and for the heating loads)
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.
Added this requirement. Validation to check if year is not nothing can happen above the check for loads_kw, doe_reference_name, etc, since the default is assigned for CRB without a year input.
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.
@Bill-Becker I think it might be worth adding this to the CHANGELOG too
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.
Added to changelog.