You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
As of today, the user can only use one likelihood model in inversion of real data : the one mixing additive and multiplicative noises, and censorship (beetroots.modelling.likelihoods.approx_censored_add_mult.py).
In some contexts, a user could want to remove the additive noise or the multiplicative noise, or not to include censorship in the model.
There are already classes implementing these likelihoods in beetroots.modelling.likelihoods, it would be nice to be able to use them in inversion.
Or some parameters could become optional (that is, absent from the input file, or preferably set to none), such as the filename_err (for the additive noise standard deviation (std)) or the sigma_m_float_linscale (for the multiplicative noise std).
In the SimulationRealDataNN class (in beetroots.simulations.astro.real_data.py) : methods setup(), that calls
the setup_observation() method from the SimulationRealData class, to import the data, additive noise std, etc.
the setup_posteriors() method from the SimulationMySampler class, to select the Likelihood class.
In this case : For now, the posterior is given a name model_name = f"mixing_{i}" (in SimulationMySampler.setup_posteriors()) that depend on the likelihood class. This could be generalized for other kinds of Likelihood classes, for clearer understanding of the sampling outputs.
As of today, the user can only use one likelihood model in inversion of real data : the one mixing additive and multiplicative noises, and censorship (
beetroots.modelling.likelihoods.approx_censored_add_mult.py
).In some contexts, a user could want to remove the additive noise or the multiplicative noise, or not to include censorship in the model.
There are already classes implementing these likelihoods in
beetroots.modelling.likelihoods
, it would be nice to be able to use them in inversion.Where to apply changes:
input.yaml
, there could be parametersOr some parameters could become optional (that is, absent from the input file, or preferably set to
none
), such as thefilename_err
(for the additive noise standard deviation (std)) or thesigma_m_float_linscale
(for the multiplicative noise std).SimulationRealDataNN
class (inbeetroots.simulations.astro.real_data.py
) : methodssetup()
, that callssetup_observation()
method from theSimulationRealData
class, to import the data, additive noise std, etc.setup_posteriors()
method from theSimulationMySampler
class, to select the Likelihood class.In this case : For now, the posterior is given a name
model_name = f"mixing_{i}"
(inSimulationMySampler.setup_posteriors()
) that depend on the likelihood class. This could be generalized for other kinds of Likelihood classes, for clearer understanding of the sampling outputs.The choice of the Likelihood class to use:
->
MixingModelsLikelihood
(frombeetroots.modelling.likelihoods.approx_censored_add_mult.py
)-> for now,
MixingModelsLikelihood
(need to define a simpler class ?)->
CensoredLogNormalLikelihood
(frombeetroots.modelling.likelihoods.log_normal_censored.py
)->
LogNormalLikelihood
(frombeetroots.modelling.likelihoods.log_normal.py
)->
CensoredGaussianLikelihood
(frombeetroots.modelling.likelihoods.gaussian_censored.py
)->
GaussianLikelihood
(frombeetroots.modelling.likelihoods.gaussian.py
)The text was updated successfully, but these errors were encountered: