equinor / sunbeam

GNU General Public License v3.0
7 stars 14 forks source link

Schedule merge functionality. #69

Closed joakim-hove closed 6 years ago

joakim-hove commented 6 years ago

This is a basic building block for a Schedule merge tool. I suggest we merge something like this after review - then Lars / Håvard can see how it fits their needs, and we can iteratively improve things.

joakim-hove commented 6 years ago

OK - I'll merge this the coming afternoon.

lahus2 commented 6 years ago

looks pretty good to me. I didn't have time to download and run yet, just looked at code and comments. I agree with @berland about keyword vs DATE. One point and this can be for the future.

Currently you support load (which just appends data to existing dates). I would like the option to filter(remove) and replace data (not just appending) on a timeframe/KEYWORD/well(or similar identifier) basis. This is probably more a feature for the future, just would be very useful.