Skip to content

ParameterModel drops non-logarithmic parameters if one of the ensembles uses a logarithmic distribution #1221

@asnyv

Description

@asnyv

The linked code drops the non-logarithmic column and renames the logarithmic ones. If someone is testing different distributions (logarithmic and non-logarithmic), the non-logarithmic distributions will be lost. I suggest that we short term skip dropping the non-logarithmic even though it creates some duplicates. Could be better to e.g. only keep the non-logarithmic which exists independent of chosen distribution, but instead have an option in the gui to use a logarithmic axis.

# Keep only LOG parameters
log_params = [param for param in self._dataframe if param.startswith("LOG10_")]
self._dataframe = self._dataframe.drop(
columns=[param.replace("LOG10_", "") for param in log_params]
)
self._dataframe = self._dataframe.rename(
columns={col: f"{col} (log)" for col in log_params}
)

Metadata

Metadata

Assignees

No one assigned

    Labels

    bug 🐛Something isn't working

    Type

    No type

    Projects

    Status

    Backlog 📝

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions