Skip to content

Add scikit-learn-contrib/stability-selection and improve randomized_l1 transformers#308

Open
teabolt wants to merge 7 commits into
TeamHG-Memex:masterfrom
teabolt:sklearn-contrib-stability-selection
Open

Add scikit-learn-contrib/stability-selection and improve randomized_l1 transformers#308
teabolt wants to merge 7 commits into
TeamHG-Memex:masterfrom
teabolt:sklearn-contrib-stability-selection

Conversation

@teabolt

@teabolt teabolt commented Apr 29, 2019

Copy link
Copy Markdown
Contributor

Closes #305.

Coverage should increase once scikit-learn 0.21 and scikit-learn-contrib/stability-selection are tested against (could modify the tox config for this?).

If more use cases come up for functions like register_notnone and instantiate_notnone, they could be moved to base_utils.py or similar.

@codecov-io

codecov-io commented Apr 29, 2019

Copy link
Copy Markdown

Codecov Report

Merging #308 into master will decrease coverage by 0.02%.
The diff coverage is 83.33%.

@@            Coverage Diff             @@
##           master     #308      +/-   ##
==========================================
- Coverage   97.09%   97.07%   -0.03%     
==========================================
  Files          44       44              
  Lines        2856     2868      +12     
  Branches      542      543       +1     
==========================================
+ Hits         2773     2784      +11     
- Misses         45       46       +1     
  Partials       38       38
Impacted Files Coverage Δ
eli5/sklearn/transform.py 93.61% <83.33%> (-0.67%) ⬇️

@teabolt

teabolt commented Apr 29, 2019

Copy link
Copy Markdown
Contributor Author

Re: two test parametrizations for RandomizedLasso, one for scikit-learn < 0.19 and another for scikit-learn >= 0.19. It might be possible to reduce these to just one test by passing certain parameters to the RandomizedLasso so that the same features are selected in both scikit-learn versions. See also history of randomized_l1 https://github.com/scikit-learn/scikit-learn/commits/0.20.X/sklearn/linear_model/randomized_l1.py for what might be causing the difference in selected features.

@codecov-commenter

codecov-commenter commented Apr 6, 2026

Copy link
Copy Markdown

⚠️ Please install the 'codecov app svg image' to ensure uploads and comments are reliably processed by Codecov.

Codecov Report

❌ Patch coverage is 83.33333% with 3 lines in your changes missing coverage. Please review.
✅ Project coverage is 97.07%. Comparing base (371b402) to head (c21a50c).
⚠️ Report is 298 commits behind head on master.

Files with missing lines Patch % Lines
eli5/sklearn/transform.py 83.33% 3 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##           master     #308      +/-   ##
==========================================
- Coverage   97.09%   97.07%   -0.03%     
==========================================
  Files          44       44              
  Lines        2856     2868      +12     
  Branches      542      543       +1     
==========================================
+ Hits         2773     2784      +11     
- Misses         45       46       +1     
  Partials       38       38              
Files with missing lines Coverage Δ
eli5/sklearn/transform.py 93.61% <83.33%> (-0.67%) ⬇️
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

add scikit-learn-contrib/stability-selection support

3 participants