-
Notifications
You must be signed in to change notification settings - Fork 68
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
Some changes necessary for subsurface #118
base: main
Are you sure you want to change the base?
Conversation
@Leguark I'm just getting to know the striplog and Markov Chain object preferences and wondering if you can suggest how you'd clean up a log in the form of a pandas table with well, top, base, log. Based on @kwinkunks notebook it looks like a .GroupBy or .to_json might be the way to go. |
Something like this, I suppose: |
… is just an lithology id). This casting makes impossible to map a table to integer properties
� Conflicts: � striplog/striplog.py
…y to fail. This is necessary to be able to use table when the lithologies are given by id (int) instead by name (string)
From dict method
from_csv
. Since I created the pull request someone else has created anotherfrom_dict
method. I just renamed my contribution tofrom_dict_advance
since it does more post processing.I am using this in subsurface to be able to first read a table with pandas, clean it and then converting the pandas dataframe to a valid dictionary that striplog understands.
Supporting integer as formation name
Component
objects with different integers was always comparing them as True.This is the test that needs all of the mention here: https://github.com/softwareunderground/subsurface/blob/6543c9924dcfd5d67068069f2a6fc1f9e17f5ef5/tests/test_io/test_welly_to_subsurface.py#L420