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

Update defaults.py #115

Closed
wants to merge 10 commits into from
Closed

Conversation

ThomasMGeo
Copy link
Contributor

Updated two resistivity curve names

Updated two resistivity curve names
added another medium res log
Added Caliper and PE cure from a super weird .las file, UKCS 14/30a-5
Added density correction
RLA# series of curves
@ThomasMGeo
Copy link
Contributor Author

Slowly working through my UKCS dataset ...

@kwinkunks
Copy link
Member

Thank you Thomas!

See Q on #114 -- as we get further from Ross Crain's list, I wonder if we should freeze his list (which is sort-of published and 'authoritative') and just have an alternative welly default for people to use.

I'll leave this until we decide on the general approach.

@kwinkunks
Copy link
Member

This has been added to develop and your list is now the default, with the original as defaults.CRAIN. Thanks Thomas!

@kwinkunks kwinkunks closed this Dec 11, 2020
kwinkunks added a commit that referenced this pull request Dec 11, 2020
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

Successfully merging this pull request may close these issues.

None yet

2 participants