welly icon indicating copy to clipboard operation
welly copied to clipboard

Possible PR: option to use well name in place of UWI

Open rgmyr opened this issue 5 years ago • 2 comments

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.

rgmyr avatar Mar 22 '19 19:03 rgmyr

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...

kinverarity1 avatar Mar 23 '19 11:03 kinverarity1

this is definitely needed

atwahsz avatar Aug 28 '20 06:08 atwahsz