Skip to content
This repository has been archived by the owner on Jan 15, 2024. It is now read-only.

[CI] Update MXNet master version tested on CI #1113

Merged
merged 2 commits into from
Jan 18, 2020
Merged

[CI] Update MXNet master version tested on CI #1113

merged 2 commits into from
Jan 18, 2020

Conversation

leezu
Copy link
Contributor

@leezu leezu commented Jan 15, 2020

No description provided.

@leezu leezu requested a review from a team as a code owner January 15, 2020 18:11
@codecov
Copy link

codecov bot commented Jan 15, 2020 •
edited
Loading

Codecov Report

Merging #1113 into master will increase coverage by 0.13%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #1113      +/-   ##
==========================================
+ Coverage   88.22%   88.36%   +0.13%     
==========================================
  Files          67       67              
  Lines        6312     6333      +21     
==========================================
+ Hits         5569     5596      +27     
+ Misses        743      737       -6
Impacted Files Coverage Δ
src/gluonnlp/data/utils.py 86.39% <0%> (+7.02%) ⬆️

@leezu leezu added the release focus Progress focus for release label Jan 15, 2020
@mli
Copy link
Member

mli commented Jan 15, 2020

Job PR-1113/1 is complete.
Docs are uploaded to http://gluon-nlp-staging.s3-accelerate.dualstack.amazonaws.com/PR-1113/1/index.html

@leezu
Copy link
Contributor Author

leezu commented Jan 15, 2020

Failing due to

[2020-01-15T19:27:03.256Z] OSError: /var/lib/jenkins/workspace/gluon-nlp-gpu-py3-master/conda/gpu/py3-master/lib/python3.5/site-packages/horovod/mxnet/mpi_lib.cpython-35m-x86_64-linux-gnu.so: undefined symbol: _Z20MXAPIHandleExceptionRKSt9exception

apache/mxnet#17292

@leezu leezu mentioned this pull request Jan 16, 2020
Closed
6 tasks
@leezu
Copy link
Contributor Author

leezu commented Jan 16, 2020 •
edited
Loading

@xinyu-intel I'm disabling the horovod test on master. Then we can merge this PR. That will allow you to test #1080 better (though your PR can't be merged prior to 0.9 release)

@mli
Copy link
Member

mli commented Jan 16, 2020

Job PR-1113/3 is complete.
Docs are uploaded to http://gluon-nlp-staging.s3-accelerate.dualstack.amazonaws.com/PR-1113/3/index.html

@mli
Copy link
Member

mli commented Jan 16, 2020

Job PR-1113/4 is complete.
Docs are uploaded to http://gluon-nlp-staging.s3-accelerate.dualstack.amazonaws.com/PR-1113/4/index.html

@eric-haibin-lin eric-haibin-lin merged commit 225c2af into dmlc:master Jan 18, 2020
eric-haibin-lin pushed a commit to eric-haibin-lin/gluon-nlp that referenced this pull request Feb 25, 2020
@leezu
[CI] Update MXNet master version tested on CI (dmlc#1113)
cb99ca3 
* [CI] Update MXNet master version tested on CI

* Disable horovod test on master
eric-haibin-lin added a commit that referenced this pull request Feb 25, 2020
@eric-haibin-lin @leezu
[CI] Update MXNet master version tested on CI (#1113) (#1176)
6520c72 
* [CI] Update MXNet master version tested on CI (#1113)

* [CI] Update MXNet master version tested on CI

* Disable horovod test on master

* update link

Co-authored-by: Leonard Lausen <leonard@lausen.nl>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@eric-haibin-lin eric-haibin-lin eric-haibin-lin approved these changes

@xinyu-intel xinyu-intel xinyu-intel approved these changes

Assignees
No one assigned
Labels
release focus Progress focus for release
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@leezu @mli @eric-haibin-lin @xinyu-intel