Order of intervals can break striplog

agilescientific / striplog

Lithology and stratigraphic logs for wells or outcrop.

https://code.agilescientific.com/striplog

Apache License 2.0

204 stars 69 forks source link

Order of intervals can break striplog #127

Closed kwinkunks closed 2 years ago

kwinkunks commented 3 years ago

The problem in the to_log() function, which is consistently getting the indices (start_ix, stop_ix, top_index, base_index) backwards in this dataset.

I think the problem is that we need to reverse indices when order is 'elevation'.

I think this affects to_log() and fill() at least.

See https://swung.slack.com/archives/C01FC2V9NVD/p1616330076013100

mtb-za commented 3 years ago

@kwinkunks this is resolved as of https://github.com/agile-geoscience/striplog/commit/527fe88aa96062f804668cfabdff768e8dd889d9, yes? Looking to see if I can close this issue.

kwinkunks commented 2 years ago

Let's say this is fixed.