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

Limit communication of group state to actually changed vectors #5711

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

vkip
Copy link
Member

@vkip vkip commented Nov 5, 2024

Also: Renames 'currentWellRates' in WellState to reflect that these are actually only used for guide rates.

…me 'currentWellRates' in WellState to reflect that these are actually only used for guide rates
@vkip
Copy link
Member Author

vkip commented Nov 5, 2024

jenkins build this please

@vkip
Copy link
Member Author

vkip commented Nov 5, 2024

jenkins build this please

@vkip
Copy link
Member Author

vkip commented Nov 5, 2024

benchmark please

@ytelses
Copy link

ytelses commented Nov 10, 2024

Benchmark result overview:

Test Configuration Relative
opm-git OPM Benchmark: drogon - Threads: 1 1.008
opm-git OPM Benchmark: drogon - Threads: 8 1.138
opm-git OPM Benchmark: punqs3 - Threads: 1 1.024
opm-git OPM Benchmark: punqs3 - Threads: 8 0.993
opm-git OPM Benchmark: smeaheia - Threads: 1 1.002
opm-git OPM Benchmark: smeaheia - Threads: 8 1
opm-git OPM Benchmark: spe10_model_1 - Threads: 1 1.005
opm-git OPM Benchmark: spe10_model_1 - Threads: 8 1.003
opm-git OPM Benchmark: flow_mpi_extra - Threads: 1 1.012
opm-git OPM Benchmark: flow_mpi_extra - Threads: 8 0.989
opm-git OPM Benchmark: flow_mpi_norne - Threads: 1 1.017
opm-git OPM Benchmark: flow_mpi_norne - Threads: 8 1.01
opm-git OPM Benchmark: flow_mpi_norne_4c_msw - Threads: 1 - FOPT (Total Oil Production At End Of Run) 1
opm-git OPM Benchmark: flow_mpi_norne_4c_msw - Threads: 8 - FOPT (Total Oil Production At End Of Run) 0.996
  • 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=2648

@vkip vkip marked this pull request as ready for review November 18, 2024 12:32
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.

2 participants