-
Notifications
You must be signed in to change notification settings - Fork 34
Remove system-level outputs from storage and replace with demand component (follow-on to 631) #666
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
Merged
elenya-grant
merged 71 commits into
NatLabRockies:develop
from
elenya-grant:storage/standalone_outputs
Apr 15, 2026
Merged
Changes from all commits
Commits
Show all changes
71 commits
Select commit
Hold shift + click to select a range
1b9f88d
renamed outputs in converter openloop controllers
elenya-grant 2b3d0c1
updated tests and examples with updated naming
elenya-grant 2af6329
Merge remote-tracking branch 'h2i_upstream/develop' into dispatch/con…
elenya-grant c59a20e
Merge branch 'develop' into dispatch/converter_ol_sync
kbrunik eab6d09
added performance model outputs to the converter control strategies
elenya-grant 1e54ed2
udpated plant configs in test_openloop_controllers.py
elenya-grant 1852db4
Merge branch 'develop' into dispatch/converter_ol_sync
johnjasa b1b62c9
moved converter control strategies to separate file
elenya-grant a97292c
Merge branch 'dispatch/converter_ol_sync' of github.com:elenya-grant/…
elenya-grant 2d6b0c5
moved converter control strategies to demand folder
elenya-grant 0e6b3e8
updated import paths for demand components
elenya-grant 2986dff
removed commodity_set_point as output
elenya-grant 1f71c9d
Merge remote-tracking branch 'h2i_upstream/develop' into dispatch/con…
elenya-grant 9672ecc
renamed demand files
elenya-grant 7ad7b59
renamed demand classes and configs
elenya-grant 9ace0ad
Merge branch 'develop' into dispatch/converter_ol_sync
kbrunik 8444cdf
renamed demand component to be performance model instead of control s…
elenya-grant 2093b81
moved output calculations to shared method in baseclass
elenya-grant 1510bd2
Merge branch 'dispatch/converter_ol_sync' of github.com:elenya-grant/…
elenya-grant d116ad8
removed the word controller from docstrings
elenya-grant 7c5b9e5
draft update to docs
elenya-grant 1046419
removed system level calcs from storage and updated most tests
elenya-grant 65dc07b
updated optimal controller tests
elenya-grant 89a4ced
updated example 1 and 2
elenya-grant 5032e41
updated examples 1, 2, and 12
elenya-grant 24cb283
updated examples with storage but dont use storage as a commodity_str…
elenya-grant 92d9854
updated examples with storage defined as a commodity_stream for a fin…
elenya-grant 3001732
tried to fix doc page
elenya-grant 572a020
fixed test failing in sql_to_csv because of additional outputs
elenya-grant 2bd17d8
updated example 12 so tests pass
elenya-grant c71ba77
Merge branch 'develop' into storage/standalone_outputs
johnjasa 7596e27
updated framework tests that used example 1
elenya-grant f47966d
Merge remote-tracking branch 'upstream/develop' into HEAD
kbrunik 9c8b7c1
made demand an input only if using feedback control
elenya-grant 90e56be
merged in develop
elenya-grant 8988ae8
undid change for demand only input for feedback controllers
elenya-grant 140c368
Merge remote-tracking branch 'origin/storage/standalone_outputs' into…
elenya-grant 7e3b3f3
updated demand docs with figures
elenya-grant d9ad39e
added time_step_bounds to demand components
elenya-grant a0202a7
added small bit about demand component to controller_demostrations.md
elenya-grant 8094ee8
updated plotting part of controller demo doc page
elenya-grant 738173c
added test for example 23
elenya-grant eaa17e8
added example for different demand between storage and demand component
elenya-grant 75fa5f5
added subtests for example 13
elenya-grant a01b5af
minor update to controller_demonstrations doc page to use EXAMPLE_DIR
elenya-grant 5a510aa
added wind resource file for example 23
elenya-grant 21082b7
added demo for the demand components using example 13
elenya-grant 9ecd357
fixed demand_demo.md
elenya-grant 6ebd865
made it so demand is only input to battery performance model if using…
elenya-grant ed710a5
typo fix in example 13 plant config
elenya-grant 46116b6
Merge branch 'develop' into storage/standalone_outputs
johnjasa 0c9c665
minor changes to docs and some logic based on feedback
elenya-grant 6308329
Merge branch 'storage/standalone_outputs' of github.com:elenya-grant/…
elenya-grant 6720139
added battery performance plots to demand demo
elenya-grant 34f5a8c
added subtests for example and minor updates based on feedback
elenya-grant 41a177b
added xdsm to demand_demo doc page
elenya-grant 864b124
added standard_capacity_factor as output from storage
elenya-grant ff1b98c
minor updates to example 13 subtests
elenya-grant 62347b5
make some subtests more loose because of floating point precision dif…
elenya-grant 98047ec
added doc page for storage performance models
elenya-grant 0704c24
added subtests for capacity factor calcs and minor update to docs
elenya-grant 15e7573
minor update to example tests
elenya-grant d7694f3
docs
kbrunik ec95ba5
docs fix
kbrunik 6254ee6
no api call but tests fail
kbrunik 7a96ec6
updated example so using pre-downloaded resource data
elenya-grant 231f1d9
updated changelog
elenya-grant 5b8e4dc
updated doc page
elenya-grant 4a228fa
added dt to doc eqn
elenya-grant 226bc74
minor update to cf doc page
elenya-grant 2329b36
added underscore t to numerator in doc page
elenya-grant 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 hidden or 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 hidden or 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 hidden or 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
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -15,4 +15,5 @@ | |
| storage | ||
| transporters | ||
| tools | ||
| demand | ||
| ``` | ||
This file contains hidden or 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
|
Collaborator
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. I think these figures really help illustrate what the calculations are doing :) thank you! |
This file contains hidden or 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.
I was looking at the docs for this and I think it could make sense to update example 14 and the outputs in this demonstration to show how the demand to the
h2_storageand theh2_load_demandcan be different and how that impacts the analysis. I would also recommend updating thetech_configexecutable block because it doesn't include theh2_load_demandand I think that would be helpful to showcase.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.
That's a good idea - I will work on that!
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 think that example 1 or 2 may be a better candidate. Those ones have a battery and we could set the battery demand profile to 10% of the electrolyzer rated capacity and then add/set an electricity component demand profile equal to the electrolyzer rated capacity. There is nothing that hydrogen is going to in example 14 which makes it hard to come up with a storage why the storage demand profile would be different than the demand component demand profile.
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 made a new example (example 13) and used that to make a demand_demo.md page in the
demanddocs folder.