DistrictDataLabs / yellowbrick

Visual analysis and diagnostic tools to facilitate machine learning model selection.
http://www.scikit-yb.org/
Apache License 2.0
4.3k stars 559 forks source link

Fixes #1248 Adds Sklearn Pipeline tests to ModelVisualizers - CPE, ClsRpt, ConfustionMatrix, PRC, ROCAUC,DroppingCurve #1249

Closed lwgray closed 2 years ago

lwgray commented 2 years ago

Adding test for utilization of Sklearn pipelines within these ModelVisualizers

CHECKLIST

codecov[bot] commented 2 years ago

Codecov Report

Merging #1249 (a0bcb55) into develop (ad0d133) will increase coverage by 0.03%. The diff coverage is n/a.

@@             Coverage Diff             @@
##           develop    #1249      +/-   ##
===========================================
+ Coverage    90.58%   90.61%   +0.03%     
===========================================
  Files           92       92              
  Lines         5213     5213              
===========================================
+ Hits          4722     4724       +2     
+ Misses         491      489       -2     
Impacted Files Coverage Δ
yellowbrick/classifier/rocauc.py 96.94% <0.00%> (+0.76%) :arrow_up:
yellowbrick/classifier/confusion_matrix.py 93.68% <0.00%> (+1.05%) :arrow_up:

Continue to review full report at Codecov.

Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update ad0d133...a0bcb55. Read the comment docs.

bbengfort commented 2 years ago

@lwgray thank you for taking on this work!