Skip to content

Remove import of scalar_broadcast_to from stemmer #4706

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
Merged
rapids-bot merged 4 commits into from
May 3, 2022
Merged

Remove import of scalar_broadcast_to from stemmer #4706

rapids-bot merged 4 commits into from
May 3, 2022

Conversation

@viclafargue
Copy link
Contributor

@viclafargue viclafargue commented Apr 22, 2022

No description provided.

@viclafargue viclafargue requested a review from a team as a code owner April 22, 2022 16:38
@github-actions github-actions bot added the Cython / Python Cython or Python issue label Apr 22, 2022
@viclafargue viclafargue added bug Something isn't working non-breaking Non-breaking change and removed Cython / Python Cython or Python issue labels Apr 22, 2022
@github-actions github-actions bot added the Cython / Python Cython or Python issue label Apr 25, 2022
@dantegd
Copy link
Member

dantegd commented Apr 25, 2022

rerun tests

1 similar comment
@hcho3
Copy link
Contributor

hcho3 commented Apr 27, 2022

rerun tests

@ayushdg ayushdg mentioned this pull request Apr 27, 2022
Merged
3 tasks
@viclafargue
Copy link
Contributor Author

viclafargue commented Apr 27, 2022

rerun tests

2 similar comments
@viclafargue
Copy link
Contributor Author

viclafargue commented Apr 27, 2022

rerun tests

@dantegd
Copy link
Member

dantegd commented Apr 29, 2022

rerun tests

@hcho3
Copy link
Contributor

hcho3 commented Apr 30, 2022

Can we disable test_hashingvectorizer_alternate_sign to unblock the CI?

@dantegd dantegd mentioned this pull request May 3, 2022
Open
cjnolet
cjnolet approved these changes May 3, 2022
View reviewed changes
Copy link
Member

@cjnolet cjnolet left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@cjnolet
Copy link
Member

cjnolet commented May 3, 2022

@gpucibot merge

@rapids-bot rapids-bot bot merged commit 19d1afc into rapidsai:branch-22.06 May 3, 2022
vimarsh6739 pushed a commit to vimarsh6739/cuml that referenced this pull request Oct 9, 2023
@viclafargue
Remove import of scalar_broadcast_to from stemmer (rapidsai#4706)
97c7742 
Authors:
  - Victor Lafargue (https://github.com/viclafargue)
  - Dante Gama Dessavre (https://github.com/dantegd)

Approvers:
  - Corey J. Nolet (https://github.com/cjnolet)

URL: rapidsai#4706
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@cjnolet cjnolet cjnolet approved these changes

Assignees

No one assigned

Labels

bug Something isn't working Cython / Python Cython or Python issue non-breaking Non-breaking change

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@viclafargue @dantegd @hcho3 @cjnolet