Skip to content

Commit

Permalink
Explicitly specify domain for AffineTransform in ScaledLogitTransform (
Browse files Browse the repository at this point in the history
  • Loading branch information
dylanhmorris authored Aug 14, 2024
1 parent 0fc0c8d commit 85726bd
Showing 1 changed file with 7 additions and 1 deletion.
8 changes: 7 additions & 1 deletion model/src/pyrenew/transformation/builtin.py
Original file line number Diff line number Diff line change
Expand Up @@ -3,6 +3,7 @@
"""

import numpyro.distributions.transforms as nt
from numpyro.distributions import constraints


def ScaledLogitTransform(
Expand All @@ -27,5 +28,10 @@ def ScaledLogitTransform(
- numpyro.distributions.transforms.SigmoidTransform().inv
"""
return nt.ComposeTransform(
[nt.AffineTransform(0.0, 1.0 / x_max), nt.SigmoidTransform().inv]
[
nt.AffineTransform(
0.0, 1.0 / x_max, domain=constraints.interval(0, x_max)
),
nt.SigmoidTransform().inv,
]
)

0 comments on commit 85726bd

Please sign in to comment.