Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Added HPD calculation to ensemble #1431

Merged
merged 17 commits into from
Sep 30, 2024
Merged
Show file tree
Hide file tree
Changes from 9 commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
91 changes: 91 additions & 0 deletions pypesto/ensemble/ensemble.py
Original file line number Diff line number Diff line change
Expand Up @@ -555,6 +555,7 @@ def __init__(
def from_sample(
result: Result,
remove_burn_in: bool = True,
f_quantile: float = None,
chain_slice: slice = None,
x_names: Sequence[str] = None,
lower_bound: np.ndarray = None,
Expand All @@ -571,6 +572,10 @@ def from_sample(
remove_burn_in:
Exclude parameter vectors from the ensemble if they are in the
"burn-in".
f_quantile:
A form of relative cutoff. Exclude parameter vectors, for which the
(non-normalized) posterior value is not within the f_quantile best
PaulJonasJost marked this conversation as resolved.
Show resolved Hide resolved
values.
chain_slice:
Subset the chain with a slice. Any "burn-in" removal occurs first.
x_names:
Expand Down Expand Up @@ -599,9 +604,21 @@ def from_sample(
geweke_test(result)
burn_in = result.sample_result.burn_in
x_vectors = x_vectors[burn_in:]
else:
burn_in = 0
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Usually just write burn_in = 0 above the if statement.


# added cutoff
if f_quantile is None:
pass
else:
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
# added cutoff
if f_quantile is None:
pass
else:
# added cutoff
if f_quantile is not None:

x_vectors = calculate_hpd(
result=result, burn_in=burn_in, ci_level=f_quantile
PaulJonasJost marked this conversation as resolved.
Show resolved Hide resolved
)

if chain_slice is not None:
x_vectors = x_vectors[chain_slice]
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

A general remark, but it is not quite clear whether this is the desired order, it might be that we want to first slice and then calculate hpd. But that is only a minor thing here i guess.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

In principle, I would rather calculate the HPD first and then take a slice, as the HPD should become more precise that way; computationally, it's not an issue. But if there are other reasons to change the order, that would also be fine with me.

x_vectors = x_vectors.T

return Ensemble(
x_vectors=x_vectors,
x_names=x_names,
Expand Down Expand Up @@ -1253,3 +1270,77 @@ def calculate_cutoff(

range = chi2.ppf(q=percentile / 100, df=df)
return fval_opt + range


def calculate_hpd(
result: Result,
burn_in: int = 0,
ci_level: float = 0.95,
):
"""
Calculate Highest Posterior Density (HPD) samples of pypesto sampling result.
PaulJonasJost marked this conversation as resolved.
Show resolved Hide resolved

The HPD is calculated for a user-defined credibility level (ci_level). The
PaulJonasJost marked this conversation as resolved.
Show resolved Hide resolved
HPD includes all parameter vectors with a (non-normalized) posterior
probability that is higher than the lowest 1-ci_level %
PaulJonasJost marked this conversation as resolved.
Show resolved Hide resolved
posterior probability values.

Parameters
----------
result:
The optimization result from which to create the ensemble.
PaulJonasJost marked this conversation as resolved.
Show resolved Hide resolved
burn_in:
Burn in index that is cut off before HPD is calculated.
ci_level:
Credibility level of the resulting HPD. 0.95 corresponds to the 95% CI.
Values between 0 and 1 are allowed.
PaulJonasJost marked this conversation as resolved.
Show resolved Hide resolved

Returns
-------
The HPD parameter vector.
PaulJonasJost marked this conversation as resolved.
Show resolved Hide resolved
"""
if ci_level < 0 or ci_level > 1:
PaulJonasJost marked this conversation as resolved.
Show resolved Hide resolved
raise ValueError(
f"ci_level={ci_level} is not valid. Choose 0<=ci_level<=1."
)
# get names of chain parameters
param_names = result.problem.get_reduced_vector(result.problem.x_names)

# Get converged parameter samples as numpy arrays
chain = np.asarray(result.sample_result.trace_x[0, burn_in:, :])
neglogpost = result.sample_result.trace_neglogpost[0, burn_in:]
indices = np.arange(
burn_in, len(result.sample_result.trace_neglogpost[0, :])
)

# create df first, as we need to match neglogpost to the according parameter values
pd_params = pd.DataFrame(chain, columns=param_names)
pd_fval = pd.DataFrame(neglogpost, columns=["neglogPosterior"])
pd_iter = pd.DataFrame(indices, columns=["iteration"])

params_df = pd.concat(
[pd_params, pd_fval, pd_iter], axis=1, ignore_index=False
)

# get lower neglogpost bound for HPD
# sort neglogpost values of MCMC chain without burn in
neglogpost_sort = np.sort(neglogpost)

# Get converged chain length
chain_length = len(neglogpost)

# most negative ci percentage samples of the posterior are kept to get the according HPD
neglogpost_lower_bound = neglogpost_sort[int(chain_length * (ci_level))]

# cut posterior to hpd
hpd_params_df = params_df[
params_df["neglogPosterior"] <= neglogpost_lower_bound
]

# convert df to ensemble vector
hpd_params_df_vals_only = hpd_params_df.drop(
columns=["iteration", "neglogPosterior"]
)
hpd_ensemble_vector = hpd_params_df_vals_only.to_numpy()

return hpd_ensemble_vector
47 changes: 47 additions & 0 deletions test/base/test_ensemble.py
Original file line number Diff line number Diff line change
Expand Up @@ -6,6 +6,7 @@

import pypesto
import pypesto.optimize as optimize
import pypesto.sample as sample
from pypesto.C import AMICI_STATUS, AMICI_T, AMICI_Y, MEAN, WEIGHTED_SIGMA
from pypesto.engine import MultiProcessEngine
from pypesto.ensemble import (
Expand Down Expand Up @@ -224,3 +225,49 @@ def post_processor(amici_outputs, output_type, output_ids):
progress_bar=False,
)
return ensemble_prediction


def test_hpd_calculation():
"""Test the calculation of Highest Posterior Density (HPD)."""
problem = create_petab_problem()

sampler = sample.AdaptiveMetropolisSampler(
options={"show_progress": False}
)

result = optimize.minimize(
problem=problem,
n_starts=3,
progress_bar=False,
)

result = sample.sample(
problem=problem,
sampler=sampler,
n_samples=100,
result=result,
)

# Manually set up sample (only for testing)
burn_in = 1
result.sample_result.burn_in = burn_in
result.sample_result.trace_neglogpost[0][1:] = np.random.permutation(
np.arange(len(result.sample_result.trace_neglogpost[0][1:]))
)

hpd_ensemble = Ensemble.from_sample(
result=result, remove_burn_in=True, f_quantile=0.95
)

expected_length = (
int((result.sample_result.trace_x[0][burn_in:].shape[0]) * 0.95) + 1
)
# Check that the HPD parameters have the expected shape
assert hpd_ensemble.x_vectors.shape == (problem.dim, expected_length)
x_indices = np.where(result.sample_result.trace_neglogpost[0][1:] <= 95)[0]
assert np.all(
[
np.any(np.all(x[:, None] == hpd_ensemble.x_vectors, axis=0))
for x in result.sample_result.trace_x[0][burn_in:][x_indices]
]
)