Remove all unused variables and enable lint rule (#26)

* remove all unused variables and enable lint rule

* revert benchmarks deletion

* further reduce unused vars using tsconfig settings

Author: dsmilkov

.vscode/settings.json

@@ -7,7 +7,6 @@
     "dist/": true,
     "**/bundle.js": true
   },
-  "tslint.configFile": "tslint.json",
   "tslint.enable": true,
   "tslint.run": "onType",
   "editor.tabSize": 2,

.vscode/tasks.json

@@ -5,10 +5,10 @@
             "command": "npm",
             "taskName": "lint",
             "type": "shell",
-            "suppressTaskName": true,
             "args": [ "run", "lint"],
             "problemMatcher": {
                 "base": "$tslint5",
+                "owner": "tslint-type-checked",
                 "fileLocation": "absolute"
             }
         }

demos/benchmarks/conv_transpose_gpu_benchmark.ts

@@ -68,8 +68,7 @@ export const BENCHMARK_TEST: BenchmarkTest = (size: number) => {
         gpgpu, program, xTex, wTex, null, resultTex, resultTexRC);
   }
 
-  const y = gpgpu.downloadMatrixFromTexture(
-      resultTex, resultTexRC[0], resultTexRC[1]);
+  gpgpu.downloadMatrixFromTexture(resultTex, resultTexRC[0], resultTexRC[1]);
 
   const end = performance.now();
 

demos/benchmarks/logsumexp_gpu_benchmark.ts

@@ -15,7 +15,6 @@ limitations under the License.
 
 import {GPGPUContext} from '../../src/math/webgl/gpgpu_context';
 import {LogSumExpProgram} from '../../src/math/webgl/logsumexp_gpu';
-import * as test_util from '../../src/test_util';
 import * as gpgpu_math from '../../src/math/webgl/gpgpu_math';
 import {Scalar, Array2D, initializeGPU} from '../../src/math/ndarray';
 import {TextureManager} from '../../src/math/webgl/texture_manager';

demos/benchmarks/math-benchmark-run-groups.ts

@@ -18,11 +18,9 @@ import * as conv_gpu_benchmark from './conv_gpu_benchmark';
 import * as conv_transpose_gpu_benchmark from './conv_transpose_gpu_benchmark';
 import * as logsumexp_cpu_benchmark from './logsumexp_cpu_benchmark';
 import * as logsumexp_gpu_benchmark from './logsumexp_gpu_benchmark';
-import * as max_pool_backprop_gpu_benchmark from './max_pool_backprop_gpu_benchmark';
 import * as max_pool_gpu_benchmark from './max_pool_gpu_benchmark';
 import * as mulmat_cpu_benchmark from './mulmat_cpu_benchmark';
 import * as mulmat_gpu_benchmark from './mulmat_gpu_benchmark';
-import * as tex_util_benchmark from './tex_util_benchmark';
 
 export const BENCHMARK_RUN_GROUPS: BenchmarkRunGroup[] = [
   {

demos/benchmarks/math-benchmark.ts

@@ -16,14 +16,13 @@ limitations under the License.
 import '../demo-header';
 import '../demo-footer';
 
-// tslint:disable-next-line:no-unused-variable
 import {PolymerElement, PolymerHTMLElement} from '../polymer-spec';
 import {BenchmarkRunGroup} from './benchmark';
 
 import {BENCHMARK_RUN_GROUPS} from './math-benchmark-run-groups';
 
 // tslint:disable-next-line:variable-name
-export let MathBenchmarkPolymer = PolymerElement(
+export let MathBenchmarkPolymer: new () => PolymerHTMLElement = PolymerElement(
     {is: 'math-benchmark', properties: {benchmarkRunGroupNames: Array}});
 
 export class MathBenchmark extends MathBenchmarkPolymer {

demos/benchmarks/mulmat_gpu_benchmark.ts

@@ -49,8 +49,7 @@ export const BENCHMARK_TEST: BenchmarkTest = (size: number) => {
   for (let i = 0; i < OP_RUNS; i++) {
     gpgpu_math.runProgram(binary);
   }
-
-  const actual = gpgpu.downloadMatrixFromTexture(resultTexture, size, size);
+  gpgpu.downloadMatrixFromTexture(resultTexture, size, size);
   const avgTime = (performance.now() - start) / OP_RUNS;
 
   gpgpu.deleteMatrixTexture(aTexture);
@@ -83,8 +82,7 @@ export const BENCHMARK_TEST_PACKED: BenchmarkTest = (size: number) => {
         gpgpu, program, aTexture, bTexture, resultTexture, [size, size]);
   }
 
-  const actual =
-      gpgpu.downloadMatrixFromPackedTexture(resultTexture, size, size);
+  gpgpu.downloadMatrixFromPackedTexture(resultTexture, size, size);
   const avgTime = (performance.now() - start) / OP_RUNS;
 
   gpgpu.deleteMatrixTexture(aTexture);

demos/benchmarks/tex_util_benchmark.ts

@@ -13,7 +13,6 @@ See the License for the specific language governing permissions and
 limitations under the License.
 ==============================================================================*/
 
-import * as gpgpu_util from '../../src/math/webgl/gpgpu_util';
 import * as tex_util from '../../src/math/webgl/tex_util';
 import * as webgl_util from '../../src/math/webgl/webgl_util';
 import * as test_util from '../../src/test_util';

demos/homepage/index.ts

@@ -17,16 +17,18 @@ import {NDArrayMathGPU} from '../deeplearnjs';
 import {ActivationFunction, ColorMode, CPPN} from '../nn-art/cppn';
 
 function isSafari() {
-  const ua = navigator.userAgent.toLowerCase(); 
-  if (ua.indexOf('safari') != -1) { 
+  const ua = navigator.userAgent.toLowerCase();
+  if (ua.indexOf('safari') !== -1) {
     if (ua.indexOf('chrome') > -1) {
       return false;
     } else {
       return true;
     }
   }
+  return false;
 }
 function isMobile() {
+  // tslint:disable-next-line:no-any
   const a = navigator.userAgent||navigator.vendor||(window as any).opera;
   return /(android|bb\d+|meego).+mobile|avantgo|bada\/|blackberry|blazer|compal|elaine|fennec|hiptop|iemobile|ip(hone|od)|iris|kindle|lge |maemo|midp|mmp|mobile.+firefox|netfront|opera m(ob|in)i|palm( os)?|phone|p(ixi|re)\/|plucker|pocket|psp|series(4|6)0|symbian|treo|up\.(browser|link)|vodafone|wap|windows ce|xda|xiino/i.test(a)||/1207|6310|6590|3gso|4thp|50[1-6]i|770s|802s|a wa|abac|ac(er|oo|s\-)|ai(ko|rn)|al(av|ca|co)|amoi|an(ex|ny|yw)|aptu|ar(ch|go)|as(te|us)|attw|au(di|\-m|r |s )|avan|be(ck|ll|nq)|bi(lb|rd)|bl(ac|az)|br(e|v)w|bumb|bw\-(n|u)|c55\/|capi|ccwa|cdm\-|cell|chtm|cldc|cmd\-|co(mp|nd)|craw|da(it|ll|ng)|dbte|dc\-s|devi|dica|dmob|do(c|p)o|ds(12|\-d)|el(49|ai)|em(l2|ul)|er(ic|k0)|esl8|ez([4-7]0|os|wa|ze)|fetc|fly(\-|_)|g1 u|g560|gene|gf\-5|g\-mo|go(\.w|od)|gr(ad|un)|haie|hcit|hd\-(m|p|t)|hei\-|hi(pt|ta)|hp( i|ip)|hs\-c|ht(c(\-| |_|a|g|p|s|t)|tp)|hu(aw|tc)|i\-(20|go|ma)|i230|iac( |\-|\/)|ibro|idea|ig01|ikom|im1k|inno|ipaq|iris|ja(t|v)a|jbro|jemu|jigs|kddi|keji|kgt( |\/)|klon|kpt |kwc\-|kyo(c|k)|le(no|xi)|lg( g|\/(k|l|u)|50|54|\-[a-w])|libw|lynx|m1\-w|m3ga|m50\/|ma(te|ui|xo)|mc(01|21|ca)|m\-cr|me(rc|ri)|mi(o8|oa|ts)|mmef|mo(01|02|bi|de|do|t(\-| |o|v)|zz)|mt(50|p1|v )|mwbp|mywa|n10[0-2]|n20[2-3]|n30(0|2)|n50(0|2|5)|n7(0(0|1)|10)|ne((c|m)\-|on|tf|wf|wg|wt)|nok(6|i)|nzph|o2im|op(ti|wv)|oran|owg1|p800|pan(a|d|t)|pdxg|pg(13|\-([1-8]|c))|phil|pire|pl(ay|uc)|pn\-2|po(ck|rt|se)|prox|psio|pt\-g|qa\-a|qc(07|12|21|32|60|\-[2-7]|i\-)|qtek|r380|r600|raks|rim9|ro(ve|zo)|s55\/|sa(ge|ma|mm|ms|ny|va)|sc(01|h\-|oo|p\-)|sdk\/|se(c(\-|0|1)|47|mc|nd|ri)|sgh\-|shar|sie(\-|m)|sk\-0|sl(45|id)|sm(al|ar|b3|it|t5)|so(ft|ny)|sp(01|h\-|v\-|v )|sy(01|mb)|t2(18|50)|t6(00|10|18)|ta(gt|lk)|tcl\-|tdg\-|tel(i|m)|tim\-|t\-mo|to(pl|sh)|ts(70|m\-|m3|m5)|tx\-9|up(\.b|g1|si)|utst|v400|v750|veri|vi(rg|te)|vk(40|5[0-3]|\-v)|vm40|voda|vulc|vx(52|53|60|61|70|80|81|83|85|98)|w3c(\-| )|webc|whit|wi(g |nc|nw)|wmlb|wonu|x700|yas\-|your|zeto|zte\-/i.test(a.substr(0,4));
 }
@@ -56,7 +58,6 @@ if (enableCPPN) {
 }
 
 function startCPPN() {
-  const MAX_Z_SCALE = 400;
   const DEFAULT_Z_SCALE = 1;
   const NUM_NEURONS = 30;
   const DEFAULT_NUM_LAYERS = 2;

demos/imagenet/imagenet-demo.ts

@@ -15,15 +15,13 @@ limitations under the License.
 
 import '../demo-header';
 import '../demo-footer';
-import {Array3D, gpgpu_util, GPGPUContext, NDArray, NDArrayMathCPU, NDArrayMathGPU} from '../deeplearnjs';
-import * as imagenet_classes from '../models/imagenet_classes';
+import {gpgpu_util, GPGPUContext, NDArrayMathCPU, NDArrayMathGPU} from '../deeplearnjs';
 import * as imagenet_util from '../models/imagenet_util';
 import {SqueezeNet} from '../models/squeezenet';
-// tslint:disable-next-line:no-unused-variable
 import {PolymerElement, PolymerHTMLElement} from '../polymer-spec';
 
 // tslint:disable-next-line:variable-name
-export const ImagenetDemoPolymer = PolymerElement({
+export const ImagenetDemoPolymer: new () => PolymerHTMLElement = PolymerElement({
   is: 'imagenet-demo',
   properties: {
     layerNames: Array,
@@ -39,14 +37,11 @@ export const ImagenetDemoPolymer = PolymerElement({
  *     http://localhost:5432
  */
 
-const NUM_CLASSES = 1000;
 const IMAGE_SIZE = 227;
 const TOP_K_CLASSES = 5;
 
 const INPUT_NAMES = ['cat', 'dog1', 'dog2', 'beerbottle', 'piano', 'saxophone'];
 export class ImagenetDemo extends ImagenetDemoPolymer {
-  private variables: {[varName: string]: NDArray};
-
   private math: NDArrayMathGPU;
   private mathCPU: NDArrayMathCPU;
   private gl: WebGLRenderingContext;