forked from kazewong/jim
-
Notifications
You must be signed in to change notification settings - Fork 3
Open
Description
In src/jimgw/core/prior.py, we have a StandardNormalDistribution prior and a GaussianPrior. They are a bit inconsistent: they subclass different priors and there is no relation between them. We should make sure both names are supported, and that the standard Gaussian prior is a specific instance of the Gaussian prior, with mu=0 and sigma=1, to avoid redundancy.
Also, for user-friendliness, we should make sure both names are supported, similar to bilby, i.e. something like
class Normal(Gaussian):
"""A synonym for the Gaussian distribution. """but then adapting the names and classes for Jim.
(Note for devs: this is a very small issue to test out the automated Claude workflow).
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
No labels