Cleanup

package.json

@@ -4,6 +4,9 @@
 		"rimraf": "^2.6.2"
 	},
 	"scripts": {
+		"build,test": "npm run --silent build && npm run --silent test",
+		"build,test,bench": "npm run --silent build && npm run --silent test && npm run --silent bench",
+		"build,bench": "npm run --silent build && npm run --silent bench",
 		"build": "bash -c '[[ -z \"${PSP_DOCKER}\" ]] && npm run --silent _build || npm run --silent _emsdk -- npm run --silent _build'",
 		"bench:build": "lerna run bench:build ${PACKAGE:+--scope=@jpmorganchase/${PACKAGE}} --stream",
 		"bench:run": "lerna exec ${PACKAGE:+--scope=@jpmorganchase/${PACKAGE}} -- npm run bench:run",

packages/perspective-jupyterlab/src/config/plugin.config.js

@@ -10,8 +10,25 @@
 const path = require('path');
 const common = require('@jpmorganchase/perspective/src/config/common.config.js');
 
+const UglifyJSPlugin = require('uglifyjs-webpack-plugin');
+const webpack = require('webpack');
+
+const plugins = [
+    new webpack.ContextReplacementPlugin(/moment[\/\\]locale$/, /(en|es|fr)$/)
+];
+
+if (!process.env.PSP_NO_MINIFY && !process.env.PSP_DEBUG) {
+    plugins.push(new UglifyJSPlugin({
+        sourceMap: false,
+        mangle: false,
+        output: {
+            ascii_only: true
+        }
+    }));
+}
 module.exports = Object.assign({}, common(), {
     entry: './src/ts/index.ts',
+    plugins: plugins,
     output: {
         filename: 'index.js',
         libraryTarget: "umd",

packages/perspective/package.json

@@ -13,15 +13,16 @@
   ],
   "typings": "index.d.ts",
   "scripts": {
-    "bench:build": "npm-run-all bench:build:*",
+    "bench:build": "npm-run-all -p bench:build:*",
     "bench:build:benchmark": "webpack --color --config bench/config/benchmark.config.js",
     "bench:build:report": "webpack --color --config bench/config/report.config.js",
     "bench:build:copy": "cp bench/csv/* build",    
     "bench:run": "node build/benchmark.js",
     "bench": "npm-run-all bench:build bench:run",
-    "build": "npm-run-all build:compile:* build:webpack:* ",
+    "build": "npm-run-all build:compile:* build:webpack",
     "build:compile:copy": "mkdir -p obj build build/wasm_async build/wasm_sync build/asmjs",
     "build:compile:emmake": "cd obj/ && emcmake cmake ../ && emmake make -j8",
+    "build:webpack": "npm-run-all -p build:webpack:*",
     "build:webpack:asmjs": "webpack --color --config src/config/perspective.asmjs.config.js",
     "build:webpack:wasm": "webpack --color --config src/config/perspective.wasm.config.js",
     "build:webpack:parallel": "webpack --color --config src/config/perspective.parallel.config.js",
@@ -30,7 +31,7 @@
     "test:build:copy": "npm-run-all -p test:build:copy:*",
     "test:build:copy:html": "cp test/html/* build",
     "test:build:copy:test": "cp test/csv/* build",
-    "test:build:webpack": "npm-run-all test:build:webpack:*",
+    "test:build:webpack": "npm-run-all -p test:build:webpack:*",
     "test:build:webpack:browser": "webpack --color --config test/config/test_browser.config.js",
     "test:build:webpack:node": "webpack --color --config test/config/test_node.config.js",
     "docs": "documentation build src/js/perspective.js -f md --shallow > README.md",

packages/perspective/src/cpp/main.cpp

@@ -249,7 +249,7 @@ _fill_col(val dcol, t_col_sptr col, t_bool is_arrow)
     } else {
         for (auto i = 0; i < nrows; ++i)
         {
-            if (dcol[i].equals(val::undefined())) continue;
+            if (dcol[i].isUndefined()) continue;
             auto elem = dcol[i].as<T>();
             col->set_nth(i, elem);
         }
@@ -299,7 +299,7 @@ _fill_col<t_time>(val dcol, t_col_sptr col, t_bool is_arrow)
     } else {
         for (auto i = 0; i < nrows; ++i)
         {
-            if (dcol[i].equals(val::undefined())) continue;
+            if (dcol[i].isUndefined()) continue;
             auto elem = static_cast<t_int64>(dcol[i].as<t_float64>());
             col->set_nth(i, elem);
         }
@@ -324,7 +324,7 @@ _fill_col<t_bool>(val dcol, t_col_sptr col, t_bool is_arrow)
     } else {
         for (auto i = 0; i < nrows; ++i)
         {
-            if (dcol[i].equals(val::undefined())) continue;
+            if (dcol[i].isUndefined()) continue;
             auto elem = dcol[i].as<t_bool>();
             col->set_nth(i, elem);
         }
@@ -381,7 +381,7 @@ _fill_col<std::string>(val dcol, t_col_sptr col, t_bool is_arrow)
     } else {
         for (auto i = 0; i < nrows; ++i)
         {
-            if (dcol[i].equals(val::undefined())) continue;
+            if (dcol[i].isUndefined()) continue;
             std::wstring welem = dcol[i].as<std::wstring>();
             std::wstring_convert<utf16convert_type, wchar_t> converter;
             std::string elem = converter.to_bytes(welem);