-
Notifications
You must be signed in to change notification settings - Fork 16
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
generating new curves #38
Comments
Hi John, I'm going back to this today. Don't think it will make it into the app, but if theres a working proof of concept by the end of the hackathon I'll continue this work and add a 'petrophysics' module. I went throught the Petropy stuff yesterday, and whilst there's some functions we can use there's some that we can't with this data. Most of the functions have inputs that we don't have with this dataset, so a rewrite would be needed with optional functionality, I'd do this via **kwargs but open to other suggestions. For example if PHI=Null, then compute from density, or another curve etc and print a warning/statement. https://toddheitmann.github.io/PetroPy/_modules/petropy/log.html#Log.fluid_properties uses default properties if args are null/not entered. But that's also pretty incomprehensible, and a lot of the input properties can be calculated. For example in that code:
Can all be calculated for only 6 required inputs; Pressure, Temperature, Salinity, Rmf at surface/pad, mf density, HC gravity/api. |
you all did a great job ... I hope that I can contribute at keeping this
going.
Cheers
John
…On Sun, Jun 14, 2020 at 4:35 AM Kieran John Blacker < ***@***.***> wrote:
Hi John, I'm going back to this today. Don't think it will make it into
the app, but if theres a working proof of concept by the end of the
hackathon I'll continue this work and add a 'petrophysics' module.
I went throught the Petropy stuff yesterday, and whilst there's some
functions we can use there's some that we can't with this data. Most of the
functions have inputs that we don't have with this dataset, so a rewrite
would be needed with optional functionality, I'd do this via **kwargs but
open to other suggestions. For example if PHI=Null, then compute from
density, or another curve etc and print a warning/statement.
https://toddheitmann.github.io/PetroPy/_modules/petropy/log.html#Log.fluid_properties
uses default properties if args are null/not entered. But that's also
pretty incomprehensible, and a lot of the input properties can be
calculated. For example in that code:
RW : (ohm.m) Resistivity of water RMF : (ohm.m) Resistivity of mud
filtrate RHO_HC : (g / cc) Density of hydrocarbon RHO_W : (g / cc) Density
of formation water RHO_MF : (g / cc) Density of mud filtrate NPHI_HC
Neutron log response of hydrocarbon NPHI_W Neutron log response of water
NPHI_MF Neutron log response of mud filtrate MU_HC : (cP) Viscosity of
hydrocarbon Z Compressiblity factor for non-ideal gas. Only output if
oil_api = 0 CG : (1 / psi) Gas Compressiblity. Only output if oil_api = 0
BG Gas formation volume factor. Only output if oil_api = 0 BP : (psi)
Bubble point. Only output if oil_api > 0 BO Oil formation volume factor.
Only output if oil_api > 0
Can all be calculated for only 6 required inputs; Pressure, Temperature,
Salinity, Rmf at surface/pad, mf density, HC gravity/api.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
<#38 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AJA2AATOF6MOJ5JFQ4ZZPTLRWSRYLANCNFSM4N466RIA>
.
--
John Pantano PhD
LinkedIn <https://www.linkedin.com/pub/john-pantano/2a/559/91b> Page
406 490-8387
|
I have written down some additional capabilities that can build off https://toddheitmann.github.io/PetroPy/index.html#
This needs more refinement and post to inquiry about possible collaboration.
log.fluid_properties_parameters (read_csv)
generate_fluid_property_curves
edit_fluid_property_curves (optional)
generate_fluid_property_curves
formation_multimineral_model
generate_list_of_curves
generate_list_of_calculations (read_csv)
select_calculation(s)
select_well(s)
map_function_arguments_to_curves
assign_values_for_arguments_for_no_curves
generate_new_curve(s)
The text was updated successfully, but these errors were encountered: