-
Notifications
You must be signed in to change notification settings - Fork 5.8k
[Add] add single process directive #57692
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
Conversation
你的PR提交成功,感谢你对开源项目的贡献! |
这次 CI 过了就不用恢复了,合入后我那边直接解决冲突就好~ |
@@ -500,6 +536,10 @@ def _extract_examples(self, api_name, docstring, **directives): | |||
def _execute_xdoctest( | |||
self, examples_to_test, examples_nocode, **directives | |||
): | |||
# if use solo(single process), execute without multiprocessing/thread |
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.
另外问一下 SOLO 是什么缩写呀?还是说 solo 这个单词?
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.
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.
一般表示音乐里面的独奏 ~ 写那个 single processing 太长了,就想到这个 🤣
好的~
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.
LGTMeow 🐾
PR types
Others
PR changes
Others
Description
示例检查中使用 multiprocessing 会导致某些 paddle 中使用多进程的接口出现嵌套进程,因此增加
SOLO
指令,使用此指令的示例在主进程中运行检查。涉及文件:
tools/sampcd_processor.py
增加SOLO
指令tools/test_sampcd_processor.py
增加测试用例python/paddle/jit/api.py
临时文件,用于测试p.s. convert-doctest 已同步修改
@SigureMo 请评审~