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 pinch point temperature #26

Merged
merged 4 commits into from
Jan 23, 2024
Merged

Conversation

BenWinchester
Copy link
Owner

Description

This pull request is being opened in response to comments in the review of a submission to npj Clean Water. In the review, the reviewer highlighted their desire for the inclusion of a pinch-point temperature. This temperature is the temperature difference between the operating temperatures of the heat pump included and the temperatures that it's operating to. I.E., the temperature difference between the heat pump's condensation temperature and the hot-water temperature as well as the temperature difference between the heat pump's evaporation temperature and the ambient temperature.

This value is determined byt he size of the heat exchangers involved in the operation of the plant, and it should be included to ensure that the model is an accurate representation of how a heat pump would oeprate int he MED plant contexts.

Linked Issues

This pull requets has no linked issues.

Unit tests

This pull request does not modify any tests.

@BenWinchester BenWinchester self-assigned this Jan 22, 2024
@BenWinchester BenWinchester merged commit f4da4e0 into main Jan 23, 2024
2 of 4 checks passed
@@ -146,7 +151,7 @@ def _coefficient_of_performance(

return (
system_efficiency
* condensation_temperature
* (condensation_temperature + pinch_point_temperature_difference)
Copy link
Owner Author

Choose a reason for hiding this comment

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

Addition of pinch-point temperature difference to the COP calculation for the heat-pump performance

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.

1 participant