Skip to content

templates for ctapipe-quickstart should not be the files used for testing #2796

@kosack

Description

@kosack

Describe the bug

ctapipe-quickstart is meant to give the user a nice starting point to do an analysis, but right now its template files are used also for unit tests, which means they contain not the most useful examples for users, but rather settings that work fast for unit tests. Those are two different goals, so we should separate the configurations used in testing from those given out by quickstart.

To Reproduce

$ ctapipe-quickstart --name "a user" --email "a@b.com" --org "dummy"  
$ cat Work/
...
  DispReconstructor:
    # prefix:  # Add a prefix of the output here, if you want to apply multiple
               # DispReconstructors on the same file (e.g. for comparing different settings)

    # How many cores to use. Overwrites model config
    n_jobs: -1

    # All regression algorithms in scikit-learn are supported
    # (https://scikit-learn.org/stable/modules/classes.html)
    norm_cls: ExtraTreesRegressor
    norm_config:
      n_estimators: 10
      max_depth: 10
...

Using a ExtraTreesRegressor is definitely not the recommended method to use here.

Expected behavior

ctapipe-quickstart give very good default values that are what a user should use for a reasonable analysis. It doesn't have to be perfect, but should also not lead to very poor science performance.

Metadata

Metadata

Assignees

No one assigned

    Labels

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions