mxnet-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Pedro Larroy <pedro.larroy.li...@gmail.com>
Subject Re: [VOTE] Release MXNet version 1.2.1.RC0 (Patch Release)
Date Thu, 21 Jun 2018 21:07:36 GMT
I guess that's why I disable openmp for development, so given that it's a
known issue an the unit tests passed on osx I change my vote accordingly
to +1 not to block the release.

On Thu, Jun 21, 2018 at 1:52 PM Anirudh <anirudh2290@gmail.com> wrote:

> Hi Pedro,
>
> I have seen this with the DEBUG flag on, not without it.
> I opened an issue here some time back:
> https://github.com/apache/incubator-mxnet/issues/10856
>
> Anirudh
>
> On Thu, Jun 21, 2018 at 12:16 PM, Pedro Larroy <
> pedro.larroy.lists@gmail.com
> > wrote:
>
> > Got it. Sorry to bring this up, and the deja-vu :-) . Makes sense not to
> > consider this issue again as you suggested, thanks.
> >
> >
> > I compiled in OSX and run unit tests. Also run SSD detector and is
> working
> > fine.
> > Compiled in ubuntu / cpu / debug with openmp. And run unit tests. I have
> > the following failure
> > in test_gluon_data:test_multi_worker_forked_data_loader
> >
> >
> > test_gluon_data.test_image_folder_dataset ... ok
> > Test should successfully run its course of multi-process/forked data
> loader
> > without errors ... Assertion failure at kmp_runtime.cpp(6479):
> > __kmp_thread_pool == __null.
> > OMP: Error #13: Assertion failure at kmp_runtime.cpp(6479).
> > OMP: Hint: Please submit a bug report with this message, compile and run
> > commands used, and machine configuration info including native compiler
> and
> > operating system versions. Faster response will be obtained by including
> > all program sources. For information on submitting this issue, please see
> > https://bugs.llvm.org/.
> > Assertion failure at kmp_runtime.cpp(6479): __kmp_thread_pool == __null.
> > OMP: Error #13: Assertion failure at kmp_runtime.cpp(6479).
> > OMP: Hint: Please submit a bug report with this message, compile and run
> > commands used, and machine configuration info including native compiler
> and
> > operating system versions. Faster response will be obtained by including
> > all program sources. For information on submitting this issue, please see
> > https://bugs.llvm.org/.
> >
> >
> > Can be reproduced, anyone has seen this error before?  Is this a known
> > issue?
> >
> > Pedro.
> >
> >
> >
> > On Thu, Jun 21, 2018 at 11:08 AM Anirudh <anirudh2290@gmail.com> wrote:
> >
> > > Hi Pedro,
> > >
> > > I think you raised this issue in 1.2.0 release here:
> > >
> > > https://lists.apache.org/thread.html/ddc088a21aac179144350ea97353a7
> > ea885b2765ccb98db08a03ba2d@%3Cdev.mxnet.apache.org%3E
> > > .
> > > I actually forgot about this issue during this release. Having said
> > that, I
> > > think since this works with make and the customers using cmake with
> > > USE_OPENMP=OFF should be considerably small we should not block the
> > release
> > > for this.
> > > The main reason we are doing this release is for this issue
> > > <https://github.com/apache/incubator-mxnet/issues/11091> . Now pulling
> > > this
> > > change for the cmake fix would be also mean we need to pull 8 more
> > commits
> > > from dmlc-core and its considerable risk to introduce for the patch
> > > release.
> > > This would also mean cutting another rc. I think in the interest of our
> > > customers who are eagerly waiting for the patch release to fix the main
> > > issue, we should move ahead here.
> > > I missed reviewing all the known issue of 1.2.0 and add it to 1.2.1
> > release
> > > notes. I will do that now.
> > >
> > > Anirudh
> > >
> > >
> > >
> > > On Thu, Jun 21, 2018 at 10:42 AM, Pedro Larroy <
> > > pedro.larroy.lists@gmail.com
> > > > wrote:
> > >
> > > > I think I have fixed this before, I will check if the patch didn't
> make
> > > it
> > > > to the branch.
> > > >
> > > > On Thu, Jun 21, 2018 at 10:24 AM Pedro Larroy <
> > > > pedro.larroy.lists@gmail.com>
> > > > wrote:
> > > >
> > > > > -1   I can't compile:
> > > > >
> > > > > 3rdparty/dmlc-core/libdmlc.a(io.cc.o): In function
> > > > > `std::thread::thread<dmlc::ThreadedIter<dmlc::io::
> > > > InputSplitBase::Chunk>::Init(std::function<bool
> > > > > (dmlc::io::InputSplitBase::Chunk**)>, std::function<void
> > > > > ()>)::{lambda()#1}&>(dmlc::ThreadedIter<dmlc::io::
> > > > InputSplitBase::Chunk>::Init(std::function<bool
> > > > > (dmlc::io::InputSplitBase::Chunk**)>, std::function<void
> > > > > ()>)::{lambda()#1}&)':
> > > > > /usr/include/c++/5/thread:137: undefined reference to
> > `pthread_create'
> > > > > collect2: error: ld returned 1 exit status
> > > > > ninja: build stopped: subcommand failed.
> > > > >
> > > > >
> > > > > No LSB modules are available.
> > > > > Distributor ID: Ubuntu
> > > > > Description:    Ubuntu 16.04.4 LTS
> > > > > Release:        16.04
> > > > > Codename:       xenial
> > > > >
> > > > >
> > > > > My build script:
> > > > >
> > > > >
> > > > > #!/bin/bash
> > > > > set -e
> > > > > set -x
> > > > >
> > > > > renice -n 19 -p $$
> > > > >
> > > > > mkdir -p build && cd build
> > > > > cmake -DUSE_CPP_PACKAGE=ON -DUSE_CUDA=OFF -DUSE_OPENMP=OFF
> > > > -DUSE_OPENCV=ON
> > > > > -DCMAKE_BUILD_TYPE=Debug -GNinja ..
> > > > > ninja -v
> > > > >
> > > > > cd ..
> > > > > if [ ! -d mxnet_py3 ]; then
> > > > >     virtualenv -p `which python3` mxnet_py3
> > > > > fi
> > > > > source mxnet_py3/bin/activate
> > > > > cd python
> > > > > pip install -e .
> > > > > cd ..
> > > > > pip install opencv-python
> > > > > pip install ipython
> > > > > pip install matplotlib
> > > > >
> > > > > On Wed, Jun 20, 2018 at 6:33 PM Indhu <indhubharathi@gmail.com>
> > wrote:
> > > > >
> > > > >> +1
> > > > >>
> > > > >> On Mon, Jun 18, 2018, 6:52 PM Anirudh <anirudh2290@gmail.com>
> > wrote:
> > > > >>
> > > > >> > Hi,
> > > > >> >
> > > > >> > This is the vote to release Apache MXNet (incubating) version
> > 1.2.1.
> > > > >> Voting
> > > > >> > will start now and close Thursday June 21st 7:00 PM PDT.
> > > > >> >
> > > > >> > Link to release candidate 1.2.1.rc0:
> > > > >> >
> > > > >> >
> https://github.com/apache/incubator-mxnet/releases/tag/1.2.1.rc0
> > > > >> >
> > > > >> > View this page for installation instructions:
> > > > >> >
> > > > >> > https://mxnet.incubator.apache.org/install/index.html
> > > > >> >
> > > > >> > (Note: The README.md points to the 1.2.1 tag and does not
work
> at
> > > the
> > > > >> > moment).
> > > > >> >
> > > > >> > Please remember to test first before voting accordingly.
> > > > >> >
> > > > >> > +1 = approve
> > > > >> > +0 = no opinion
> > > > >> > -1 = disapprove (provide reason)
> > > > >> >
> > > > >> > Anirudh
> > > > >> >
> > > > >>
> > > > >
> > > >
> > >
> >
>

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