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

Issue3538 air2 air recovery #4088

Conversation

SenHuang19
Copy link

This is to address #3554 (review)

@SenHuang19
Copy link
Author

@JayHuLBL can you please review this PR?

@JayHuLBL
Copy link
Contributor

JayHuLBL commented Jan 9, 2025

@SenHuang19
How did you address the comment: The examples use sta_an and/or sta_b, but these are not defined at zero flow rate. No example should use these quantities. (We removed them in other models due to verification issues across tools.)?

@SenHuang19
Copy link
Author

@SenHuang19 How did you address the comment: The examples use sta_an and/or sta_b, but these are not defined at zero flow rate. No example should use these quantities. (We removed them in other models due to verification issues across tools.)?

@JayHuLBL for some examples (e.g., Buildings.Fluid.HeatExchangers.ThermalWheels.Latent.BaseClasses.Validation.HeatExchangerWithInputEffectiveness), I replace the sta_a/sta_b related quantities with sensors (e.g., replacing hex.sta_a1.T with senExhTemIn.T).
For the rest examples (e.g., Buildings.Fluid.HeatExchangers.ThermalWheels.Latent.Examples.SpaceCooling), I deleted the sta_a/sta_b related quantities (e.g., cooCoi.sta_a2.T) as they are not relevant to the thermal wheel.

@@ -86,12 +119,12 @@ corrections versus wheel speed ratio are correction factors that are multiplied
with the heat exchange effectiveness that the wheel has a full rotational speed.
</li>
<li>
When <code>haveLatentHeatExchange = true</code>,
When <code>haveLatentHeatExchange=true</code>,
Copy link
Contributor

Choose a reason for hiding this comment

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

Has the flag been used to enable/disable parameters? I didn't see it.

Copy link
Author

Choose a reason for hiding this comment

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

no, it was not used. I fixed this issue.

@@ -1,37 +1,71 @@
within Buildings.Fluid.HeatExchangers.ThermalWheels.Data;
record Generic "Generic data record for variable-speed wheels"
record Generic "Generic data record for thermal wheels"
Copy link
Contributor

Choose a reason for hiding this comment

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

In the class Buildings.Fluid.HeatExchangers.ThermalWheels.Data.ASHRAE, please final haveVariableSpeed=true, as below. I believe the ASHRAE is only for the wheels with variable speed drive, right?

record ASHRAE = Buildings.Fluid.HeatExchangers.ThermalWheels.Data.Generic(
  final haveVariableSpeed=true,
  senHeatExchangeEffectiveness(

With the change, when instantiating it, like in Buildings.Fluid.HeatExchangers.ThermalWheels.Latent.Validation.SpeedControlled, uses will not be able to change the value.

Copy link
Author

Choose a reason for hiding this comment

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

add it as suggested

"Part load (75% of the nominal supply flow rate) sensible heat exchanger effectiveness at the heating mode"
annotation (Dialog(group="Part load effectiveness"));
parameter Buildings.Fluid.HeatExchangers.ThermalWheels.Data.Generic per
"Record with performance data"
Copy link
Contributor

Choose a reason for hiding this comment

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

per(final haveLatentHeatExchange=false)

Copy link
Author

Choose a reason for hiding this comment

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

add it as suggested

"Part load (75% of the nominal supply mass flow rate) latent heat exchanger effectiveness at the heating mode"
annotation (Dialog(group="Part load effectiveness"));

parameter Buildings.Fluid.HeatExchangers.ThermalWheels.Data.Generic per
Copy link
Contributor

Choose a reason for hiding this comment

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

per(final haveLatentHeatExchange=true)

Copy link
Author

Choose a reason for hiding this comment

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

add it as suggested

Copy link
Contributor

@JayHuLBL JayHuLBL left a comment

Choose a reason for hiding this comment

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

@SenHuang19 Please see the inline comments.

@JayHuLBL JayHuLBL merged commit 566a56e into lbl-srg:issue3538_Air2Air_recovery Jan 15, 2025
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