ibpsa / modelica-ibpsa

Modelica library for building and district energy systems developed within IBPSA Project 1
https://ibpsa.github.io/project1
145 stars 84 forks source link

condensing cooling coil model #1109

Closed Mathadon closed 2 years ago

Mathadon commented 5 years ago

We are working on a cooling coil model that supports condensation, without using discretisations.

thorade commented 5 years ago

Would that be a moving boundary model, with 3 segments, or a different approach?

Mathadon commented 5 years ago

No, rather a simple model with average specific enthalpy between inlet and outlet and the e-NTU method. PR soon!

Mathadon commented 5 years ago

1113 contains a first version of the model

Many things still need to happen, but the main issue right now is to choose a parameterization for the computation of UA_nominal. At the Aachen meeting I showed that there was an issue when cross-validating the cooling coil model. This is caused by the fact that the current parameterization of the heating coil model implicitly assumes that the nominal conditions (Q_flow_nominal, m_flow_nominal, etc) are at fully dry conditions. Based on these conditions the UA value of the heat exchanger is computed inconsistently, which causes the observed errors. The question is now how to include the degree of condensation at nominal conditions into the model parameters. Options include parameters for:

Depending on the choice, we may have to do (many) inverse computations of the normal equations to compute UA. If wrong parameters values are supplied, the resulting inverse computations may give infeasible results. From that point of view option 3) or 4) look most dummy-proof, however 1) and 2) are easier to use. @mwetter @icupeiro Any suggestions here?

edit: I tried using Q_sen_nominal as a parameter and to rescale cp_nominal accordingly, but that does not lead to Q_flow_nominal at the nominal conditions. Commit a186432 implements code where the nominal absolute inlet humidity is a parameter of the model. This parameter allows to consistently compute the inverse of the model equations such that UA is computed correctly. The question is. though whether users will have access to this parameter. An alternative would be to allow users to provide the wet bulb temperature instead. One more issue: based on the model, UA is significantly larger in the wet regime than in the dry regime (270 W/K instead of 200 W/K). This will lead to an overestimation of the heat transfer in the dry regime based on the wet regime nominal conditions. Is there an alternative to IBPSA.Fluid.HeatExchangers.BaseClasses.HADryCoil that we can use to overcome this?

icupeiro commented 5 years ago

@Mathadon the manufacturer information we are using provides absolute values of latent and sensible heat flow rate, so I'd stick to this

Mathadon commented 5 years ago

@icupeiro indeed that makes sense. The question is also how to specify the 'nominal conditions'. For a dry coil this is just the nominal flow rate and temperature difference but for a wet coil the model also requires some indication of the model humidity. This can be provided either using:

Right now I'm using absolute humidity. The disadvantage is that this value is not that easy to know and it's hard to provide a good default value. Therefore it's perhaps better to use relative humidity, or allow both?

icupeiro commented 5 years ago

@Mathadon standard EN 1397:2015 establishes test conditions of 19 degC air wet bulb temperature and 27degC of dry bulb temperature (therefore the 46.3% on humidity I think it was), so I would stick to the wet bulb temperature as the default parameter. Perhaps is also convenient to check if there are other test procedures (ASHRAE maybe?) and what is the humidity parameter they fix.

Mathadon commented 5 years ago

@mwetter can you provide feedback on this before further implementing the model?

mwetter commented 5 years ago

I think using the absolute humidity is fine for the parameterization. I have not done yet a more thorough review. Donghun from our team will work this summer on cooling coil modeling.

Mathadon commented 5 years ago

@mwetter I have fixed some issues. The unit tests now pass but the model is not yet ready.

Open issues:

mwetter commented 5 years ago

Since it is a "coil", and uses the heat transfer coefficients of a coil, we should not allow specifying condensation on the water side. More examples would be good.

Mathadon commented 5 years ago

I see, I did not realize the specific meaning. In that case it would make sense to add more generic heat exchanger models though?

mwetter commented 5 years ago

Yes, we could start with moving Buildings.Fluid.HeatExchangers.PlateHeatExchangerEffectivenessNTU to IBPSA, and in a 2nd step, it would be good to refactor those effectiveness models to make them graphically (as we did for the Mover) which I think would make them much clearer.

Mathadon commented 5 years ago

Open questions from my side for today's discussion:

Mathadon commented 5 years ago

Todo:

Mathadon commented 5 years ago

@mwetter can you check the parameterization of EP? Then we can try to be consistent.