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.

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

Fixes exception when using predict_proba on fitted Pipeline object with a ColumnTransformer step #4774

Merged
merged 11 commits into from
Jun 29, 2022

Conversation

VamsiTallam95
Copy link
Contributor

This PR fixes a subtle bug in check_array of cuml.thirdparty_adapters.adapters which is the primary cause for the bug. Fix #4368.

@VamsiTallam95 VamsiTallam95 requested a review from a team as a code owner June 8, 2022 22:46
@github-actions github-actions bot added the Cython / Python Cython or Python issue label Jun 8, 2022
@VamsiTallam95 VamsiTallam95 marked this pull request as draft June 8, 2022 22:47
@cjnolet
Copy link
Member

cjnolet commented Jun 14, 2022

@VamsiTallam95, would you mind adding a more descriptive title to this PR? It helps when folks (including potential reviewers) are browsing the list of open PRs.

@VamsiTallam95 VamsiTallam95 changed the title Fix #4368 Fixes exception when using predict_proba on fitted Pipeline object with a ColumnTransformer step Jun 14, 2022
@VamsiTallam95
Copy link
Contributor Author

rerun tests

@VamsiTallam95 VamsiTallam95 marked this pull request as ready for review June 27, 2022 15:07
@codecov-commenter
Copy link

Codecov Report

❗ No coverage uploaded for pull request base (branch-22.08@b26fe7e). Click here to learn what that means.
The diff coverage is n/a.

@@               Coverage Diff               @@
##             branch-22.08    #4774   +/-   ##
===============================================
  Coverage                ?   77.62%           
===============================================
  Files                   ?      180           
  Lines                   ?    11382           
  Branches                ?        0           
===============================================
  Hits                    ?     8835           
  Misses                  ?     2547           
  Partials                ?        0           
Flag Coverage Δ
dask 45.51% <0.00%> (?)
non-dask 67.25% <0.00%> (?)

Flags with carried forward coverage won't be shown. Click here to find out more.


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 b26fe7e...ecc9922. Read the comment docs.

@caryr35 caryr35 added this to PR-WIP in v22.08 Release via automation Jun 29, 2022
@caryr35 caryr35 moved this from PR-WIP to PR-Needs review in v22.08 Release Jun 29, 2022
@dantegd dantegd added bug Something isn't working non-breaking Non-breaking change labels Jun 29, 2022
v22.08 Release automation moved this from PR-Needs review to PR-Reviewer approved Jun 29, 2022
@dantegd dantegd added bug Something isn't working and removed bug Something isn't working labels Jun 29, 2022
@dantegd
Copy link
Member

dantegd commented Jun 29, 2022

@gpucibot merge

@rapids-bot rapids-bot bot merged commit 46937dd into rapidsai:branch-22.08 Jun 29, 2022
v22.08 Release automation moved this from PR-Reviewer approved to Done Jun 29, 2022
jakirkham pushed a commit to jakirkham/cuml that referenced this pull request Feb 27, 2023
…th a ColumnTransformer step (rapidsai#4774)

This PR fixes a subtle bug in check_array of cuml.thirdparty_adapters.adapters which is the primary cause for the bug. Fix rapidsai#4368.

Authors:
  - https://github.com/VamsiTallam95
  - Ray Douglass (https://github.com/raydouglass)

Approvers:
  - Dante Gama Dessavre (https://github.com/dantegd)

URL: rapidsai#4774
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working Cython / Python Cython or Python issue non-breaking Non-breaking change
Projects
No open projects
Development

Successfully merging this pull request may close these issues.

[BUG] .predict_proba on fitted Pipeline object with a ColumnTransformer step raises exception
5 participants