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

Synthetics Bugs #152

Open
wants to merge 14 commits into
base: develop
Choose a base branch
from

Conversation

hyperiongeo
Copy link

Matt

I had some time so I took a look at this. Turns out the problem was with the wavelet. At some point in the past, the input into the wavelet calculation was made in seconds rather than milliseconds but the ricker calculation expected ms. I changed the ricker calculation to accept seconds (which I believe matches the bruges version).

During testing, I noticed a bug on the as_curve function so I fixed that as well with an improvement over the handling of the log output interval. I also made a change in the make_synthetic where the user can add their chosen sonic and density mneonics rather than having to rename them to DT and RHOB (which remain the defaults).

Hope this meets with your approval.

Corey

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

Successfully merging this pull request may close these issues.

5 participants