search

OPM / opm-core

Collection of utilities, solvers and other components.
http://www.opm-project.org
GNU General Public License v3.0
44 stars 50 forks source link

Read PVTW from EclipseState #1119

Closed jokva closed 7 years ago

jokva commented 7 years ago

Read the PVTW table entries from EclipseState rather than manually through the Deck object.

jokva commented 7 years ago

Ok, so updated to tablemanager and removed dead exception code path.

bska commented 7 years ago

jenkins build this with downstreams opm-parser=1003 please

bska commented 7 years ago

jenkins build this with downstreams opm-parser=1003 please

The build fails somewhere in opm-parser.

jokva commented 7 years ago

Yeah, missed a git add because it was a new file. It's in the upstream PR now, so it should go through.

bska commented 7 years ago

It's in the upstream PR now, so it should go through.

Thanks. I'll relaunch Jenkins to verify.

bska commented 7 years ago

jenkins build this with downstreams opm-parser=1003 please

jokva commented 7 years ago

Might still fail because I managed to not properly merge some downstreams of another PR.

jokva commented 7 years ago

Alright, should be fine now.

bska commented 7 years ago

should be fine now.

Mr. Jenkins concurs. I approve this patch-set.