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

Update jupyter examples for HOPP 3.1.1 #76

Merged
merged 1 commit into from
Jan 23, 2025

Conversation

jaredthomas68
Copy link
Collaborator

Update jupyter examples for HOPP 3.1.1

This PR provides edits to the financial config files for HOPP used in the Jupyter notebook examples. This changes are required to run with HOPP 3.1.1.

PR Checklist

  • [-] CHANGELOG.md has been updated to describe the changes made in this PR
  • Documentation
    • [-] Docstrings are up-to-date
    • [-] Related docs/ files are up-to-date, or added when necessary
    • [-] Documentation has been rebuilt successfully
    • Examples have been updated
  • Tests pass (If not, and this is expected, please elaborate in the tests section)
  • PR description thoroughly describes the new feature, bug fix, etc.

Related issues

This PR updates the examples to work with HOPP 3.1.1, which essentially means this is a bug fix to PR #58.

Impacted areas of the software

  • examples/inputs/plant/default_fin_config.yaml
  • examples/inputs/plant/greenheart_config_onshore-ammonia.yaml
  • examples/inputs/plant/greenheart_config_onshore-hydrogen.yaml
  • examples/inputs/plant/greenheart_config_onshore-steel.yaml

Additional supporting information

The need for this update was brought up while @dakotaramos was working on #56

Test results, if applicable

Copy link
Collaborator

@dakotaramos dakotaramos left a comment

Choose a reason for hiding this comment

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

Based on passing tests and reviewing the error from the original issue, I believe these changes will correct the errors and is good to merge.

@johnjasa johnjasa merged commit 25baf8f into NREL:develop Jan 23, 2025
3 checks passed
RHammond2 pushed a commit that referenced this pull request Feb 4, 2025
RHammond2 added a commit that referenced this pull request Feb 4, 2025
* Merge patches in `main` into `develop` (#41)

* Enhancement: Update issue and PR templates (#35)

* fix bad merge conflict resolution from ad7d7c8 (#40)

* Add dependency: CoolProp (#42)

* Bugfix: correct LCOE units in OpenMDAO HOPP wrapper (#43)

* Add `pre-commit` workflow for automated linting and formatting (#57)

* Financial update: steel/ammonia plant cost year updated to 2022 and updates to work with HOPP 3.1.1 (#58)

* Feedstock transport costs (carbon, lime, iron ore pellet) (#59)

* add weighted average financial parameter function (#60)

* update MIRR calculation and allow parameter weighting without specifying parameter values for tech with zero-cost technology (i.e. excluded tech) (#73)

* update jupyter examples for HOPP 3.1.1 (#76)

* LCA Integration (#56)

* Create the GH Pages Site and Enable a Documentation CI Pipeline (#78)

* bump version for release

* Revert "bump version for release"

* Add initial new GreenHEART documentation (#72)

* Enhancement: Update issue and PR templates (#35)
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.

3 participants