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

Add getem_costs_eqn_calc and test #182

Merged
merged 3 commits into from
Jan 13, 2025
Merged

Add getem_costs_eqn_calc and test #182

merged 3 commits into from
Jan 13, 2025

Conversation

dguittet
Copy link
Collaborator

@dguittet dguittet commented Jan 3, 2025

Added src/GeothermalCosts_eqns.c

@mjprilliman The documentation, getem_om_cost_calc_doc, in the above file needs to be updated. This is what is displayed to the user. Note that the variable list here is different than in ssc/ssc/cmod_geothermal_costs_eqns.h because there are 5 input variables that are already within cmod_geothermal_costs. So instead of asking the user to input these, they are taken directly from the ssc_data of the GeothermalCosts module.

Example of how the equation works is in the test file below.

@dguittet dguittet requested a review from sjanzou January 3, 2025 00:44
Copy link
Collaborator

@sjanzou sjanzou left a comment

Choose a reason for hiding this comment

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

Awesome - working for me in local build of pysam. Thanks for all the pull requests and the steps to add an equation!

@sjanzou sjanzou merged commit 42af6de into GETEM Jan 13, 2025
0 of 20 checks passed
@sjanzou sjanzou deleted the getem_costs_eqn branch January 13, 2025 08:49
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.

2 participants