Skip to content
This repository has been archived by the owner on Nov 17, 2023. It is now read-only.

remove warning in amp tutorial #15135

Merged
merged 1 commit into from
Jun 3, 2019
Merged

remove warning in amp tutorial #15135

merged 1 commit into from
Jun 3, 2019

Conversation

roywei
Copy link
Member

@roywei roywei commented Jun 3, 2019

Description

Continue on
#15039

Somehow the fix(PR to fork) @ptrendx made to #15039 was not reflected when it merged. So tutorial nightly is still failing.

I m adding the change. Credit goes to @ptrendx, thanks for the fix.

@roywei roywei requested a review from szha as a code owner June 3, 2019 07:44
@roywei
Copy link
Member Author

roywei commented Jun 3, 2019

I have verified the nightly job passed.

ci/build.py --docker-registry mxnetci --nvidiadocker --platform ubuntu_nightly_gpu --docker-build-retries 3 --shm-size 1500m /work/runtime_functions.sh nightly_tutorial_test_ubuntu_python2_gpu

log

[success] 0.25% test_tutorials.test_basic_reshape_transpose: 4.5422s
[success] 0.25% test_tutorials.test_control_flow: 4.5363s
[success] 0.25% test_tutorials.test_gluon_customop: 4.5326s
[success] 0.25% test_tutorials.test_gluon_hybrid: 4.5311s
[success] 0.25% test_tutorials.test_gluon_custom_layer: 4.5294s
[success] 0.25% test_tutorials.test_gluon_gluon: 4.5278s
[success] 0.20% test_tutorials.test_gluon_autograd: 3.5224s
[success] 0.20% test_tutorials.test_vision_large_scale_classification: 3.5196s
[success] 0.20% test_tutorials.test_speech_recognition_ctc: 3.5192s
----------------------------------------------------------------------
Ran 48 tests in 1797.691s

OK
build.py: 2019-06-03 06:46:02,229Z INFO Waiting for status of container e276f9889631 for 600 s.
build.py: 2019-06-03 06:46:02,417Z INFO Container exit status: {'Error': None, 'StatusCode': 0}
build.py: 2019-06-03 06:46:02,417Z INFO Container exited with success 👍
build.py: 2019-06-03 06:46:02,417Z INFO Stopping container: e276f9889631
build.py: 2019-06-03 06:46:02,419Z INFO Removing container: e276f988963

@ptrendx ptrendx merged commit 9125f6a into apache:master Jun 3, 2019
@roywei roywei mentioned this pull request Jun 4, 2019
haohuanw pushed a commit to haohuanw/incubator-mxnet that referenced this pull request Jun 23, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants