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

Incorrect parsing of null values with negative values #427

Closed
ghost opened this issue Mar 9, 2021 · 2 comments
Closed

Incorrect parsing of null values with negative values #427

ghost opened this issue Mar 9, 2021 · 2 comments
Labels

Comments

@ghost
Copy link

ghost commented Mar 9, 2021

I have a las file where one of the log values is -0.0733. When this is read in with null_policy="aggressive", the value is incorrectly parsed as the string NaN733, presumably because -0.0 is one of the values in NULL_POLICIES.

To avoid this unwanted behaviour, it would be useful if lasio checked that -0.0 was followed by a word boundary and not further digits.

@dcslagel
Copy link
Collaborator

Hi @Connossor ,

This should be fixed by pull-request #429 "Change null_policy to handle small non-zero values", which is now merged. If you are working with the GitHub source, could you update to the latest Lasio code and see if your las file's log values are parsed properly when read with `null_policy="aggressive"?

Thank you,
DC

@kinverarity1
Copy link
Owner

Hi @Connossor I'm closing this based on @dcslagel's comment. Please re-open if you find the solution is not working for you!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants