Skip to content

Give correct labeling of nodes in parallel#917

Open
hnil wants to merge 1 commit into
OPM:masterfrom
hnil:fixed_add_corners
Open

Give correct labeling of nodes in parallel#917
hnil wants to merge 1 commit into
OPM:masterfrom
hnil:fixed_add_corners

Conversation

@hnil

@hnil hnil commented Aug 19, 2025

Copy link
Copy Markdown
Member

This corrects several issues with labeling of nodes. It include correct addition of corners for main the call used in opm-flow and for proper use depend on edge conformal processing.

…ing to be correct.

The correct behavoir will currently only happen in normal calls from opm-flow
@hnil hnil added the manual:irrelevant This PR is a minor fix and should not appear in the manual label Aug 19, 2025
@hnil hnil requested a review from atgeirr August 19, 2025 13:22
@blattms

blattms commented Aug 21, 2025

Copy link
Copy Markdown
Member

jenkins build this please

@blattms

blattms commented Aug 21, 2025

Copy link
Copy Markdown
Member

benchmark please

@ytelses

ytelses commented Aug 23, 2025

Copy link
Copy Markdown

Benchmark result overview:

Test Configuration Relative
opm-git OPM Benchmark: drogon - Threads: 1 0.86
opm-git OPM Benchmark: drogon - Threads: 8 0.946
opm-git OPM Benchmark: punqs3 - Threads: 1 1.024
opm-git OPM Benchmark: punqs3 - Threads: 8 0.989
opm-git OPM Benchmark: smeaheia - Threads: 1 1
opm-git OPM Benchmark: smeaheia - Threads: 8 0.995
opm-git OPM Benchmark: spe10_model_1 - Threads: 1 1.014
opm-git OPM Benchmark: spe10_model_1 - Threads: 8 1.012
opm-git OPM Benchmark: flow_mpi_extra - Threads: 1 0.967
opm-git OPM Benchmark: flow_mpi_extra - Threads: 8 0.973
opm-git OPM Benchmark: flow_mpi_norne - Threads: 1 1.008
opm-git OPM Benchmark: flow_mpi_norne - Threads: 8 1.004
opm-git OPM Benchmark: flow_mpi_norne_4c_msw - Threads: 1 - FOPT (Total Oil Production At End Of Run) 0.998
opm-git OPM Benchmark: flow_mpi_norne_4c_msw - Threads: 8 - FOPT (Total Oil Production At End Of Run) 1.002
  • 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=2857

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

manual:irrelevant This PR is a minor fix and should not appear in the manual

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants