-
Notifications
You must be signed in to change notification settings - Fork 4
Labels
gempyorConcerns the Python core.Concerns the Python core.high priorityHigh priority.High priority.inferenceConcerns the parameter inference framework.Concerns the parameter inference framework.
Description
Label
gempyor, inference
Priority Label
high priority
Is your feature request related to a problem? Please describe.
- There is a typo at
pdist=istribution_from_confuse_config( |
and
pdist=istribution_from_confuse_config( |
- The
InferenceParameters
class assumes that lower/upper bounds for modifiers are given viaa
/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
- Correct the typo.
- The
DistributionABC
needs to be extended to expose asupport
property that can be used to obtain the lower/upper bounds as a tuple. Then this needs to be incorporated into theInferenceParameters
class to replace the lines that look like:
flepiMoP/flepimop/gempyor_pkg/src/gempyor/inference_parameter.py
Lines 63 to 64 in fda910d
lb=parameter_config["value"]["a"].get(float), | |
ub=parameter_config["value"]["b"].get(float), |
Metadata
Metadata
Assignees
Labels
gempyorConcerns the Python core.Concerns the Python core.high priorityHigh priority.High priority.inferenceConcerns the parameter inference framework.Concerns the parameter inference framework.