Skip to content

add new selector based on information gain - #542

Merged
solegalli merged 12 commits into
mainfrom
ivselector
Oct 24, 2022
Merged

add new selector based on information gain#542
solegalli merged 12 commits into
mainfrom
ivselector

Conversation

@solegalli

Copy link
Copy Markdown
Collaborator

FYI @Morgan-Sell

closes #488
closes #280

@solegalli

Copy link
Copy Markdown
Collaborator Author

Still need to fix some errors, and go over the documentation

@solegalli

Copy link
Copy Markdown
Collaborator Author

Hi @Morgan-Sell

I just fixed the last bugs and went over the documentation. You did an awesome job with the user guide and the docstrings. Thank you so much!!

@codecov

codecov Bot commented Oct 14, 2022

Copy link
Copy Markdown

Codecov Report

Merging #542 (57e444f) into main (07de62b) will increase coverage by 0.05%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main     #542      +/-   ##
==========================================
+ Coverage   97.41%   97.47%   +0.05%     
==========================================
  Files          89       90       +1     
  Lines        3410     3480      +70     
  Branches      673      681       +8     
==========================================
+ Hits         3322     3392      +70     
  Misses         32       32              
  Partials       56       56              
Impacted Files Coverage Δ
feature_engine/selection/target_mean_selection.py 97.05% <ø> (ø)
feature_engine/encoding/woe.py 100.00% <100.00%> (ø)
feature_engine/selection/__init__.py 100.00% <100.00%> (ø)
feature_engine/selection/information_value.py 100.00% <100.00%> (ø)

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@solegalli solegalli closed this Oct 17, 2022
@solegalli
solegalli deleted the ivselector branch October 17, 2022 09:48
@solegalli
solegalli restored the ivselector branch October 24, 2022 09:41
@solegalli

Copy link
Copy Markdown
Collaborator Author

reopening, because I seem to have forgotten to merge!

@solegalli solegalli reopened this Oct 24, 2022
@solegalli
solegalli merged commit 99c7ee6 into main Oct 24, 2022
@solegalli
solegalli deleted the ivselector branch October 24, 2022 09:42
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 selection transformer based on the Information Value (IV) for nominal variables

1 participant