Skip to content
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.

Sign up for GitHub

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

Jump to bottom

EstimatorFailure exception class #26

Closed
wants to merge 34 commits into from
Closed

EstimatorFailure exception class #26

wants to merge 34 commits into from

Conversation

johnharveymath
Copy link

Rather than raising a bare Exception when an estimator doesn't return a value, I suggest a custom class EstimatorFailure.

kmyim and others added 30 commits March 12, 2024 00:12
@kmyim
added shell for flexnbhdestimator
37f70d5
@kmyim
Merge branch 'scikit-learn-contrib:master' into master
6ff9d6b
@kmyim
added meeting notes
54346d6
@kmyim
filled in more of flexnbhdestimator
c63dc85
@kmyim
basic eps neighbouhood + sparse distance matrix
c140f80
@kmyim
Integrated neighbourhood methods and filled in smooth and aggr
d0c20b3
@kmyim
extra comments to explain code
853d9f3
@kmyim
Corrected inputs and outputs of methods in FlexNbhdEstimator
0ba14fd
@Malinon
Merge pull request #1 from kmyim/ambrose_eps_nbhd
9184be3 
Ambrose eps nbhd
@Malinon
Update FlexNbhdEstimator class
4febc2b 
This commit:
1) Forwards metric parameter
2) Fix return description for get_neigh
@Malinon
Init commit with basic flex MLE
bade4c7
@Malinon
Revert "Init commit with basic flex MLE"
1cb69dc 
This reverts commit bade4c7.
@kmyim
started coding levina bickel
c30e357
@kmyim
updated abstract method
459f719
@Malinon
Merge pull request #2 from kmyim/flex_class_update
ca540d9 
Flex class update
@kmyim
modified flexnbhdestimator with revisions after MLE trial
23b7e2d
@kmyim
updated flex nbhd estimator with vector output of pw dim
79055c0
@kmyim
updated mle_basic to use radial dists from nbhds
08c0e82
@Malinon
Add basic tests for MLE
d70b6fc
@kmyim
Fixed typo in flex_mle
bd9ef99
@Malinon
Correct MLE tests and add general tests
99b5b2e 
This commits fix error in MLE_base specific tests and add general tests for all
FlexNbhdEstimators.
@kmyim
Merge pull request #3 from kmyim/basic_tests
365efd8 
Basic tests
@kmyim
Updated FlexNbhdEstimator with fit_pw
efdcc85
@kmyim
Removed self.is_fitted_pw from MLE_basic
73e1ddb
@kmyim
Merge branch 'ambrose_eps_nbhd' into basic_tests
9ad967e
@kmyim
Merge pull request #4 from kmyim/basic_tests
3de3e6e 
Basic tests
@kmyim
passed test_flexnbhdestimator.py tests
8985905
@kmyim
Passing almost all tests except test_when_eps_and_knn_almost_equivalent
95ddb6b
@Malinon
Fix flex MLE tests
4013114 
This commmit fix test_when_eps_and_knn_almost_equivalent and
test_on_exponential_seq_of_distances.
@kmyim
Merge pull request #5 from kmyim/fix_mle_tests
992845a 
Fix flex MLE tests
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@johnharveymath @kmyim @Malinon