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

Implement Pandas Curve Object #172

Closed
wants to merge 120 commits into from
Closed

Implement Pandas Curve Object #172

wants to merge 120 commits into from

Conversation

patrick-reinhard
Copy link
Collaborator

@patrick-reinhard patrick-reinhard commented Nov 16, 2021

This is a very large PR. The goal is to replace the Curve object (subclassed np.array) with a base class that is composed of a pd.DataFrame() object to hold the curve data. This will set-up welly to be able handle categorical and multi-dimensional curve data. It will also remove the necessity to handle a separate curve basis since we can leverage the pd.DataFrame.Index.

  • Reading/writing, plotting and quality logic is moved out to respectively las.py, plot.py and quality.py. Added wrapper methods to classes.
  • Replaced Curve object
  • Adapted interfaces of functions & methods for new Curve objects
  • Added Github Actions workflows for testing and publishing to PyPi
  • Added tests

More details on this refactor effort and the HLD can be found in this issue.

Kent Inverarity and others added 30 commits April 15, 2021 09:17
Wellpathpy not found with `pip install -e .`
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

3 participants