OPM/opm-common

Parsing WSEGVALV keyword with string failed

hrsut opened this issue · 6 comments

hrsut commented

Hi
I tried to parse WSEGVALV keyword with a string as follows for example:

WSEGVALV
  WELL_A 2 1.0 FUARE_CC 5* 0.008 /
/

Returning to error:

Internal error: Malformed floating point number 'FUARE_CC'

Hei,

Thanks for reporting, but the forth item should be a Cross-section area, which should be a numerical number.

I think FUARE_CC is a string here. Do you define it is actually a number somewhere?

Sorry if I miss something here.

@hrsut

bska commented

Thanks for reporting, but the fourth item should be a Cross-section area, which should be a numerical number.

I think FUARE_CC is a string here

It is indeed a string, but the initial prefix FU leads me to believe that it is the name of a (field level) user-defined quantity (UDQ). In other words, the model is trying to use this UDQ as a user-defined argument (UDA) in WSEGVALV.

Regrettably, the Flow simulator does not permit UDAs in WSEGVALV at the moment, only in control keywords like WCONPROD and WELTARG. There is at least some amount of work needed to allow UDAs here.

hrsut commented

Hi, yes it is correct that this FUAREA_CC is a UDQ defined in the schedule files.
So before it, there is keyword of

UDQ
DEFINE FUAREA_CC 0.005 /
/

I saw a PR on fixing this for WCONPROD few years ago. Hopefully this can be solved as well.

bska commented

I saw a PR [adding support for UDAs in] WCONPROD few years ago. Hopefully this can be solved as well.

Permitting UDAs in WSEGVALV for calculation purposes is probably fairly straightforward. The biggest part of adding UDAs here, however, is how to represent those UDAs in our restart files. In any case, I have added an item for this to our "feature backlog" for the simulator. We will keep informed of our development in this area.

hrsut commented

thanks ! looking forward for the new development 👍

bska commented

thanks ! looking forward for the new development 👍

I'd like to give an update here. In the latest development sources, we have added preliminary support for for UDAs in item 4 of WSEGVALV. At present we do not have support for restarting simulation runs with those UDAs, but it may nevertheless be of some assistance to you. Unfortunately, this development did not make it into the main development branch in time for the October release, so if you run released versions of the simulator you will have to wait for the H1 2024 release–tentatively scheduled for late April of 2024.