-
-
Notifications
You must be signed in to change notification settings - Fork 5.1k
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
ENH/MAINT: qmc.LatinHypercube
: deprecate centered with scramble
#17152
Conversation
Co-authored-by: Matt Haberland <mhaberla@calpoly.edu>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks Matt! I did the updates.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This looks good on the deprecation side of things now
Looks like it's time for a mailing list post. I'll merge a week after that. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Last round of suggestions.
Co-authored-by: Matt Haberland <mhaberla@calpoly.edu>
Thanks Matt, just sent an email: https://mail.python.org/archives/list/scipy-dev@python.org/thread/PGUHB7L3ZKCLAG2PQ2MAOEFAOTJR36IK/ And thanks @j-bowhay for the review 👍 |
Co-authored-by: Matt Haberland <mhaberla@calpoly.edu>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The email went out about a week ago. I'll merge shortly if we don't hear otherwise. Oops I got mixed up on when it was sent. I'll merge later this week, though.
@h-vetinari another for #15765 |
@j-bowhay Thanks for the heads up. Hopefully I added it correctly. |
…scramble` (scipy#17152) * ENH/MAINT: `qmc.LatinHypercube`: deprecate `centered`; replace with `scramble`
We can simplify the interface of
qmc.LatinHypercube
and make it consistent with other engines by removing thecentered
parameter and addingscramble
. Not centering, the default, can be seen as scrambling. Which is the default for other engines. Hence there is an easy deprecation path with added consistency.In this PR:
centered
in favour ofscramble
. Scrambling is about "adding some randomness" to a sample. As not scrambling might be interpreted as having a constant output, I added a note explaining that it's not the case.d<0
inQMCEngine
. We were only testing for integers