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

parameters to extend drsdtcon #4097

Merged
merged 1 commit into from
Jun 28, 2024
Merged

parameters to extend drsdtcon #4097

merged 1 commit into from
Jun 28, 2024

Conversation

trinemykk
Copy link
Contributor

@trinemykk trinemykk commented Jun 7, 2024

connected to opm-models (OPM/opm-models#902) and opm-simulators (OPM/opm-simulators#5418)

@totto82
Copy link
Member

totto82 commented Jun 7, 2024

jenkins build this opm-models=902 opm-simulators=5418 please

@totto82
Copy link
Member

totto82 commented Jun 7, 2024

The test failures are in the opm-models blackoil models i.e. they dont use opmcommon stuff. Adding a enableConvectiveMixing flag and make the convectiveMixingModule depent on this flag is the best way IMO to solve this. (ala what is done for the rest of the blackoilmodules)

@totto82
Copy link
Member

totto82 commented Jun 12, 2024

jenkins build this opm-models=902 opm-simulators=5418 please

5 similar comments
@totto82
Copy link
Member

totto82 commented Jun 17, 2024

jenkins build this opm-models=902 opm-simulators=5418 please

@totto82
Copy link
Member

totto82 commented Jun 18, 2024

jenkins build this opm-models=902 opm-simulators=5418 please

@totto82
Copy link
Member

totto82 commented Jun 18, 2024

jenkins build this opm-models=902 opm-simulators=5418 please

@totto82
Copy link
Member

totto82 commented Jun 19, 2024

jenkins build this opm-models=902 opm-simulators=5418 please

@totto82
Copy link
Member

totto82 commented Jun 19, 2024

jenkins build this opm-models=902 opm-simulators=5418 please

@totto82
Copy link
Member

totto82 commented Jun 20, 2024

jenkins build this please

@totto82
Copy link
Member

totto82 commented Jun 24, 2024

jenkins build this opm-models=902 opm-simulators=5418 please

@totto82
Copy link
Member

totto82 commented Jun 25, 2024

Thanks. The test failure in the DRSDTCON test is expected. I have manually checked the results and they are ok. To be on the safe side I will run a benchmark to check the performance. After that this can be merged.

@totto82
Copy link
Member

totto82 commented Jun 25, 2024

benchmark please opm-models=902 opm-simulators=5418

@ytelses
Copy link

ytelses commented Jun 25, 2024

Benchmark result overview:

Test Configuration Relative
  • Speed-up = Total time master / Total time pull request. Above 1.0 is an improvement. *

FAILED RUNS:
OPM Benchmark: drogon - Threads: 1 - opm-common 4097: parameters to extend drsdtcon: The test run did not produce a result. E: mpirun was unable to launch the specified application as it could not access
OPM Benchmark: drogon - Threads: 8 - opm-common 4097: parameters to extend drsdtcon: The test run did not produce a result. E: mpirun was unable to launch the specified application as it could not access
OPM Benchmark: punqs3 - Threads: 1 - opm-common 4097: parameters to extend drsdtcon: The test run did not produce a result. E: mpirun was unable to launch the specified application as it could not access
OPM Benchmark: punqs3 - Threads: 8 - opm-common 4097: parameters to extend drsdtcon: The test run did not produce a result. E: mpirun was unable to launch the specified application as it could not access
OPM Benchmark: smeaheia - Threads: 1 - opm-common 4097: parameters to extend drsdtcon: The test run did not produce a result. E: mpirun was unable to launch the specified application as it could not access
OPM Benchmark: smeaheia - Threads: 8 - opm-common 4097: parameters to extend drsdtcon: The test run did not produce a result. E: mpirun was unable to launch the specified application as it could not access
OPM Benchmark: spe10_model_1 - Threads: 1 - opm-common 4097: parameters to extend drsdtcon: The test run did not produce a result. E: mpirun was unable to launch the specified application as it could not access
OPM Benchmark: spe10_model_1 - Threads: 8 - opm-common 4097: parameters to extend drsdtcon: The test run did not produce a result. E: mpirun was unable to launch the specified application as it could not access
OPM Benchmark: flow_mpi_extra - Threads: 1 - opm-common 4097: parameters to extend drsdtcon: The test run did not produce a result. E: mpirun was unable to launch the specified application as it could not access
OPM Benchmark: flow_mpi_extra - Threads: 8 - opm-common 4097: parameters to extend drsdtcon: The test run did not produce a result. E: mpirun was unable to launch the specified application as it could not access
OPM Benchmark: flow_mpi_norne - Threads: 1 - opm-common 4097: parameters to extend drsdtcon: The test run did not produce a result. E: mpirun was unable to launch the specified application as it could not access
OPM Benchmark: flow_mpi_norne - Threads: 8 - opm-common 4097: parameters to extend drsdtcon: The test run did not produce a result. E: mpirun was unable to launch the specified application as it could not access
OPM Benchmark: flow_mpi_norne_4c_msw - Threads: 1 - opm-common 4097: parameters to extend drsdtcon: The test run did not produce a result. E: mpirun was unable to launch the specified application as it could not access
OPM Benchmark: flow_mpi_norne_4c_msw - Threads: 8 - opm-common 4097: parameters to extend drsdtcon: The test run did not produce a result. E: mpirun was unable to launch the specified application as it could not access

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

@totto82
Copy link
Member

totto82 commented Jun 25, 2024

Did I do something wrong when I called benchmark?

@totto82
Copy link
Member

totto82 commented Jun 25, 2024

benchmark opm-models=902 opm-simulators=5418 please

@totto82
Copy link
Member

totto82 commented Jun 26, 2024

benchmark please opm-models=902 opm-simulators=5418

@ytelses
Copy link

ytelses commented Jun 26, 2024

Benchmark result overview:

Test Configuration Relative
  • Speed-up = Total time master / Total time pull request. Above 1.0 is an improvement. *

FAILED RUNS:
OPM Benchmark: drogon - Threads: 1 - opm-common 4097: parameters to extend drsdtcon: The test run did not produce a result. E: mpirun was unable to launch the specified application as it could not access
OPM Benchmark: drogon - Threads: 8 - opm-common 4097: parameters to extend drsdtcon: The test run did not produce a result. E: mpirun was unable to launch the specified application as it could not access
OPM Benchmark: punqs3 - Threads: 1 - opm-common 4097: parameters to extend drsdtcon: The test run did not produce a result. E: mpirun was unable to launch the specified application as it could not access
OPM Benchmark: punqs3 - Threads: 8 - opm-common 4097: parameters to extend drsdtcon: The test run did not produce a result. E: mpirun was unable to launch the specified application as it could not access
OPM Benchmark: smeaheia - Threads: 1 - opm-common 4097: parameters to extend drsdtcon: The test run did not produce a result. E: mpirun was unable to launch the specified application as it could not access
OPM Benchmark: smeaheia - Threads: 8 - opm-common 4097: parameters to extend drsdtcon: The test run did not produce a result. E: mpirun was unable to launch the specified application as it could not access
OPM Benchmark: spe10_model_1 - Threads: 1 - opm-common 4097: parameters to extend drsdtcon: The test run did not produce a result. E: mpirun was unable to launch the specified application as it could not access
OPM Benchmark: spe10_model_1 - Threads: 8 - opm-common 4097: parameters to extend drsdtcon: The test run did not produce a result. E: mpirun was unable to launch the specified application as it could not access
OPM Benchmark: flow_mpi_extra - Threads: 1 - opm-common 4097: parameters to extend drsdtcon: The test run did not produce a result. E: mpirun was unable to launch the specified application as it could not access
OPM Benchmark: flow_mpi_extra - Threads: 8 - opm-common 4097: parameters to extend drsdtcon: The test run did not produce a result. E: mpirun was unable to launch the specified application as it could not access
OPM Benchmark: flow_mpi_norne - Threads: 1 - opm-common 4097: parameters to extend drsdtcon: The test run did not produce a result. E: mpirun was unable to launch the specified application as it could not access
OPM Benchmark: flow_mpi_norne - Threads: 8 - opm-common 4097: parameters to extend drsdtcon: The test run did not produce a result. E: mpirun was unable to launch the specified application as it could not access
OPM Benchmark: flow_mpi_norne_4c_msw - Threads: 1 - opm-common 4097: parameters to extend drsdtcon: The test run did not produce a result. E: mpirun was unable to launch the specified application as it could not access
OPM Benchmark: flow_mpi_norne_4c_msw - Threads: 8 - opm-common 4097: parameters to extend drsdtcon: The test run did not produce a result. E: mpirun was unable to launch the specified application as it could not access

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

@totto82
Copy link
Member

totto82 commented Jun 27, 2024

@akva2 Do I do something wrong when calling benchmark?

@akva2
Copy link
Member

akva2 commented Jun 27, 2024

i'm not responsible for the benchmark support but i cannot see anything wrong as such. but the PR is also red, are you sure this actually builds?

One guess would be that the trigger builds the PR branches rather than the PR branches pulled into the respective masters so maybe you should rebase the PR's ?

@totto82
Copy link
Member

totto82 commented Jun 27, 2024

i'm not responsible for the benchmark support but i cannot see anything wrong as such. but the PR is also red, are you sure this actually builds?

I was unsure about the syntax to call benchmark across multiple repos. "benchmark please opm-models=902 opm-simulators=5418" Will this work. Or do I need to add please in the end.

@akva2
Copy link
Member

akva2 commented Jun 27, 2024

i believe your syntax is correct, although I must point out that whoever implemented it did not understand the practical role of the please in the trigger structure (ie it's there to have an end anchor). I have never seen the scripts behind the trigger so I can't really help other than my faint memory seeing that structure work before.

@totto82
Copy link
Member

totto82 commented Jun 27, 2024

jenkins build this opm-models=902 opm-simulators=5418 please

@ytelses
Copy link

ytelses commented Jun 27, 2024

Benchmark result overview:

Test Configuration Relative
  • Speed-up = Total time master / Total time pull request. Above 1.0 is an improvement. *

FAILED RUNS:
OPM Benchmark: drogon - Threads: 1 - opm-common 4097: parameters to extend drsdtcon: The test run did not produce a result. E: mpirun was unable to launch the specified application as it could not access
OPM Benchmark: drogon - Threads: 8 - opm-common 4097: parameters to extend drsdtcon: The test run did not produce a result. E: mpirun was unable to launch the specified application as it could not access
OPM Benchmark: punqs3 - Threads: 1 - opm-common 4097: parameters to extend drsdtcon: The test run did not produce a result. E: mpirun was unable to launch the specified application as it could not access
OPM Benchmark: punqs3 - Threads: 8 - opm-common 4097: parameters to extend drsdtcon: The test run did not produce a result. E: mpirun was unable to launch the specified application as it could not access
OPM Benchmark: smeaheia - Threads: 1 - opm-common 4097: parameters to extend drsdtcon: The test run did not produce a result. E: mpirun was unable to launch the specified application as it could not access
OPM Benchmark: smeaheia - Threads: 8 - opm-common 4097: parameters to extend drsdtcon: The test run did not produce a result. E: mpirun was unable to launch the specified application as it could not access
OPM Benchmark: spe10_model_1 - Threads: 1 - opm-common 4097: parameters to extend drsdtcon: The test run did not produce a result. E: mpirun was unable to launch the specified application as it could not access
OPM Benchmark: spe10_model_1 - Threads: 8 - opm-common 4097: parameters to extend drsdtcon: The test run did not produce a result. E: mpirun was unable to launch the specified application as it could not access
OPM Benchmark: flow_mpi_extra - Threads: 1 - opm-common 4097: parameters to extend drsdtcon: The test run did not produce a result. E: mpirun was unable to launch the specified application as it could not access
OPM Benchmark: flow_mpi_extra - Threads: 8 - opm-common 4097: parameters to extend drsdtcon: The test run did not produce a result. E: mpirun was unable to launch the specified application as it could not access
OPM Benchmark: flow_mpi_norne - Threads: 1 - opm-common 4097: parameters to extend drsdtcon: The test run did not produce a result. E: mpirun was unable to launch the specified application as it could not access
OPM Benchmark: flow_mpi_norne - Threads: 8 - opm-common 4097: parameters to extend drsdtcon: The test run did not produce a result. E: mpirun was unable to launch the specified application as it could not access
OPM Benchmark: flow_mpi_norne_4c_msw - Threads: 1 - opm-common 4097: parameters to extend drsdtcon: The test run did not produce a result. E: mpirun was unable to launch the specified application as it could not access
OPM Benchmark: flow_mpi_norne_4c_msw - Threads: 8 - opm-common 4097: parameters to extend drsdtcon: The test run did not produce a result. E: mpirun was unable to launch the specified application as it could not access

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

@totto82
Copy link
Member

totto82 commented Jun 27, 2024

I have tested Norne and drogon and smeaheia and they all run fine (i.e. no changes in results and performance) locally.

@totto82
Copy link
Member

totto82 commented Jun 28, 2024

I will try to merge this step by step. First opm-common.

@totto82
Copy link
Member

totto82 commented Jun 28, 2024

jenkins build this please

@totto82 totto82 merged commit 53af14e into OPM:master Jun 28, 2024
1 check passed
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.

4 participants