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

Nightly test fix: add filter to warnings #14532

Merged
merged 1 commit into from
Mar 27, 2019

Conversation

ThomasDelteil
Copy link
Contributor

Description

nightly test fails because of warnings that have been introduced. However there are no ways to not get the warnings at the moment. Adding filters to ignore them.

That's a bad solution, but I think it's worst to simply ignore warnings in the nightly tests. We need to either remove the warnings that have been introduced in : #14214, suppress the warnings in the imports / onnx load API or update the API to specify the type at model loading time.

@abhinavs95
Copy link
Contributor

abhinavs95 commented Mar 26, 2019

@mxnet-label-bot update [Test, pr-awaiting-merge]

@marcoabreu marcoabreu added pr-awaiting-review PR is waiting for code review Test labels Mar 26, 2019
@Ishitori
Copy link
Contributor

I run it on my EC2 instance and all failed tutorials are working now.

Copy link
Member

@wkcn wkcn left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks you for the fix! LGTM: )

@marcoabreu marcoabreu added the pr-awaiting-merge Review and CI is complete. Ready to Merge label Mar 27, 2019
@ThomasDelteil ThomasDelteil merged commit 3d20f2a into apache:master Mar 27, 2019
@marcoabreu marcoabreu removed the pr-awaiting-review PR is waiting for code review label Mar 27, 2019
vdantu pushed a commit to vdantu/incubator-mxnet that referenced this pull request Mar 31, 2019
ZhennanQin pushed a commit to ZhennanQin/incubator-mxnet that referenced this pull request Apr 3, 2019
nswamy pushed a commit that referenced this pull request Apr 5, 2019
@roywei roywei mentioned this pull request May 23, 2019
5 tasks
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
pr-awaiting-merge Review and CI is complete. Ready to Merge Test
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants