GEOS icon indicating copy to clipboard operation
GEOS copied to clipboard

Verification of two-phase flow in porous media with Buckley-Leverett problem

Open jhuang2601 opened this issue 2 years ago • 6 comments

This PR

  • [x] documents the analytical verification of CO2 core flood experiment with Buckley-Leverett theory;

image

jhuang2601 avatar Aug 24 '22 21:08 jhuang2601

@jhuang2601 @francoishamon So this is not added into the integratedTests, we're OK with this? As a result we do not need to rebaseline.

TotoGaz avatar Aug 31 '22 16:08 TotoGaz

@jhuang2601 @francoishamon So this is not added into the integratedTests, we're OK with this? As a result we do not need to rebaseline.

@TotoGaz We need to rebaseline as new smoke test is added into integratedTest. But I cannot run it on Quartz and have requested @francoishamon to do it for me.

jhuang2601 avatar Aug 31 '22 16:08 jhuang2601

@TotoGaz We need to rebaseline as new smoke test is added into integratedTest. But I cannot run it on Quartz and have requested @francoishamon to do it for me.

Yes, I was trying to give it a go myself to purge the merge queue.

But as it is now, the new case is not added to the integratedTests suite. IIUC you would need some action here and there? As a matter of fact, this can be merged, but would never get played (I think).

Better wait for @francoishamon for extra information.

TotoGaz avatar Aug 31 '22 16:08 TotoGaz

@TotoGaz We need to rebaseline as new smoke test is added into integratedTest. But I cannot run it on Quartz and have requested @francoishamon to do it for me.

Yes, I was trying to give it a go myself to purge the merge queue.

But as it is now, the new case is not added to the integratedTests suite. IIUC you would need some action here and there? As a matter of fact, this can be merged, but would never get played (I think).

Better wait for @francoishamon for extra information.

Yes, I have created a PR in the integratedTests branch for adding the new smoke test (see here); but I cannot merge this PR in the integratedTests branch as this new test should be running on Quartz for rebaseline, which needs help from @francoishamon.

jhuang2601 avatar Aug 31 '22 17:08 jhuang2601

Oh, OK. I did not see the PR + I could not see the modified integratedTests repository modified in the files.

TotoGaz avatar Aug 31 '22 17:08 TotoGaz

@francoishamon I let you deal with it then?

TotoGaz avatar Aug 31 '22 17:08 TotoGaz

@francoishamon I let you deal with it then?

Hello @jhuang2601 and @TotoGaz yes, I am going to work on the rebaseline today

francoishamon avatar Aug 31 '22 20:08 francoishamon