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

correct CHP O&M cost per hr per kw bug #310

Merged
merged 1 commit into from
Mar 25, 2022
Merged

correct CHP O&M cost per hr per kw bug #310

merged 1 commit into from
Mar 25, 2022

Conversation

hdunham
Copy link
Collaborator

@hdunham hdunham commented Mar 16, 2022

Please check if the PR fulfills these requirements

  • The commit message follows our guidelines
  • Tests for the changes have been added (for bug fixes / features)
  • Docs have been added / updated (for bug fixes / features)

What kind of change does this PR introduce?

(Bug fix, feature, docs update, ...)
bug fix

What is the current behavior?

(You can also link to an open issue here)
Units of CHP om_cost_us_dollars_per_hr_per_kw_rated are incorrectly converted when it's already the correct units

What is the new behavior (if this is a feature change)?

the CHP O&M om_cost_us_dollars_per_hr_per_kw_rated given to the optimization model is in units of dollars per hr per kw

Does this PR introduce a breaking change?

(What changes might users need to make in their application due to this PR?)
no

Other information:

@hdunham hdunham requested a review from adfarth March 16, 2022 21:58
@hdunham hdunham merged commit cf725de into develop Mar 25, 2022
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