welly
welly copied to clipboard
Possible PR: option to use well name in place of UWI
We've been using the HTML table Project
method to QC a log dataset (great feature, btw!). Most of the wells in the dataset have names but not UWIs, so I added the option to well names instead of UWIs as the "identifier".
I added names
(which acts like uwis
) and identifier
(one of {'uwi', 'name'}
) arguments to Project.curve_table_html
, as well as @property Project.names
and @property Well.name
. These are implemented just like Project.uwis
and Project.uwi
, respectively.
Just wanted to flag this as a possible enhancement and see if there are any issues with PRing that, or other places in the code where things should be modified to reflect those additions.
The other thing that might be nice is an ability to provide a function to parse the UWI (or name) from the well header. Many wells in my dataset have what are essentially UWIs stored in a range of three or four sometimes missing fields.
I'd be happy to work on that as a PR, following any suggestions or notes here...
this is definitely needed