Skip to content
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

disable ut quickly #27793

Merged
merged 13 commits into from
Oct 14, 2020
Merged

Conversation

XieYunshen
Copy link
Contributor

@XieYunshen XieYunshen commented Oct 9, 2020

PR types

Others

PR changes

Others

Describe

disable ut quickly;

@paddle-bot-old
Copy link

paddle-bot-old bot commented Oct 9, 2020

Thanks for your contribution!
Please wait for the result of CI firstly. See Paddle CI Manual for details.

zhwesky2010
zhwesky2010 previously approved these changes Oct 12, 2020
Copy link
Contributor

@zhwesky2010 zhwesky2010 left a comment

Choose a reason for hiding this comment

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

LGTM for windows

@chalsliu
Copy link
Contributor

建议日志中打印disable了哪些单测

chalsliu
chalsliu previously approved these changes Oct 12, 2020
Copy link
Contributor

@chalsliu chalsliu left a comment

Choose a reason for hiding this comment

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

LGTM

@XieYunshen XieYunshen dismissed stale reviews from chalsliu and zhwesky2010 via a30afd9 October 13, 2020 07:28
@XieYunshen
Copy link
Contributor Author

建议日志中打印disable了哪些单测
done
Linux和Mac中会显示
image

chalsliu
chalsliu previously approved these changes Oct 13, 2020
@XieYunshen XieYunshen merged commit fea09fe into PaddlePaddle:develop Oct 14, 2020
chen-zhiyu pushed a commit to chen-zhiyu/Paddle that referenced this pull request Oct 15, 2020
* disable ut quickly

* fix some error

* fix some error

* install urllib2 package

* use requests package instead of urllib2

* fix error caused by windows regular parameter

* fix error on windows

* fix some error

* fix with format error

* show disable ut in log

* fix some error

* fix some error

* add the handling of error in executing get_quickly_disable_ut
@tianshuo78520a tianshuo78520a mentioned this pull request Oct 22, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants