Totally agree with Henry and Pedro. On Wed, May 2, 2018 at 12:56 PM, Pedro Larroy <pedro.larroy.li...@gmail.com> wrote:
> For me the situation with DMLC is problematic. > > I often find myself having to fix things in the DMLC subrepos. > > * These changes are impossible to test with the MXNet CI system without > doing shenanigans like changing the submodules to my own forks. > * Slows down development as fixes need to be merged on the DMLC subrepo > first, then MXNet updated with the new subrepo commit, effectively doubling > the turnaround time for any fix / commit. > * The DMLC subrepos are in a different organization and subject to > different set of committers and rules. > > I think we should have a plan forward to have MXNet be more self contained, > so it's faster and easier to contribute. > > Pedro. > > > On Tue, May 1, 2018 at 1:25 AM, Hen <bay...@apache.org> wrote: > > > Naive question - How often does this happen? It should be rare that a > > project needs to send a PR to a dependency, and much rarer that it > blocks a > > release. > > > > It sounds like it’s too coupled a dependency. I suspect the DMLC code > > situation is going to be a major blocker to any chance of graduating. > > > > Hen > > > > On Mon, Apr 30, 2018 at 1:23 PM Gautam <gautamn...@gmail.com> wrote: > > > > > +1 on merging this "https://github.com/dmlc/mshadow/pull/319 " > > > > > > I am curious how are we tracking the sub-modules's PRs which are really > > > important for MXNet? > > > This PR has been waiting to merge for almost 4 months. > > > > > > > > > > > > On Mon, Apr 30, 2018 at 1:51 AM, Pedro Larroy < > > > pedro.larroy.li...@gmail.com> > > > wrote: > > > > > > > -1 > > > > > > > > We should merge this and update mshadow before the next release: > > > > https://github.com/dmlc/mshadow/pull/319 so we compile cuda for > > Volta. > > > > > > > > On Sat, Apr 28, 2018 at 12:53 AM, Steffen Rochel < > > > steffenroc...@gmail.com> > > > > wrote: > > > > > > > > > Hi Chris - acknowledge that building the docs is not as good as it > > > should > > > > > be and needs to be improved. Is it worse compared to 1.1.0 release > to > > > > > consider as release blocker? > > > > > > > > > > > > > > > On Fri, Apr 27, 2018 at 9:53 AM Chris Olivier < > cjolivie...@gmail.com > > > > > > > > wrote: > > > > > > > > > > > -1 > > > > > > > > > > > > Building the docs locally is an absolute nightmare. I haven;t > been > > > > able > > > > > to > > > > > > get it to work yet. > > > > > > > > > > > > On Thu, Apr 26, 2018 at 3:36 PM, Marco de Abreu < > > > > > > marco.g.ab...@googlemail.com> wrote: > > > > > > > > > > > > > Hello, > > > > > > > > > > > > > > I'd like to request to pause this vote since I have identified > an > > > > issue > > > > > > > with our CMakeLists, breaking all UNIX builds with the latest > > > version > > > > > > > (3.11) of cmake. This issue is tracked at [1]. The PR to fix > this > > > is > > > > > > > available at [2]. > > > > > > > > > > > > > > Best regards, > > > > > > > Marco > > > > > > > > > > > > > > > > > > > > > [1]: https://github.com/apache/incubator-mxnet/issues/10708 > > > > > > > [2]: https://github.com/apache/incubator-mxnet/pull/10712 > > > > > > > > > > > > > > > > > > > > > On Thu, Apr 26, 2018 at 3:05 PM, Sheng Zha <szha....@gmail.com > > > > > > wrote: > > > > > > > > > > > > > > > -1 for the following reasons: > > > > > > > > > > > > > > > > 1. due to addition of support for fp16, the build breaks for > > > > windows > > > > > > and > > > > > > > > older version of osx (clang 8 for example). fix is in > > > > > > > > https://github.com/dmlc/mshadow/pull/333 > > > > > > > > > > > > > > > > 2. due to addition of quantized fully connected op, cuda 7.5 > > > build > > > > is > > > > > > > > broken. Jun Wu is tracking the issue. > > > > > > > > > > > > > > > > -sz > > > > > > > > > > > > > > > > On Thu, Apr 26, 2018 at 3:01 PM, Anirudh < > > anirudh2...@gmail.com> > > > > > > wrote: > > > > > > > > > > > > > > > > > Hi all, > > > > > > > > > > > > > > > > > > As part of RC1 release, We have addressed the issue with > > > respect > > > > to > > > > > > > > > asymmetric padding in ONNX Import module ( > > > > > > > > > https://github.com/apache/incubator-mxnet/pull/10676). > > > > > > > > > We have also added existing silent failures for MXNet Conv > > and > > > > the > > > > > > > > > incompatibility in behavior for certain use cases between > > MXNet > > > > > > without > > > > > > > > > MKLDNN and MXNet with MKLDNN support to the release notes. > We > > > > have > > > > > > > marked > > > > > > > > > both ONNX and MKLDNN support as "Experimental" in the > release > > > > > notes. > > > > > > > The > > > > > > > > > tutorial > > > > > > > > > <https://github.com/apache/incubator-mxnet/blob/master/ > > > > > > > > > docs/tutorials/onnx/fine_tuning_gluon.md> > > > > > > > > > which was called out still seems to be failing with cpu > > context > > > > and > > > > > > we > > > > > > > > have > > > > > > > > > mentioned this as a known issue in the release notes. > Since, > > > both > > > > > > > MKLDNN > > > > > > > > > support and ONNX import module have been marked > experimental > > > this > > > > > > > should > > > > > > > > > not be a blocking issue. > > > > > > > > > > > > > > > > > > I would like to propose a vote to release Apache MXNet > > > > (incubating) > > > > > > > > version > > > > > > > > > 1.2.0.RC1. Voting will start now (Thursday, April 26th) and > > end > > > > at > > > > > > 3:00 > > > > > > > > PM > > > > > > > > > PDT, Monday, April 30th. > > > > > > > > > > > > > > > > > > Link to release notes: > > > > > > > > > https://cwiki.apache.org/confluence/display/MXNET/ > > > > > > > > > Apache+MXNet+%28incubating%29+1.2.0+Release+Notes > > > > > > > > > > > > > > > > > > Link to release candidate 1.2.0.rc1: > > > > > > > > > > > > https://github.com/apache/incubator-mxnet/releases/tag/1.2.0.rc1 > > > > > > > > > > > > > > > > > > Voting results for 1.2.0.rc0: > > > > > > > > > https://lists.apache.org/thread.html/ > > > > > c79327b822c47ec4f97dad463fc6cd > > > > > > > > > fe2aa517772d22eca24c5c300f@%3Cdev.mxnet.apache.org%3E > > > > > > > > > > > > > > > > > > View this page, click on “Build from Source”, and use the > > > source > > > > > code > > > > > > > > > obtained from the 1.2.0.rc1 tag: > > > > > > > > > https://mxnet.incubator.apache.org/install/index.html > > > > > > > > > > > > > > > > > > (Note: The README.md points to the 1.2.0 tag and does not > > work > > > at > > > > > the > > > > > > > > > moment.) > > > > > > > > > > > > > > > > > > Please see a preview of the 1.2.0 version of the website > at: > > > > > > > > > http://mxnet.incubator.apache. > org/versions/1.2.0/index.html > > > > > > > > > > > > > > > > > > > > > > > > > > > Anirudh > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > -- > > > Best Regards, > > > Gautam Kumar > > > > > >
