singa-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From GitBox <...@apache.org>
Subject [GitHub] [singa] XJDKC commented on a change in pull request #731: fix the bug of param name
Date Tue, 16 Jun 2020 10:05:16 GMT

XJDKC commented on a change in pull request #731:
URL: https://github.com/apache/singa/pull/731#discussion_r440735160



##########
File path: python/singa/layer.py
##########
@@ -499,7 +581,7 @@ def initialize(self, x):
             self.kernel_size[0],
             self.kernel_size[1],
         )
-        w_name = self.name + Layer.sep + 'W'
+        w_name = self.get_param_name('W')

Review comment:
       That's the current implementation.https://github.com/apache/singa/pull/731/files#diff-3240fd3b573c3eaf8f8d6da8470e68e7R232
   If users name a param, the tensor is not a dummy tensor. So we won't change its name.




----------------------------------------------------------------
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