-
Notifications
You must be signed in to change notification settings - Fork 27
Removing more of old GreenHEART #384
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
johnjasa
merged 35 commits into
NatLabRockies:develop
from
johnjasa:remove_more_of_old_greenheart
Dec 10, 2025
Merged
Changes from all commits
Commits
Show all changes
35 commits
Select commit
Hold shift + click to select a range
763c530
Removing unneeded files and tests
johnjasa 7b2a6fc
WIP working through steel
johnjasa 7d4f41d
Merging
johnjasa cb5b04c
Removed offshore
johnjasa 70320f8
Deleting more old files
johnjasa 7fbab4a
Removed extra file from merge
johnjasa 6cb074c
Removed simple unused files
johnjasa 2af6cda
Moving basic H2 cost
johnjasa 88affb7
Removed defunct pressure vessel and pem_control_type
johnjasa 71ba00f
Removed old tests
johnjasa 14785ef
Updated test values
johnjasa 35e88fe
moved singlitico model
johnjasa 55666aa
Removing unused electrolyzer files
johnjasa df38966
Added back pem master
johnjasa ac8573b
Moving PEM model
johnjasa 562df9e
Big refactor and removal of hydrogen storage from simulation/technolo…
johnjasa 227126a
Shifting more electrolyzer files
johnjasa ee7f21f
Fixed and noted behavior
johnjasa 531726d
Fixed bug in steel calc config for o2 heat integration
johnjasa dc67bb4
Moving iron files and adding back singlitico test
johnjasa cea8ac8
Fixed iron transport
johnjasa 4552e91
Reverted steel example test values
johnjasa 6dc6b1c
Reverted hydrogen storage test values
johnjasa d9167a8
fixing iron paths
johnjasa e7475f4
Merge branch 'develop' into remove_more_of_old_greenheart
johnjasa e39035f
Updates for PR
johnjasa 056c77b
Merge branch 'develop' of https://github.com/NREL/H2Integrate into re…
johnjasa c0eaf18
Updating based on PR comments
johnjasa 1908b0d
removed h2 pipe code and tests
johnjasa ecc0e55
Added back many more docstrings and comments for the hydrogen storage
johnjasa 04cef5a
fix tests
kbrunik b0d7440
Merge branch 'develop' of https://github.com/NREL/H2Integrate into re…
johnjasa febbf5b
Removing defunct code per PR review
johnjasa 2cbb07c
Minor refactoring based on PR feedback
johnjasa 1e49cbf
Added note on primary commodity electricity tech
johnjasa 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
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 |
|---|---|---|
| @@ -0,0 +1,49 @@ | ||
| # Bulk Hydrogen Storage Cost Model | ||
|
|
||
| ## Storage Types | ||
|
|
||
| H2Integrate models at least three types of bulk hydrogen storage technologies: | ||
|
|
||
| - **Underground Pipe Storage**: Hydrogen stored in underground pipeline networks | ||
| - **Lined Rock Caverns (LRC)**: Hydrogen stored in rock caverns with engineered linings | ||
| - **Salt Caverns**: Hydrogen stored in solution-mined salt caverns | ||
|
|
||
| These storage options provide different cost-capacity relationships suitable for various scales of hydrogen production and distribution. | ||
|
|
||
| ## Cost Correlations | ||
|
|
||
| The bulk hydrogen storage costs are modeled as functions of storage capacity using exponential correlations: | ||
|
|
||
| $$Cost = \exp(a(\ln(m))^2 - b\ln(m) + c)$$ | ||
|
|
||
| where $m$ is the useable amount of H₂ stored in tonnes. | ||
|
|
||
| ## Installed Capital Cost and Lifetime Storage Cost | ||
|
|
||
| The figures below show how storage costs scale with capacity for different storage technologies: | ||
|
|
||
|  | ||
|
|
||
| *Figure 1a: Installed capital cost (\$/kg-H₂) as a function of usable hydrogen storage capacity* | ||
|
|
||
|  | ||
|
|
||
| *Figure 1b: Lifetime storage cost (\$/kg-H₂-stored) as a function of usable hydrogen storage capacity* | ||
|
|
||
| ## Cost Correlation Coefficients | ||
|
|
||
| ### Capital Cost Coefficients (Figure 1a) | ||
|
|
||
| | Storage | a | b | c | | ||
| |--------------------------------|----------|---------|--------| | ||
| | Underground pipe storage | 0.004161 | 0.06036 | 6.4581 | | ||
| | Underground lined rock caverns | 0.095803 | 1.5868 | 10.332 | | ||
| | Underground salt caverns | 0.092548 | 1.6432 | 10.161 | | ||
|
|
||
| ### Annual Cost Coefficients (Figure 1b) | ||
|
|
||
| | Storage | a | b | c | | ||
| |--------------------------------|----------|---------|--------| | ||
| | Underground pipe storage | 0.001559 | 0.03531 | 4.5183 | | ||
| | Underground lined rock caverns | 0.092286 | 1.5565 | 8.4658 | | ||
| | Underground salt caverns | 0.085863 | 1.5574 | 8.1606 | |
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
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
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
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
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
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.
It seems like the control type is pretty important. @elenya-grant has this been handled satisfactorily?