-
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
paddle inference interface implementation #10956
Conversation
fa12c70
to
4fb6bc5
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
#include <glog/logging.h> | ||
#include <gtest/gtest.h> | ||
|
||
#include "gflags/gflags.h" |
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.
<gflags/gflags.h>
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.
will fix in follow up.
DEPS paddle_fluid_api paddle_inference_api paddle_inference_api_impl | ||
ARGS --dirname=${PYTHON_TESTS_DIR}/book/) | ||
# set_tests_properties(${TARGET_NAME} | ||
# PROPERTIES DEPENDS ${DEP_TEST}) |
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.
clean these
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.
I'll add this later. set_tests_properties is needed so that test_word2vec is run first. However, this line throws error.
* 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.
wrong code format
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.
will fix in followup.
No description provided.