You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@mxnet.apache.org by zh...@apache.org on 2021/03/16 16:42:26 UTC

[incubator-mxnet] branch master updated: Fix for test_lans failure (#20036)

This is an automated email from the ASF dual-hosted git repository.

zhasheng pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/incubator-mxnet.git


The following commit(s) were added to refs/heads/master by this push:
     new b8dc162  Fix for test_lans failure (#20036)
b8dc162 is described below

commit b8dc162dc4c02d5305a4242230799205677e5a5d
Author: Anna Karbownik <69...@users.noreply.github.com>
AuthorDate: Tue Mar 16 17:39:27 2021 +0100

    Fix for test_lans failure (#20036)
    
    Make NumTensors function unique to multi_lans.cc & multi_lamb.cc.
---
 src/operator/contrib/multi_lamb.cc | 2 +-
 src/operator/contrib/multi_lans.cc | 2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/src/operator/contrib/multi_lamb.cc b/src/operator/contrib/multi_lamb.cc
index 6ddc4df..710908d 100644
--- a/src/operator/contrib/multi_lamb.cc
+++ b/src/operator/contrib/multi_lamb.cc
@@ -167,7 +167,7 @@ std::vector<std::string> LAMBParamToVector(uint32_t num_tensors,
   return ret;
 }
 
-inline uint32_t NumTensors(const nnvm::NodeAttrs& attrs) {
+static inline uint32_t NumTensors(const nnvm::NodeAttrs& attrs) {
   return static_cast<uint32_t>(dmlc::get<MultiLAMBParam>(attrs.parsed).num_tensors);
 }
 
diff --git a/src/operator/contrib/multi_lans.cc b/src/operator/contrib/multi_lans.cc
index 37d6001..8cb9080 100644
--- a/src/operator/contrib/multi_lans.cc
+++ b/src/operator/contrib/multi_lans.cc
@@ -183,7 +183,7 @@ std::vector<std::string> LANSParamToVector(uint32_t num_tensors,
   return ret;
 }
 
-inline uint32_t NumTensors(const nnvm::NodeAttrs& attrs) {
+static inline uint32_t NumTensors(const nnvm::NodeAttrs& attrs) {
   return static_cast<uint32_t>(dmlc::get<MultiLANSParam>(attrs.parsed).num_tensors);
 }