+1 On Tue, Aug 28, 2018 at 12:39 AM Hagay Lupesko <[email protected]> wrote:
> Thanks for the feedback Chris. Will follow up. > > On Fri, Aug 24, 2018 at 10:53 AM Chris Olivier <[email protected]> > wrote: > > > +1 for pointing to NVidia's repo for the newer Cub and subsequent > versions. > > > > On Fri, Aug 24, 2018 at 10:01 AM Hagay Lupesko <[email protected]> > wrote: > > > > > Hi all, > > > > > > > > > One of MXNet’s submodule dependencies is a snapshot of Nvidia Cub ( > > > https://github.com/dmlc/cub) – the snapshot is of an older version of > > Cub > > > (1.7), while the latest Nvidia Cub release is 1.8. Note that dmlc/cub > > has > > > no customizations of the source Cub repo. > > > > > > > > > I’d like to suggest to update the existing Cub submodule to Nvidia’s > Cub > > > repo. Instead of the snapshot, MXNet will be using Nvidia’s repo and > the > > > latest release (both repos have the same BSD-3 license, so licensing > > should > > > not be an issue). > > > > > > > > > Wanted to get feedback from the community to make sure I'm not missing > > > anything. > > > > > > if there are no objections I'll submit a PR for the change. > > > > > > > > > Cheers, > > > > > > Hagay > > > > > >
