ThomasDelteil commented on issue #11752: Update doc for activation symbol
URL: 
https://github.com/apache/incubator-mxnet/issues/11752#issuecomment-408949698
 
 
   @azai91 can you point to the documentation that says that the 
`simple_forward` function accepts `act_data` as an argument?
   
   Current docs as I see it is:
   
   ```
       A simple forward function for a symbol.
   
       Primarily used in doctest to test the functionality of a symbol.
       Takes NumPy arrays as inputs and outputs are also converted to NumPy 
arrays.
   
       Parameters
       ----------
       ctx : Context
           If ``None``, will take the default context.
       inputs : keyword arguments
           Mapping each input name to a NumPy array.
   
       Returns
       -------
       The result as a numpy array. Multiple results will
       be returned as a list of NumPy arrays.
   ```
   
   I think you are confused by the `act_data` argument in the `Activation` 
example. This is a key/value argument that is used to bind the symbol with the 
data. Because in the example the Activation symbol is named `act`, the default 
input symbol should be named `act_data`
   
   For example this would work too:
   
   ```
       op = Activation(data=mx.sym.var('data0'), act_type=act_type, name='act')
       y = test_utils.simple_forward(op, data0=x)
   ```
   
   @sandeep-krishnamurthy can you close please?

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


With regards,
Apache Git Services

Reply via email to