-
Notifications
You must be signed in to change notification settings - Fork 913
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Update build and test with mxnet 1.7.0 #546
Conversation
docs/build.yml
Outdated
- jupyter-sphinx>=0.2.2 | ||
- mxnet-cu100==1.6.0b20191013 | ||
- mxnet-cu100==1.7.0b20200629 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I believe this 1.7.0b actually is a build of the 1.x branch.
@ciyongch We are going to test AutoGluon with MXNet 1.7. Currently, I'm using the nightly builds of 1.x but will update that to 1.7.rc0 once that's ready. |
Thanks for tagging me on this @sxjscience , I'm not sure whether there'll be nightly image for v1.7.x or later v1.7.rc0. Anyway, I will keep you posted when 1.7.rc0 is announced. |
Hi @sxjscience , MXNet 1.7.0.rc0 is cut off, would you mind to have a try with the latest code base? Thanks. |
@ciyongch Would you know how should I try it in CI? |
push an [empty] commit? |
I actually mean if there is any pip package because the previous one seems to be the 1.x nightly build. I’m currently rebuilding the static pip package.
Get Outlook for iOS<https://aka.ms/o0ukef>
…________________________________
From: ciyong <notifications@github.com>
Sent: Wednesday, July 8, 2020 11:44:15 PM
To: awslabs/autogluon <autogluon@noreply.github.com>
Cc: Xingjian SHI <xshiab@connect.ust.hk>; Mention <mention@noreply.github.com>
Subject: Re: [awslabs/autogluon] [WIP] Update build and test with mxnet 1.7.0 nightly (#546)
push an [empty] commit?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub<#546 (comment)>, or unsubscribe<https://github.com/notifications/unsubscribe-auth/ABHQH3WSLWQD77E5QDOX373R2VRL7ANCNFSM4OL34URQ>.
|
@ciyongch Basically the CI fails by throwing |
Job PR-546-8 is done. |
@ciyongch The CI of AutoGluon has passed after the PR in MXNet is reverted |
@Jerryzcn @zhreshold @jwmueller @Innixma We are able to pass the test with 1.7.0. Currently, I'm using the wheels built from the 1.7.x branch in MXNet: The wheel packages used: |
Great! Is it ok if we wait until 1.7.0 is officially released before merging this PR? |
@Innixma Yes, it's fine to wait until the official 1.7.0 so I will leave it open for now. |
@sxjscience Good job, thanks for pushing this |
It's great to root cause the issue, thanks @sxjscience ! |
Job PR-546-10 is done. |
update update use the new wheel add new pip wheel for rc0 update update installation guide + use prebuilt MXNet 1.7
7f1d65b
to
ce229d5
Compare
Job PR-546-11 is done. |
Job PR-546-12 is done. |
Job PR-546-13 is done. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for the quick turnaround on this fix!
Test for MXNet 1.7.0 nightly to see if there are any issue.