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.

Sign up for GitHub

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

Jump to bottom

More useful time-depth conversion #32

Open

kwinkunks opened this issue Nov 24, 2017 · 2 comments

Member

kwinkunks commented Nov 24, 2017

This module should be refactored.

It does not tolerate NaNs in the t-d function, eg if you pass a Vp curve with Nans in it.

I'd like to be able to look up times and depths singly.

Member Author

kwinkunks commented Nov 24, 2017

It might be nice to optionally return the basis too.

And I'm not sure kind='nearest' is the best choice of interpolation.

Member Author

kwinkunks commented Nov 24, 2017

And... I don't think it works in 3D

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