IBPSA.Fluid.Geothermal.Borefields.Data.Configuration error with 5 or 6 boreholes

[xenkGit]

Dear IBPSA team,

it seems IBPSA.Fluid.Geothermal.Borefields.Data.Configuration does not work with 5 or 6 boreholes.
Works:
cooBor={{22.24,50.03},{17.21,46.06},{11.12,43.68},{4.76,43.68},{0,43.68},{0,37.59},{0,31.5}}
cooBor={{22.24,50.03},{17.21,46.06},{11.12,43.68},{4.76,43.68}}
Does not work:
cooBor={{22.24,50.03},{17.21,46.06},{11.12,43.68},{4.76,43.68},{0,43.68}}
cooBor={{22.24,50.03},{17.21,46.06},{11.12,43.68},{4.76,43.68},{0,43.68},{0,37.59}}

The issue is reproducible by adding one borehole to IBPSA.Fluid.Geothermal.Borefields.Data.Configuration.Example and then simulating IBPSA.Fluid.Geothermal.Borefields.Examples.Borefields.

Sorry if I'm overlooking something obvious...

[mwetter]

I can reproduce this error by overriding cooBor with

cooBor={{22.24,50.03},{17.21,46.06},{11.12,43.68},{4.76,43.68},{0,43.68}}

in IBPSA.Fluid.Geothermal.Borefields.Data.Configuration.Example and then simulate IBPSA.Fluid.Geothermal.Borefields.Examples.Borefields

I made the change on the branch issue1976_borefield_coord.

@MassimoCimmino : Can you please look into what causes this division by zero. The same error also happens with OpenModelica.

[mwetter]

I can reproduce this error by overriding cooBor with

cooBor={{22.24,50.03},{17.21,46.06},{11.12,43.68},{4.76,43.68},{0,43.68}}

in IBPSA.Fluid.Geothermal.Borefields.Data.Configuration.Example and then simulate IBPSA.Fluid.Geothermal.Borefields.Examples.Borefields

I made the change on the branch issue1976_borefield_coord.

@MassimoCimmino : Can you please look into what causes this division by zero. The same error also happens with OpenModelica.

[MassimoCimmino]

I was able to reproduce the error. The issue is likely with the identification of the maximum number of borehole clusters in Buildings.Fluid.Geothermal.Borefields.BaseClasses.HeatTransfer.ThermalResponseFactors.clusterBoreholes.

I will investigate and find a fix.

@xenkGit A temporary fix to get your simulations running is to adjust the parameter nClu (under the advanced tab) in the borefield models. For the 6 boreholes case, nClu=4 seems to run correctly.

[xenkGit]

Thank you so much, the temporary fix works :)

[mwetter]

This has been addressed through #1993