Skip to content
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

Support showing multiple VFP names/curves at the same time #758

Open
anders-kiaer opened this issue Oct 14, 2024 · 3 comments
Open

Support showing multiple VFP names/curves at the same time #758

anders-kiaer opened this issue Oct 14, 2024 · 3 comments
Assignees

Comments

@anders-kiaer
Copy link
Collaborator

...in order to compare them. Use case mentioned from Troll. What do you think @lindjoha?

@lindjoha
Copy link
Collaborator

Hi, yes I have thought about that usecase. I agree that it would be useful and I think it could be done. Obviously the curves can only be compared if they have the same flow rate type, WFR type, GFR type, unit system, datum depth (?).

In addition, if the table input values (THP values etc) are not the same, one of the tables needs to be "resampled" to the values of other. It's not exactly clear for me how to do that, but I will think about it.

@lindjoha lindjoha self-assigned this Oct 18, 2024
@lindjoha
Copy link
Collaborator

@runhol, you have developed this resampling functionality earlier?

@runhol
Copy link

runhol commented Oct 18, 2024

I have a vfp prod class implementation that has some functionality for doing that. If the two curves has the same parametrization type (rate type, GFR type, WFR type) it already has functionality to do the resampling. It uses Scipy for interpolation and there are different options to handle values that are outside the ranges (extrapolation etc). If the curves have different parametrization type, there is no functionality that takes care of that automatically. However, to avoid dealing with every possible combination of parametrizations there is an interpolation option that takes in rates (oil, gas,water,gaslift) and calculates BHP. So if the rates + fractions that are going to be interpolated are transformed to rates first, there is a function to do the interpolation.

For a first implementation of such functionality it is perhaps best to assume that the parametrization (type and ranges) is the same for the vlps. I think that simplifies the QC and covers most of the use cases.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants