-
Notifications
You must be signed in to change notification settings - Fork 117
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
[testing] sequential running for flow #2723
Draft
GitPaean
wants to merge
94
commits into
OPM:master
Choose a base branch
from
GitPaean:hnil_sequential_august
base: master
Could not load branches
Branch not found: {{ refName }}
Could not load tags
Nothing to show
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Draft
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
GitPaean
force-pushed
the
hnil_sequential_august
branch
4 times, most recently
from
August 12, 2020 21:23
06b0b24
to
62753e2
Compare
jenkins build this please |
1 similar comment
jenkins build this please |
jenkins build this opm-models=624 please |
GitPaean
force-pushed
the
hnil_sequential_august
branch
from
August 19, 2020 12:43
0bea6ab
to
fbe5f79
Compare
jenkins build this opm-models=624 please |
1 similar comment
jenkins build this opm-models=624 please |
It looks like jenkins did not catch all the commits in OPM/opm-models#624 . |
jenkins build this opm-models=624 please |
there is some bug in the Jenkins script. It could not get the latest commit from the opm-models. |
…al will be replaced by inheretence
… step adde pure pressure step
when there is no pressuresolver.json file is provided.
to make sure we have the consistent information when calculating relative change for time step calculation.
…t border with derivative with respect to rs. - corrected update chopping sT - added asserts - output of iterarations to scrren - commented line on alternative inital guess for transport - skip well switching in transport
and adding DIR to the test for flow_blackoil_dunecpr_seq
from different functions. It only happens during the sequential solution. This is to recover the running with flow.
It looks like not used and might be able to do different way with the added inner iteration for StandardWell.
removing the things not needed.
blackoilprimaryvariables.hh. Preparing for refactoring.
it needs more adjustment.
the fuction assignNaive does not work, need to adjust more.
against blackoilintensivequantities.hh, preparing for the possible refactoring.
it will be cleaned up when merging.
it looks like there are only a few lines different from BlackOilNewtonMethod
GitPaean
force-pushed
the
hnil_sequential_august
branch
from
August 21, 2020 20:24
ad94405
to
35e2d68
Compare
not sure it is totally corect yet, while it fixes the compilation.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
No description provided.