-
Notifications
You must be signed in to change notification settings - Fork 267
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
OutlierDetector components #2604
Conversation
This comment has been minimized.
This comment has been minimized.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks nice, a couple of naming/doc suggestions, but otherwise is good to me.
This comment has been minimized.
This comment has been minimized.
This PR needs to be rebased and "outlier files" have to moved to the |
cc91abf
to
d3fda92
Compare
This comment has been minimized.
This comment has been minimized.
…cription removed 'Based' in the class names
This comment has been minimized.
This comment has been minimized.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just some minor changes to make. .
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
Analysis Details0 IssuesCoverage and DuplicationsProject ID: cta-observatory_ctapipe_AY52EYhuvuGcMFidNyUs |
This PR added outlier detection approaches. The sub classes can be applied to any astropy column of shape (n_entries, n_channels, n_pix) returning an outlier mask of the same shape. The code utilizing this components later need to handle multiple instances and therefore combine the returning outlier masks.
–––
Related: #2542