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

Update index_en.rst #9286

Merged
merged 2 commits into from
Mar 22, 2018
Merged

Update index_en.rst #9286

merged 2 commits into from
Mar 22, 2018

Conversation

legend06hvl
Copy link
Contributor

@legend06hvl legend06hvl commented Mar 21, 2018

Update en version
fix #8915

Update en version
@shanyi15 shanyi15 self-requested a review March 21, 2018 06:26
Copy link
Contributor

@abhinavarora abhinavarora left a comment

Choose a reason for hiding this comment

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

Thank you for the PR. There are some suggestions for the grammar and sentence structure. Please look at that before merging.

@@ -1,6 +1,23 @@
C-API Prediction Library
========================

After we trained a neural network model, then we use it to make predictions. Prediction is a process of preparing input data and processing the model to get the result.
Copy link
Contributor

Choose a reason for hiding this comment

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

trained -> train

Copy link
Contributor

Choose a reason for hiding this comment

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

I think we should replace the word prediction with Inference. The title can also read C-API Inference Library.

Copy link
Contributor

Choose a reason for hiding this comment

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

The sentence should be as follows:

After we train a neural network, we use it to do inference. Inference is the process of preparing input data and propagating it through the model to produce the result.


Compared with model training, prediction has the following features:

#. Prediction does not require backpropagation and parameter updates during training.
Copy link
Contributor

Choose a reason for hiding this comment

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

Inference does not require backpropagation and parameter updates, as required during training.


Therefore, the model prediction SDK needs to be designed separately and has the following features:

#. The predictive SDK does not include reverse propagation and parameter updates to reduce the size of the SDK.
Copy link
Contributor

Choose a reason for hiding this comment

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

reverse propagation -> backpropagation

#. The predictive SDK does not include reverse propagation and parameter updates to reduce the size of the SDK.
#. The predictive SDK needs a simple user interface for ease of use.
#. Since the input data may have a variety of structures, the format of the input data is clearly and compactly packaged.
#. In order to be compatible with the user's system, the SDK's interface must conform to the C-standard interface.
Copy link
Contributor

Choose a reason for hiding this comment

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

with the user's system -> with user's system

#. Since the input data may have a variety of structures, the format of the input data is clearly and compactly packaged.
#. In order to be compatible with the user's system, the SDK's interface must conform to the C-standard interface.

PaddlePaddle provides C-API to solve the above problem. The use of C-API, we provide the following guidelines:
Copy link
Contributor

Choose a reason for hiding this comment

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

Following are the guidelines to use the C-API:

Update refer to commits and thank you for the suggestion.
Copy link
Contributor

@abhinavarora abhinavarora left a comment

Choose a reason for hiding this comment

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

LGTM!

@shanyi15
Copy link
Collaborator

Thank you so much! @abhinavarora

@shanyi15 shanyi15 merged commit 466f28a into PaddlePaddle:develop Mar 22, 2018
mikeseven added a commit to mikeseven/Paddle that referenced this pull request Mar 22, 2018
* commit '9c35b0dc1ba0ace5acf721685802a21045ea1249': (36 commits)
  Fix dist compile error (PaddlePaddle#9320)
  Fix bug for backward tanspiler when using parallel_do operator. (PaddlePaddle#9282)
  update
  fix transpiler bug
  Update index_en.rst (PaddlePaddle#9286)
  "fix mixed_vector bug" (PaddlePaddle#9319)
  Update index_en.rst (PaddlePaddle#9280)
  Adjust some contents in write_docs_en.rst for Contribue Documentation (PaddlePaddle#9147)
  CMake refine for HIP support. Fix CI.
  Reuduce memory copy when communication between trainer and pserver. (PaddlePaddle#9271)
  Modified build.sh and remove build_doc.sh
  fix doc
  Enhance device context pool (PaddlePaddle#9293)
  Device blobs are created only in training. Added testing attribute
  Shrink batch_norm_grad's inputs
  updates
  prepare and create op before run
  wip
  small fix
  initial commit
  ...

# Conflicts:
#	cmake/external/eigen.cmake
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.

Translation Plan-C-API预测库-汉译英-概述
3 participants