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

Reference design examples #296

Merged
merged 13 commits into from
Mar 29, 2024
Merged

Conversation

kbrunik
Copy link
Collaborator

@kbrunik kbrunik commented Mar 28, 2024

Creating reference design for green steel and ammonia plants

Adding reference design examples for five locations around the United States. Two onshore examples and three offshore wind examples. Locations and outputs are as follows

  1. Minnesota onshore - steel production example
  2. Texas onshore - ammonia production example
  3. Gulf of Mexico offshore - fixed-bottom hydrogen production example
  4. New York offshore - fixed-bottom hydrogen production example
  5. California offshore - floating hydrogen production example

Related issue

Impacted areas of the software

Additional supporting information

Test results, if applicable

Copy link
Collaborator

@jaredthomas68 jaredthomas68 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think that perhaps your formatter settings need to be addressed, but otherwise looks good. Thanks!

"2064": 84.74628619479452

# Additional parameters passed to ProFAST
financial_assumptions:
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

These should probably all be in the finance parameters, and I think they are duplicative as well. We need to discuss this.

output_level=7,
)

config.hopp_config["technologies"]["wind"]["fin_model"]["system_costs"]["om_fixed"][
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This seems like stuff that we should add to the software, not do in the example. I think it makes sense to have the cost_info override the default fin config for now, but we should do it in the init, not the run script.

@jaredthomas68 jaredthomas68 merged commit 17d1ac9 into NREL:greensteel-eco-sync Mar 29, 2024
4 checks passed
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

Successfully merging this pull request may close these issues.

None yet

3 participants