-
Notifications
You must be signed in to change notification settings - Fork 273
Open
Labels
Description
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.