Skip to content

Remove ABCSMC.__init__(summary_statistics) #426

Open
@yannikschaelte

Description

For major refactoring and the introduction of learned summary statistics, it would make sense to remove this Callable, which in most cases should anyway be an id mapping. Nothing is effectively lost, as the model can just return the "summary statistics / raw data" which are saved in the database. Only problem would be if this feature is used by many applicants. Opinions?

The summary statistics function would then mid-term be replaced by actual summary statistics, which are however not stored in the database, and act differently.

Activity

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

Metadata

Assignees

No one assigned

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions