-
Notifications
You must be signed in to change notification settings - Fork 229
[MRG] Add preprocessor option #117
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
perimosocordiae
merged 122 commits into
scikit-learn-contrib:new_api_design
from
wdevazelhes:feat/preprocessor
Dec 14, 2018
Merged
Changes from all commits
Commits
Show all changes
122 commits
Select commit
Hold shift + click to select a range
7b3d739
WIP create MahalanobisMixin
f21cc85
ENH Update algorithms with Mahalanobis Mixin:
6f8a115
Merge branch 'new_api_design' into feat/mahalanobis_class
f9e3c82
FIX: add missing import
1a32c11
FIX: update sklearn's function check_no_fit_attributes_set_in_init to…
d0f5019
FIX: take function ``_get_args`` from scikit-learn's PR https://githu…
eba2a60
ENH: add transformer_ attribute and improve docstring
b5d966f
WIP: move transform() in BaseMetricLearner to transformer_from_metric…
ee0d1bd
WIP: refactor metric to original formulation: a function, with result…
6b5a3b5
WIP: make all Mahalanobis Metric Learner algorithms have transformer_…
6eb65ac
ENH Add score_pairs function
35ece36
TST add test on toy example for score_pairs
dca6838
ENH Add embed function
3254ce3
FIX fix error in slicing of quadruplets
e209b21
FIX minor corrections
abea7de
FIX minor corrections
65e794a
FIX fix PEP8 errors
12b5429
FIX remove possible one-sample scoring from docstring for now
eff278e
REF rename n_features_out to num_dims to be more coherent with curren…
810d191
MAINT: Adress https://github.com/metric-learn/metric-learn/pull/96#pu…
585b5d2
ENH: Add check_tuples
af0a3ac
FIX: fix parenthesis
f1dd4c2
ENH: First commit adding a preprocessor
47fbf46
ENH: Improve check_tuples with more comments and deal better with ens…
8eb419a
STY: remove unexpected spaces
033da60
FIX: Raise more appropriate error message
6fae262
FIX: fix string formatting and refactor name and context to use conte…
758d4cc
FIX: only allow 2D if preprocessor and 3D if not preprocessor
deb6d5d
FIX: put format arguments in the right order
01ee081
MAINT: better to say the preprocessor than a preprocessor in messages
92f9651
FIX: numeric should be default if NO preprocessor
4c41c37
FIX: preprocessor argument has to be a boolean (before this change wa…
4b7e89b
FIX: fix preprocessor argument passed in check_tuples in base_metric
609b80e
MAINT: say a preprocessor rather than the preprocessor
764e2f1
Merge branch 'new_api_design' into feat/preprocessor
4342660
DOC: fix docstring of t in check_tuples
e50cbae
MAINT: make error messages better by only printing presence of prepro…
56838b4
TST: Add tests for check_tuples
9f05c24
TST: simplify tests by removing the test for messages with the estima…
12ce8ac
STY: remove unnecessary parenthesis
62a989c
FIX: put back else statement that probably was wrongfully merged
33c5d8b
TST: add tests for weakly supervised estimators and preprocessor that…
5eba5fa
TST: add tests for preprocessor
42b34e0
FIX: remove redundant metric and transformer function, wrongly merged
a380cd3
MAINT: rename format_input into preprocess_tuples and input into tuples
54d1710
MAINT: fixes and enhancements
3b716f0
MAINT: mutualize check_tuples
bac835e
MAINT: refactor SimplePreprocessor into ArrayIndexer
2b0f495
MAINT: improve check_tuples and tests
6ae7ba5
TST: add random seed for _Supervised classes
a1c8a67
TST: Adapt test pipeline
735f975
TST: fix test_progress_message_preprocessor_tuples by making func ret…
3586208
Remove deprecated cross_validation import and put model_selection ins…
51d7e07
WIP replace checks by unique check_input function
27e215c
Fixes some tests:
e23554f
TST: Cherry pick from new sklearn version ac0e230000556b7c413e08b77d8…
9ded846
FIX: get changes from master to pass test iris for NCA
50514bc
FIX fix tests that were failing due to the error message
96b58b4
TST: fix test_check_input_invalid_t that changed since we test t at t…
9cab2ee
TST fix NCA's iris test taking code from master
069b8e2
FIX fix tests:
e8d8795
FIX fix previous modification that removed self.X_ but was modifying …
7c539c7
FIX ensure at least 2d only for checking the metric because after che…
f801fae
STY: Fix PEP8 violations
b38b223
MAINT: Refactor error messages with the help of numerical codes
cc6d661
MAINT: mutualize check_preprocessor and check_input for every estimator
192a042
FIX: remove format_map for python2.7 compatibility
0328941
DOC: Add docstring for check_input and fix some bugs
00078c2
DOC: add docstrings
1ded46a
MAINT: Removing changes not related to this PR, and fixing previous p…
52a1aec
STY: Fix PEP8 errors
072e834
STY: fix indent problems
80929e2
Fixing docstring spaces
40a0172
DOC: add preprocessor docstring when missing
5a3af89
STY: PEP8 fixes
968e36e
MAINT: refactor the global check function into _prepare_input
e5b5f57
FIX: fix quadruplets scoring and delete useless comments
84c9d56
MAINT: remove some enhancements to be coherent with previous code and…
7605fa4
MAINT: Improve test messages
69de333
MAINT: reorganize tests
b29a555
FIX: fix typo in LMNN shogun and clean todo for the equivalent code i…
29be5e2
MAINT: Rename inputs and input into input_data
9d849f7
STY: add backticks to None
d9ba29e
MAINT: add more detailed comment of first checks and remove old comment
c1dcc1f
MAINT: improve comments for checking num_features
6ccd25d
MAINT: Refactor t into tuple_size
2d96eda
MAINT: Fix small PEP8 error
4b50ff1
MAINT: FIX remaining t into tuple_size and replace hasattr if None by…
f69f135
MAINT: remove misplaced comment
192d208
MAINT: Put back/add docstrings for decision_function/predict
2c09d9a
MAINT: remove unnecessary ellipsis and upadate docstring of decision_…
1b7e55f
Add comments in LMNN for arguments useful for the shogun version that…
8b600be
MAINT: Remove useless mock_preprocessor
e4468d0
MAINT: Remove useless loop
a1c95fa
MAINT: refactor test_dict_unchanged
d95b22a
MAINT: remove _get_args copied from scikit-learn and replace it by an…
bc06f8f
MAINT: Fragment check_input by extracting blocks into check_input_cla…
9260a8e
MAINT: ensure min_samples=2 for supervised learning algorithms (we sh…
93c3e34
ENH: Return custom error when some error is due to the preprocessor
f2d0cd7
MAINT: Refactor algorithms preprocessing steps
02e82ff
MAINT: finish the work of the previous commit
99206b3
TST: add test for cross-validation: comparison of manual cross-val an…
8ee08b8
ENH: put y=None by default in LSML for better compatibility. This als…
784f697
ENH: add error message when type of inputs is not some expected type
c46bbe1
TST: add test that checks that 'classic' is the default behaviour
39a7256
TST: remove unnecessary conversion to vertical vector of y
082bca5
FIX: remove wrong condition hasattr 'score' at top of loop
6abbcd6
MAINT: Add comment to explain why we return twice X for build_regress…
f0a1dc2
ENH: improve test for preprocessor and return error message if the gi…
ab5f2e3
FIX: fix wrong type_of_inputs in a test
5324e85
FIX: deal with the case where preprocessor is None
48bce7d
WIP refactor build_dataset
c0cc882
MAINT: refactor bool preprocessor to with_preprocessor
0b3e58a
FIX: fix build_pairs and build_quadruplets because 'only named argume…
fbd7242
STY: fix PEP8 error
148012e
MAINT: mututalize test_same_with_or_without_preprocessor_tuples and t…
8c5675b
TST: give better names in test_same_with_or_without_preprocessor
30061e4
MAINT: refactor list_estimators into metric_learners
e241c28
TST: uniformize names input_data - tuples, labels - y
b6d7de7
FIX: fix build_pairs and build_quadruplets
a44e29a
MAINT: remove forgotten code duplication
2db6410
MAINT: address https://github.com/metric-learn/metric-learn/pull/117#…
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
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.
Uh oh!
There was an error while loading. Please reload this page.