-
Notifications
You must be signed in to change notification settings - Fork 6.8k
skipping tests that cannot fit in nightly CI machine #17450
Conversation
8561a63
to
3651f02
Compare
@mxnet-label-bot add [pr-awaiting-review] |
@apeforest @ChaiBapchya PR ready for review |
tests/nightly/test_large_array.py
Outdated
@@ -689,6 +693,7 @@ def check_pick(): | |||
res = mx.nd.pick(a, b) | |||
assert res.shape == b.shape | |||
|
|||
@unittest.skip("Memory doesn't free up after stacked execution with other ops, tracked at /~https://github.com/apache/incubator-mxnet/issues/17411") |
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.
this line is too long. please break it to meet PEP8 style
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.
aah ... got it ! Will do
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.
please do lint
3651f02
to
dd951e3
Compare
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.
LGTM
@mxnet-label-bot update [pr-awaiting-merge] |
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.
LGTM!
Retriggered windows-gpu for you. Hopefully it passes!
Description
skipping following tests as they require memory more than present in nightly CI.
A Stacked run of tests of these operators along with other ops doesn't free up memory on time resulting in Memory Error on nightly CI machine. Also updates system path for correct imports.
Fixes #14981
Fixes #14980
Checklist
Essentials
Please feel free to remove inapplicable items for your PR.
Tests