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

Truncnorm translation layer #162

Open
aimalz opened this issue Mar 30, 2023 · 0 comments
Open

Truncnorm translation layer #162

aimalz opened this issue Mar 30, 2023 · 0 comments
Labels
enhancement New feature or request

Comments

@aimalz
Copy link
Collaborator

aimalz commented Mar 30, 2023

https://docs.scipy.org/doc/scipy/reference/generated/scipy.stats.truncnorm.html is set up counterintuitively such that the limits the user provides to instantiate the frozen rv_continuous object are scaled to the standard normal rather than the physical dimension over which the distribution is defined. This is necessary for the way vectorization is implemented for all the rv_continuous methods, but it causes problems if users provide those arguments in the physical dimensions rather than converting to the standard normal. I wonder if we can wrap something around it that enables users to create the object with physical limits and have the conversion to internal units, knowing that the actual use case will always be the one that can't take advantage of vectorization anyway so might as well be intuitive to access.

@aimalz aimalz added the enhancement New feature or request label Mar 30, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

1 participant