Skip to content

Homogenise NonLinearPerturbations input #543

Description

@LisaGoh

⚠️🐛 Bug Summary

Currently, the input of CLASSNonLinearPerturbations is given by

def __init__(
        self,
        background: Background,
        linearperturbations: Optional[object],
        redshifts: np.ndarray,
        nonlinear_model: Optional[str] = None,
        hmcode_version: Optional[str] = None,
    ):

which does not allow for the passing of the log10TAGN parameter, unlike in CAMBNonLinearPerturbations or HMemuNonLinearPerturbations.

🔗 Optional: Additional Context

Proposed solution:

Change the input of CLASSNonLinearPerturbations to match the others:

def __init__(
        self,
        background: Background,
        linearperturbations: Optional[object],
        redshifts: np.ndarray,
        nonlinear_model: Optional[str] = None,
        log10TAGN: Optional[float] = None,
    ):

and change the lines 477-478 to:

        if log10TAGN is not None:
            self.interface_args["CLASSparams"]["hmcode_version"] = "2020_baryonic_feedback"
            self.interface_args["CLASSparams"]["log10T_heat_hmcode"] = log10TAGN

Activity

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

Labels

background-perturbationsTasks related to Background and PerturbationsbugSomething isn't working

Type

No type

Projects

No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions