Skip to content

[Feature request]: Incorporate gempyor.distributions Into gempyor.inference_parameter.InferenceParameters #598

@TimothyWillard

Description

@TimothyWillard

Label

gempyor, inference

Priority Label

high priority

Is your feature request related to a problem? Please describe.

  1. There is a typo at

pdist=istribution_from_confuse_config(

and

pdist=istribution_from_confuse_config(

  1. The InferenceParameters class assumes that lower/upper bounds for modifiers are given via a/b, which limits them to truncated normal distributions.

Is your feature request related to a new application, scenario round, pathogen? Please describe.

No response

Describe the solution you'd like

  1. Correct the typo.
  2. The DistributionABC needs to be extended to expose a support property that can be used to obtain the lower/upper bounds as a tuple. Then this needs to be incorporated into the InferenceParameters class to replace the lines that look like:

lb=parameter_config["value"]["a"].get(float),
ub=parameter_config["value"]["b"].get(float),

Metadata

Metadata

Labels

gempyorConcerns the Python core.high priorityHigh priority.inferenceConcerns the parameter inference framework.

Projects

No projects

Milestone

No milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions