-
-
Notifications
You must be signed in to change notification settings - Fork 25.8k
[MRG+1] Novelty detection for LOF #10700
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
Merged
glemaitre
merged 24 commits into
scikit-learn:master
from
albertcthomas:novelty_for_lof
Jul 19, 2018
Merged
Changes from all commits
Commits
Show all changes
24 commits
Select commit
Hold shift + click to select a range
78440ed
start novelty detection for lof
albertcthomas 3e08ea6
use ValueError and always raise exception before return in methods
albertcthomas 3a5fe5c
fix outlier detection common tests
albertcthomas 45988d8
add tests for novelty ValueError
albertcthomas 60eb434
update plot_lof to plot_lof for outlier detection
albertcthomas 536b73d
[circle full] novelty detection example
albertcthomas c74a570
fix travis and outlier_detection example
albertcthomas ae49a51
add 2 components gaussian mixture with same variance in plot_anomaly_…
albertcthomas 9a3ec47
[doc build] rm plot_outlier_detection and update outlier detection doc
albertcthomas 6fae094
[doc build] sc in doc + add test for training scores
albertcthomas 223bd95
[doc build] sc and add table for LOF behavior in doc
albertcthomas 2510258
address review
albertcthomas 04e273a
remove comma in plot docstring
albertcthomas e34a93d
move details on comparison between outlier detection estimators from …
albertcthomas 7c8b28e
sc in beginning of novelty and outlier detection doc + add distinctio…
albertcthomas a3f84b4
sc in doc
albertcthomas 8f9e88a
use properties to make method disappear depending on novelty value
albertcthomas c3826ae
remove novelty=True statements in estimator_checks and use check_esti…
albertcthomas cb800ff
add non regression tests checking availability of prediction methods …
albertcthomas 3ae0152
add entry in whatsnew
albertcthomas 2a4e69f
pass on docstrings
agramfort 15dfb29
remove decision_function in outlier examplea and use negative_outlier…
albertcthomas a7af1fd
update whatsnew with author
albertcthomas 9e59491
ignore deprecationwarning for new lof tests
albertcthomas File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file was deleted.
Oops, something went wrong.
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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.
Is there any sensible way to use it in a pipeline then?
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.
Do you mean what would be the correct way to use LOF in a Pipeline?