-
Notifications
You must be signed in to change notification settings - Fork 29
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
Fixed inverse q-spacing in the geometric extrapolation function #571
Conversation
…s-per-decade-is-provided-or-data-is-a-single-point
…og_delta_q is calculated when points_per_decade is specified
Testing with: geometric_extrapolation(np.array([1e-4]), 1e-5, 1e-3, points_per_decade=4) gives an extra point in the preceding decade:
because We may find that we have different results on different computers because of small differences in floating point processing, but this will likely be less than the effect of using single rather than double precision if a GPU is available so I'm inclined to ignore it. |
I'm not sure why this is marked as draft, but it seems ready to merge. |
@caitwolf, Paul thinks this is ready to move from draft. Did you have some issues you were holding this back for? In particular do we want this to be in the sasmodels release that will build SasView 6.0? |
@caitwolf , @dehoni @rmdalgliesh @gnsmith @wimbouwman - Should this be part of 6.0? |
…s-per-decade-is-provided-or-data-is-a-single-point
@butlerpd was this question for PR #536 after our discussion on the call? |
Converting this back to a draft, many of the checks for slit smearing a model are broken after the change. |
Oops .. I think so. Sorry and thanks for the catch |
@pkienzle it looks like the opencl tests are failing specifically. Would you be able to take a look? |
@pkienzle I'm circling back to this pull request during contributor camp; would you be able to help me out with the opencl tests that are failing with this change? |
… name is accurate
Determined that failing opencl tests is external of this PR and is more broadly affecting the master branch and all PR's. @pkienzle this fix will affect test_simple_interface in direct_model.py. As discussed, can you update the target values? |
2fc80e0
into
use-dQ-Data-slit-length-and-width-switched
This PR fixes the geometric extrapolation bug outlined in #568. The log spacing for the q extrapolation was calculated as the inverse term depending on whether or not points_per_decade was specified when calling the function, resulting in two different outputs.