moazreyad commented on a change in pull request #529: SINGA-484 Code analysis 
with LGTM

 File path: include/singa/io/transformer.h
 @@ -56,12 +56,12 @@ class ImageTransformer: public Transformer {
   Tensor Apply(int flag, Tensor& input) override;
-  const bool featurewise_center() const { return featurewise_center_; }
-  const bool featurewise_std_norm() const { return featurewise_std_norm_; }
-  const bool horizontal_mirror() const { return horizontal_mirror_; }
-  const int resize_height() const { return resize_height_; }
-  const int resize_width() const { return resize_width_; }
-  const float rescale() const { return rescale_; }
+  bool featurewise_center() const { return featurewise_center_; }
 Review comment:
   Yes, LGTM suggested to the remove the const because it is useless. More 
information can be found [here]( 
   There are more alerts that I could try to fix, but because the [testing 
pipeline is not working](, I 
can't be sure that my fixes will not cause other problems.
   All the alerts can be found in this 
which is also available in the Jira ticket 
   In this pull request, I propose to add the code analysis badges to SINGA 
readme and I fixed few alerts. But there are many more to be fixed. 

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:

With regards,
Apache Git Services

Reply via email to