-
Notifications
You must be signed in to change notification settings - Fork 3
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
Move support
and insupport
here from Distributions
#11
Comments
|
Well I would say these |
Hm, yes, that's true. But maybe |
Hm - @cscherrer , for a measure, as there's not just one density, is asking about the support actually the right question? Is it about whether a point is a member of the set that the measure is defined on, or whether the point contributes mass? |
Also, if we go with the plan that Distributions will depend on MeasureBase, then MeasureBase could own |
It's definitely tricky for embeddings, usually in those cases we're in the support by construction, or sometimes we project to it. DomainSets is very cool, we've looked into it for MeasureTheory. My biggest concerns had been the nonstandard |
Oops, I had missed this. We have some guarantees about (local) absolute continuity by construction. I think we really should have insupport(d, x) == logdensityof(d, x) > -Inf In some cases this might be tricky to make efficient, so it could be useful to also have an overapproximation of the support, with a guarantee that values in that overapproximation won't lead to errors (no logs of negatives, etc) |
Maybe in that case |
I don't think this would useful since it would lead to strange results (that even depend on the type of julia> logdensityof(Normal(0, 1e-100), 1e100)
-Inf
julia> logdensityof(Normal(0, 1f-10), 1f10)
-Inf32
julia> logdensityof(Normal(0, 1e-10), 1e10)
-5.0e39 |
Good point @devmotion , I think it's pretty clear we don't want to be considered outside the support due to underflow. |
I'd like to have access to these for MeasureBase, and depending on Distributions seems backwards. Any thoughts on moving these to DensityInterface? Anything else that should move here from Distributions?
The text was updated successfully, but these errors were encountered: