[GitHub] [incubator-tvm] jackwish edited a comment on issue #3859: [TOPI][AutoTVM] NHWC conv2d templates for ARM

[GitBox <gi...@apache.org>]

jackwish edited a comment on issue #3859: [TOPI][AutoTVM] NHWC conv2d templates for ARM
URL: https://github.com/apache/incubator-tvm/pull/3859#issuecomment-557033246
 
 
   For the [CI sanity check](http://ci.tvm.ai:8080/blue/organizations/jenkins/tvm/detail/PR-3859/5/pipeline) , I failed to reproduce it locally :(. And, I think the warning (as below) should be ignored because we are really using `cfg['compat'].val` as integer rather than bool.
   
   ```
   python3 -m pylint topi/python/topi --rcfile=/workspace/tests/lint/pylintrc
   
   Using config file /workspace/tests/lint/pylintrc
   
   ************* Module topi.arm_cpu.conv2d_spatial_pack
   
   topi/python/topi/arm_cpu/conv2d_spatial_pack.py:314: [R1706(consider-using-ternary), schedule_conv2d_spatial_pack_nhwc] Consider using ternary (oco if cfg['compat'].val else owo)
   
   topi/python/topi/arm_cpu/conv2d_spatial_pack.py:331: [R1706(consider-using-ternary), schedule_conv2d_spatial_pack_nhwc] Consider using ternary (oco if cfg['compat'].val else owo)
   
   
   
   --------------------------------------------------------------------
   
   Your code has been rated at 10.00/10 (previous run: 10.00/10, -0.00)
   ```
   
   I suggest to disable that warning @tqchen . Anyway, if disabling globally is too strong, I will disable it in this file.

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


With regards,
Apache Git Services