Skip to content

Conversation

@HansOlsson
Copy link
Contributor

From 10 K to 1 K to be specific.

The reason is two-fold:

  • It is an order of magnitude too large; compared to what is actually seen.
  • In some configurations it can cause simulation problems.

Basically it should be the initial guess for temperature difference across the wall of the heat-exchanger.
Obviously the size depend on the temperature difference between the pipes and the relatives heat capacity of the medium compared to the heat-exchanger walls, so I think that giving an order of magnitude is fine - as long as it is the right order of magnitude.

If you simulate it in Dymola you get this plot for those temperature differences:

HeatExchangerSimulation

plotExpression(apply(HeatExchangerSimulation[end].HEX.wall.Tb[1]-HeatExchangerSimulation[end].HEX.wall.Ta[1]), false, "Diff 1", 1, unit="K");
plotExpression(apply(HeatExchangerSimulation[end].HEX.wall.Tb[10]-HeatExchangerSimulation[end].HEX.wall.Ta[10]), false, "Diff 10", 1, unit="K");
plotExpression(apply(HeatExchangerSimulation[end].HEX.wall.Tb[20]-HeatExchangerSimulation[end].HEX.wall.Ta[20]), false, "Diff 20", 1, unit="K");

So they start 0.4 K apart, not 10 K, and the maximum difference is 2.5 K (the temperature in the mediums are 32 degC and 82 degC at that point - remember that the pipes go in different directions).

The reason is two-fold:
* It is an order of magnitude too large.
* In some configurations it can cause simulation problems.
@HansOlsson HansOlsson added the L: Fluid Issue addresses Modelica.Fluid (excl. Dissipation) label Sep 15, 2022
@HansOlsson HansOlsson requested a review from casella September 15, 2022 13:22
@HansOlsson
Copy link
Contributor Author

Note that even if it is modifying the same parameter as in #4032 the PRs are independent (since one is modifier and one is declaration) and can be merged separately.

@HansOlsson HansOlsson added the example Issue only addresses example(s) label Sep 15, 2022
@Harisankar-Allimangalath
Copy link
Contributor

@casella can you please review the changes , Thankyou

@Harisankar-Allimangalath Harisankar-Allimangalath merged commit 3bbb172 into modelica:master Dec 12, 2023
@HansOlsson HansOlsson deleted the ReduceDt branch December 13, 2023 08:11
@beutlich beutlich removed the request for review from casella December 13, 2023 21:35
@beutlich beutlich added this to the MSL4.1.0 milestone Dec 13, 2023
@beutlich beutlich changed the title Reduce guess for initial temperature difference across wall. Reduce guess for initial temperature difference across wall Feb 12, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

example Issue only addresses example(s) L: Fluid Issue addresses Modelica.Fluid (excl. Dissipation)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants