samskalicky commented on a change in pull request #17486: Update CustomOp doc 
with changes for GPU support
URL: https://github.com/apache/incubator-mxnet/pull/17486#discussion_r375023276
 
 

 ##########
 File path: example/extensions/lib_custom_op/README.md
 ##########
 @@ -146,21 +186,160 @@ Let’s take a closer look at those registry functions:
 
 * **inferShape**: This function is similar to the `inferType` function, except 
it is used for populating the output data shapes. You need to figure out the 
shapes of each output tensors for this computation. For example, if the inputs 
are images with shape (224,224,3) and you write a padding operator to make 10px 
borders for the images, then your output shape will be (234,234,3).
 
-* **forward**: This function executes the main forward computation. It takes 
four arguments. The 1st argument is the attributes. The 2nd argument is the 
input `MXTensors` which stores all data and info of input ndarrays. The 3rd 
argument is the output `MXTensors`. The 4th argument is the `OpResource` object 
for memory allocation and other utilities. Additionally, you can use a 
`dltensor` tensor structure stored in the `MXTensor` as a more standardized 
data structure for computing.
+* **forward**: This function executes the main forward computation. It takes 
four arguments. The 1st argument is the attributes. The 2nd argument is the 
input `MXTensors` which stores all data and info of input ndarrays. The 3rd 
argument is the output `MXTensors`. The 4th argument is the `OpResource` object 
for memory allocation and other utilities. The details of `OpResource` are 
covered in the below section.
 
 Review comment:
   "in the below section" --> "in the section below"

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
[email protected]


With regards,
Apache Git Services

Reply via email to