-
Notifications
You must be signed in to change notification settings - Fork 5.6k
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
inference API little fix #11069
inference API little fix #11069
Conversation
|
||
if(WITH_CONTRIB) |
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.
这样写,有点多,能不能改成:
if(WITH_CONTRIB)
set(fluid_modules ${fluid_modules} paddle_inference_api)
endif()
后面就可以保持不变了。
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.
我改下
c92481b
to
1463aca
Compare
* non-sequence scenerios. | ||
* TODO(Superjomn) Prepare another API for NLP-related usages. | ||
*/ | ||
* A simple Inference API for Paddle. Currently this API might just be used by |
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.
might -> can
*/ | ||
* A simple Inference API for Paddle. Currently this API might just be used by | ||
* non-sequence scenerios. | ||
* TODO(Superjomn) Prepare another API for NLP-related usages. |
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.
" Prepare another API for" -> "Support"
No description provided.