mxnet-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Marco de Abreu <marco.g.ab...@googlemail.com>
Subject Re: Module maintainers proposal
Date Tue, 09 Jan 2018 07:37:41 GMT
Lgtm

Am 09.01.2018 8:24 vorm. schrieb "YiZhi Liu" <javelinjs@gmail.com>:

> +1 for adding @CodingCat (Nan Zhu) as owner of Scala package.
>
> 2018-01-08 21:58 GMT-08:00 Mu Li <muli.cmu@gmail.com>:
>
> > It has been a while for discussing having maintainers for individual
> > modules. A maintainer for a module will be the main person who reviews
> and
> > approves PRs contributed to that module.
> >
> > Currently, some maintainers are listed in the file CODEOWNERS
> > <https://github.com/apache/incubator-mxnet/blob/master/CODEOWNERS>:
> >
> > # Owners of Apache MXNet
> > # Global owners
> > *            @apache/mxnet-committers
> > # Owners of language bindings
> > R-package/*        @thirdwing
> > scala-package/*        @javelinjs
> > perl-package/*        @sergeykolychev
> > # CMake owners
> > CMakeLists.txt        @cjolivier01
> > cmake/*            @cjolivier01
> >
> > However, this list is incomplete. This document proposes how to partition
> > mxnet codes into modules and put tentative maintainers for each module.
> >
> >    - I tried to select the activate contributor who contributed most to
> the
> >    module, not the new contributor who is going to work for it. But I may
> > be
> >    outdated.
> >    - Code review is not easy. So maintainers may need help at the
> >    beginning.
> >    - Eric (@piiswrong) did most PR reviews, so I didn't put his name on
> >    every module.
> >
> > Any suggestion is welcome.
> > FrontendPYTHON: @szha
> >
> > python/
> >
> > R: @thirdwing @hetong007
> >
> > R-package/
> >
> > SCALA: @CodingCat @javelinjs
> >
> > scala-package/
> >
> > PERL@sergeykolychev
> >
> > perl-package/
> >
> > C++?
> >
> > cpp-package/
> >
> > MATLAB: DEPRECATE IT?
> >
> > matlab/
> >
> > AMALGAMATION: DEPRECATE IT?
> >
> > amalgamation/
> >
> > Backend@reminisce @eric-haibin-lin
> >
> > include/
> > src/
> >
> > Build@cjolivier01
> >
> > docker/
> > docker_multiarch/
> > make/
> > Makefile
> > cmake/
> > CMakeLists.txt
> > setup-utils/
> > prepare_mkl.sh
> >
> > Test@marcoabreu
> >
> > tests/
> > Jenkinsfile
> >
> > Doc & Website@kevinthesun
> >
> > docs/
> >
> > ExamplesNot sure, since we have a lot of contributors here. We probably
> > need to remove low quality examples and assign one maintainer for each of
> > the rest.
> >
> > example/
> >
> > ToolsNot sure as well, since we have a bunch of things there. Probably
> need
> > to the same thing as examples
> >
> > tools/
> > plugin/
> >
> > AppendixLines of codes added into each folder on the last two months:
> > Lines of codes added into example/
> > Lines of codes added into src/
> >
>
>
>
> --
> Yizhi Liu
> DMLC member
> Amazon Web Services
> Vancouver, Canada
>

Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message