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
[MRG+1] OPTICS fit uses the selected extract_method
parameter
#12087
Merged
GaelVaroquaux
merged 39 commits into
scikit-learn:master
from
adrinjalali:optics/choose_extractor
Mar 3, 2019
Merged
Changes from 8 commits
Commits
Show all changes
39 commits
Select commit
Hold shift + click to select a range
3ab2e63
add extract method parameter
adrinjalali 7190098
change fancy to sqlnk
adrinjalali eb12095
test for invalid extract_method value
adrinjalali 58913d8
test easy dbscan
adrinjalali 4b06ba8
pep8
adrinjalali c9819d2
add extract_sqlnk
adrinjalali 61b440f
mention which parameter is used for which extact method
adrinjalali 9a65082
add tests for extract methods with no params given
adrinjalali bff04ee
merge master
adrinjalali 36b8126
Merge remote-tracking branch 'upstream/master' into optics/choose_ext…
adrinjalali 0d6006d
Merge branch 'master' into optics/choose_extractor
adrinjalali 9a63e27
Add the reference for the SQLNK method.
adrinjalali 456bd58
make extract_optics_dbscan and extract_optics_sqlnk public
adrinjalali 36da998
merge upstream
adrinjalali 841b19a
fix references, sync docstrings
adrinjalali 80f3a13
Merge remote-tracking branch 'upstream/master' into optics/choose_ext…
adrinjalali 4afc4f6
fix more morege conflicts
adrinjalali 9811585
Merge remote-tracking branch 'upstream/master' into optics/choose_ext…
adrinjalali fbaf24b
reorganize the code and get it closer to what we want
adrinjalali 8a9346a
pep8
adrinjalali 7cfb68c
Merge remote-tracking branch 'upstream/master' into optics/choose_ext…
adrinjalali 4a7821e
remove sqlnk, make compute_optics_graph public
adrinjalali 3265c7d
pep8
adrinjalali 79e12cb
remove unused reference
adrinjalali fa3f89d
make tests pass
adrinjalali 751b33d
fix docstrings
adrinjalali d4ac534
fix removal of removed parameter
adrinjalali 8b3c4fb
add back min_cluster size, xi needs it
adrinjalali 6eead2f
Merge remote-tracking branch 'upstream/master' into optics/choose_ext…
adrinjalali 1377cb0
pep8
adrinjalali 48d7702
fix docstring issue
adrinjalali d06e5ff
address Hanmin's comments
adrinjalali 4230567
Merge remote-tracking branch 'upstream/master' into optics/choose_ext…
adrinjalali fd3e16a
remove core_sample_indices_
adrinjalali 98a34a9
fix comment
adrinjalali c926c9c
pep8
adrinjalali a8a40ca
apply comments, remove example
adrinjalali bb99ab0
add public functions to classes.rst and __init__
adrinjalali 43447a6
address Joel's comments
adrinjalali File filter
Filter by extension
Conversations
Failed to load comments.
Jump to
Jump to file
Failed to load files.
Diff view
Diff view
There are no files selected for viewing
This file contains 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 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
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.
sqlnk is not mentioned in the repo. The user has little way to identify its meaning.