bp / bp/ResSimpy

Removing QMULTs removes the CONSTRAINT keyword at the top of the table

Open
#159 0 comments 0 reactions 0 assignees View on GitHub
bug good first issue
Dominant language
Python
Stars
23
Forks
8
Avg merge
16d 4h
Merged PRs (30d)
1

Description

The following test case fails when removing the constraint. Calling the remove command via the W001 constraint ID.
```sim.network.constraints.remove(constraint_id=constraint_id) ```

With the following input surface file:
```
TIME 01/04/1980
QMULT
WELL QOIL QGAS QWATER
W001 420 99991.0 0
W002 10 20 30
ENDQMULT
CONSTRAINTS
W001 ACTIVATE
W001 PMIN 5 QALLRMAX MULT
W002 PMIN 5 QALLRMAX MULT
ENDCONSTRAINTS
```
Results in:
```
TIME 01/04/1980
QMULT
W001 420 99991.0 0
W002 10 20 30
ENDQMULT
W001 PMIN 5 QALLRMAX MULT
W002 PMIN 5 QALLRMAX MULT
ENDCONSTRAINTS
```

Where as it should result in:

```
TIME 01/04/1980
QMULT
WELL QOIL QGAS QWATER
W002 10 20 30
ENDQMULT
CONSTRAINTS
W002 PMIN 5 QALLRMAX MULT
ENDCONSTRAINTS
```

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.