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

Simple heat consumer implementation #599

Merged
merged 18 commits into from
Apr 9, 2024

Conversation

dlohmeier
Copy link
Collaborator

This first implementation includes a model that combines the two components heat_exchanger and flow_control. A user can set the desired extracted heat flux and the mass flow through this consumer that can thereby directly connect flow and return system of a district heating network.

…hermal properties of a consumer (mass flow, heat extraction, temperature difference and return temperature)

Note: the model currently only works with set mass flow and heat extraction, as the other modes require a more dedicated simulation wrt. the update of fluid properties between heat transfer iterations and a feedback of heat transfer results to hydraulics
# Conflicts:
#	src/pandapipes/component_models/heat_consumer_component.py
#	src/pandapipes/test/api/test_components/test_heat_consumer.py
Copy link

codecov bot commented Mar 18, 2024

Codecov Report

Attention: Patch coverage is 97.34513% with 3 lines in your changes are missing coverage. Please review.

Project coverage is 86.72%. Comparing base (9385283) to head (181abb6).
Report is 1 commits behind head on develop.

❗ Current head 181abb6 differs from pull request most recent head a347b82. Consider uploading reports for the commit a347b82 to get more accurate results

Files Patch % Lines
src/pandapipes/create.py 93.75% 2 Missing ⚠️
...apipes/component_models/heat_consumer_component.py 98.76% 1 Missing ⚠️
Additional details and impacted files
@@             Coverage Diff             @@
##           develop     #599      +/-   ##
===========================================
- Coverage    90.49%   86.72%   -3.78%     
===========================================
  Files          139       89      -50     
  Lines        10549     6168    -4381     
===========================================
- Hits          9546     5349    -4197     
+ Misses        1003      819     -184     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

EPrade
EPrade previously approved these changes Mar 25, 2024
- shifted some branch initialization to branch_wzerolength_models.py (also for heat_exchanger_component.py)
- switched order of arguments in component (qext and mdot)
@SimonRubenDrauz SimonRubenDrauz merged commit e18dfa6 into e2nIEE:develop Apr 9, 2024
20 of 21 checks passed
@dlohmeier dlohmeier deleted the feature/heat_consumer branch April 11, 2024 06:00
@dlohmeier dlohmeier restored the feature/heat_consumer branch April 11, 2024 06:01
@dlohmeier dlohmeier deleted the feature/heat_consumer branch April 11, 2024 06:02
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