Skip to content
This repository has been archived by the owner on Nov 17, 2023. It is now read-only.

update cub commit for license issue #15963

Closed
wants to merge 1 commit into from
Closed

Conversation

roywei
Copy link
Member

@roywei roywei commented Aug 21, 2019

fix #15569

updated cub to latest commit which the author updated license header.

cc @PatricZhao @TaoLv

@roywei roywei changed the title update cub commit update cub commit for license issue Aug 21, 2019
@anirudhacharya
Copy link
Member

@mxnet-label-bot add [pr-awaiting-review]

@marcoabreu marcoabreu added the pr-awaiting-review PR is waiting for code review label Aug 26, 2019
@roywei
Copy link
Member Author

roywei commented Aug 26, 2019

It seems we need to update operator code to use latest cub.

@roywei roywei added the pr-work-in-progress PR is still work in progress label Aug 26, 2019
@lanking520
Copy link
Member

How was the status now?

@ciyongch
Copy link
Contributor

Hi @roywei, is there any updates for this issue? As we're now preparing for v1.7.x release (roadmap: #16864), do you think we still need the fix in this release, thanks!

@ciyongch
Copy link
Contributor

Gentle ping @roywei , what's he current status? May I know if it's still needed in v1.7.0 release, thanks!

@roywei
Copy link
Member Author

roywei commented Apr 23, 2020

@ciyongch Hi I don't think it's a hard requirement as the author already stated he allows us to use it. It's seem there are some code update included that breaks mxnet operator.
I'm currently packed and not able to fix the break now. It will be great if someone else can help take a look on the code change.
Closing this PR.

@roywei roywei closed this Apr 23, 2020
@ciyongch ciyongch mentioned this pull request Jul 22, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
pr-awaiting-review PR is waiting for code review pr-work-in-progress PR is still work in progress
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Cub license issue
5 participants