-
Notifications
You must be signed in to change notification settings - Fork 33
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Use SOM peaklets classification by default #1471
Conversation
Do we want the default plugins to be called |
I do not like it, either. Any suggestions? :) |
What is the problem with simply naming the class |
E.g, But maybe rename |
Before you submit this PR: make sure to put all operations-related information in a wiki-note, a PR should be about code and is publicly accessible
Depends on #1472
What does the code in this PR do / what does it improve?
SIGNIFICANT CHANGE
BREAKING CHANGE
Can you briefly describe how it works?
Use SOM classification (https://xe1t-wiki.lngs.infn.it/doku.php?id=xenon:xenonnt:lsanchez:som_summary_note) by default.
Following #1269 and #1300.
Can you give a minimal working example (or illustrate with a figure)?
Please include the following if applicable:
Notes on testing
All italic comments can be removed from this template.