OPM / opm-models

The models module for the Open Porous Media Simulation (OPM) framework
Other
17 stars 69 forks source link

avoid divition by zero for case without conductivity #863

Closed totto82 closed 7 months ago

totto82 commented 7 months ago

For numerical aquifers the thermal half transmissibiltes are zero which results in infinite update. We may think about adding the option for setting these from deck. As a first approach we fix the code to allow for zero values.

GitPaean commented 7 months ago

good. I was about to create the same PR.

GitPaean commented 7 months ago

For numerical aquifers the thermal half transmissibiltes are zero which results in infinite update.

It is not for only the numerical aquifers, it looks the problem the same with other NNCs.

totto82 commented 7 months ago

It is not for only the numerical aquifers, it looks the problem the same with other NNCs.

That's right. We could think about extending NCC to allow for thermal conductivity.

totto82 commented 7 months ago

jenkins build this please

totto82 commented 7 months ago

jenkins build this please