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

[bugfix?] Add overlap cell even if intersection has zero transmissibility #726

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

blattms
Copy link
Member

@blattms blattms commented Jun 7, 2024

While there might be no flow of oil/gas/water over an intersection with zero transmissibility, there might still be heat conduction. I think in that case we need to see the neighbor in parallel.

I have to admit that I don't have the slightest idea how heat transport is implemented. So this wild guess might very well be wrong.

Hence this is up for discussion,

…lity.

While there might be now flow of oil/gas/water, there might be heat
conduction. I think in that case we need to see the neighbor in parallel.
@blattms
Copy link
Member Author

blattms commented Jun 7, 2024

jenkins build this please

@blattms
Copy link
Member Author

blattms commented Jun 7, 2024

benchmark please

@ytelses
Copy link

ytelses commented Jun 7, 2024

Benchmark result overview:

Test Configuration Relative
opm-git OPM Benchmark: drogon - Threads: 1 1.008
opm-git OPM Benchmark: drogon - Threads: 8 1.212
opm-git OPM Benchmark: punqs3 - Threads: 1 1.013
opm-git OPM Benchmark: punqs3 - Threads: 8 1.012
opm-git OPM Benchmark: smeaheia - Threads: 1 1.014
opm-git OPM Benchmark: smeaheia - Threads: 8 0.996
opm-git OPM Benchmark: spe10_model_1 - Threads: 1 1.01
opm-git OPM Benchmark: spe10_model_1 - Threads: 8 0.993
opm-git OPM Benchmark: flow_mpi_extra - Threads: 1 0.995
opm-git OPM Benchmark: flow_mpi_extra - Threads: 8 1.001
opm-git OPM Benchmark: flow_mpi_norne - Threads: 1 1.003
opm-git OPM Benchmark: flow_mpi_norne - Threads: 8 0.999
opm-git OPM Benchmark: flow_mpi_norne_4c_msw - Threads: 1 0.989
opm-git OPM Benchmark: flow_mpi_norne_4c_msw - Threads: 8 0.988
  • Speed-up = Total time master / Total time pull request. Above 1.0 is an improvement. *

View result details @ https://www.ytelses.com/opm/?page=result&id=2501

@atgeirr
Copy link
Member

atgeirr commented Jun 11, 2024

While there might be no flow of oil/gas/water over an intersection with zero transmissibility, there might still be heat conduction. I think in that case we need to see the neighbor in parallel.

Good point. Would it be easy to make this configurable? The original implementation comments that not adding this overlap may improve parallel performance, which sounds like an option is a good idea. Also we do not have any idea if this condition has ever been triggered (or at least I don't), maybe we should try to make a count?

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.

3 participants