DC Slagel

Results 27 comments of DC Slagel

Triage notes: This looks like it is a bug in Lasio:reader.py:read_header_line():value_re_for_param_section. It needs to catch the ' 01'. I'll make a pull-request to Lasio to fix this next week. DC

The fix for this merged to Lasio's main line in commit https://github.com/kinverarity1/lasio/commit/ada0bff46a44bed623d5bde9327009d46b17c0dd. It will probably be in the next Lasio release (current release is 0.25.1). DC

This is fix is available as of Lasio v.26. #### Test for parsing the "RUN . 01: RUN NUMBER". string ```python def test_value_field_with_num_colon(): line = "RUN . 01: RUN NUMBER"...

@kinverarity1, I should get to review this on the weekend.

@kinverarity1, I was able to an initial scan this weekend. The initial review looks good. If I could spend a bit more time understanding the impact of this change, I...

@kinverarity1, Quick update. I've been delayed in fully looking at this. Currently, I have it on my schedule for late this week. (Fri - Sun). DC

In addition change looks like it also fixes: https://github.com/kinverarity1/lasio/issues/511. I just ran the time_dummy_example.las and the date-time fields stay formatted and the negative numbers also stay formatted properly.

Hi @eucalypt1 , Is it correct that there are 3 issues as follows: 1. The previous (old) data section shouldn't be added to the ~Other section. 2. NULL values should...

For `How to change the number format in the ~ASCII section`, there are 2 parameters that can be passed into to the write function: `fmt` and column_fmt. These use python's...

Some discussion... If we add this feature would it be always on? Or would it be controlled by a flag/parameter and what would be the default (off|on)? --- If a...