singa-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From GitBox <...@apache.org>
Subject [GitHub] [singa] XJDKC edited a comment on pull request #697: New Model Layer Operator API
Date Tue, 02 Jun 2020 02:50:01 GMT

XJDKC edited a comment on pull request #697:
URL: https://github.com/apache/singa/pull/697#issuecomment-637237579


   > > Summary:
   > > ```
   > > * set hx.creator and cx.creator to None. (Still can't use the graph to train
correctly but can be executed normally)
   > > 
   > > * create ReLU layer instance
   > > ```
   > 
   > update conv and linear layers to include an argument for `activation`
   > 
   > > ```
   > > * create Loss layer instance
   > > 
   > > * remove set_attribute function, just copy the initial value from tensor directly.
Raise warning in __setattr__ when the types do not match
   > > ```
   > 
   > Shall we totally disable reassignment, like self.W=..., because it may affect the
graph as it replace the tensor W with another tensor?
   > 
   > > ```
   > > * remove on_device function, get device info from input tensors
   > > ```
   
   But I'm not sure where the reassignment is used in the whole project. Maybe it's used in
many places
   


----------------------------------------------------------------
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:
users@infra.apache.org



Mime
View raw message