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

Missing value management on the reference curve #45

Closed
VelizarVESSELINOV opened this issue Aug 4, 2015 · 3 comments
Closed

Missing value management on the reference curve #45

VelizarVESSELINOV opened this issue Aug 4, 2015 · 3 comments
Assignees

Comments

@VelizarVESSELINOV
Copy link
Contributor

If NULL is setup as 0, and the LAS file have reference DEPTH with value 0, today the library is replacing the value with nan for all the Curves, I know some other software packages that do this only for curves that are not reference.

LAS specification document is not very clear about how should be managed reference with NULL value, but I found it more logical to keep the reference without NULL replacement.

What do you think?

@kinverarity1
Copy link
Owner

I think your suggestion should be the default, with a keyword argument to control whether a substitution is made everywhere or not at all or with a different value to numpy.nan

@kinverarity1
Copy link
Owner

In light of #49 this could be a keyword argument index_null_replacement which is False by default.

@kinverarity1
Copy link
Owner

This issue is the same as #227 - which is now fixed!

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

No branches or pull requests

2 participants