Remove Plotly lib ultimately reducing bundle size

Author: risenW

danfojs/src/core/frame.js

@@ -18,8 +18,8 @@ import { Series } from "./series";
 import { Utils } from "./utils";
 import { GroupBy } from "./groupby";
 import { Plot } from "../plotting/plot";
-import { indexLoc } from "../core/indexing";
-import { concat } from "../core/concat.js";
+import { indexLoc } from "./indexing";
+import { concat } from "./concat.js";
 
 const utils = new Utils();
 import { std, variance } from "mathjs";

danfojs/src/core/series.js

@@ -23,7 +23,7 @@ import { table } from 'table';
 import { Configs } from '../config/config';
 import { TimeSeries } from './timeseries';
 import { Plot } from '../plotting/plot';
-import { indexLoc } from '../core/indexing';
+import { indexLoc } from './indexing';
 
 
 const utils = new Utils();

danfojs/src/plotting/plot.js

@@ -1,10 +1,21 @@
 import { Utils } from "../core/utils";
 import { Series } from "../core/series";
-import Plotly from 'plotly.js-dist';
-
-
 const utils = new Utils();
 
+/**
+ * Change: Rising Odegua 6th Feb 2021
+ * We no longer bundle Plotjs with DanfoJs. I noticed that removing this single
+ * inport reduces the bundle size from 5.7mb to 2.1mb. I know it's still a huge
+ * bundle, but it's progress!
+ * If you need to use Plotly, then you can add the Plotly CDN
+ * to your script before DanfoJs CDN
+ */
+try {
+  const version = Plotly.version;
+  console.info(`Using Plotly version ${version}`);
+} catch (error) {
+  console.info(`Plotly CDN not found. If you need to make Plots, then add the Plotly CDN to your script`);
+}
 
 /**
  * Plotting methods and Functions performed on Series and DataFrames

danfojs/tests/core/generic.js

@@ -1,5 +1,5 @@
 /* eslint-disable no-undef */
-const tf = require("@tensorflow/tfjs-core");
+const tf = require("@tensorflow/tfjs");
 
 describe("Generic (NDFrame)", function () {
   describe("NDframe Created from Array", function () {

lib/bundle.js

@@ -1,60 +1,3 @@
-/*
- * @copyright 2016 Sean Connelly (@voidqk), http://syntheti.cc
- * @license MIT
- * @preserve Project Home: https://github.com/voidqk/polybooljs
- */
-
-/*
-object-assign
-(c) Sindre Sorhus
-@license MIT
-*/
-
-/*!
- * @overview es6-promise - a tiny implementation of Promises/A+.
- * @copyright Copyright (c) 2014 Yehuda Katz, Tom Dale, Stefan Penner and contributors (Conversion to ES6 API by Jake Archibald)
- * @license   Licensed under MIT license
- *            See https://raw.githubusercontent.com/stefanpenner/es6-promise/master/LICENSE
- * @version   v4.2.8+1e68dce6
- */
-
-/*!
- * Determine if an object is a Buffer
- *
- * @author   Feross Aboukhadijeh <https://feross.org>
- * @license  MIT
- */
-
-/*!
- * The buffer module from node.js, for the browser.
- *
- * @author   Feross Aboukhadijeh <feross@feross.org> <http://feross.org>
- * @license  MIT
- */
-
-/*!
- * The buffer module from node.js, for the browser.
- *
- * @author   Feross Aboukhadijeh <https://feross.org>
- * @license  MIT
- */
-
-/*!
- * pad-left <https://github.com/jonschlinkert/pad-left>
- *
- * Copyright (c) 2014-2015, Jon Schlinkert.
- * Licensed under the MIT license.
- */
-
-/*!
- * repeat-string <https://github.com/jonschlinkert/repeat-string>
- *
- * Copyright (c) 2014-2015, Jon Schlinkert.
- * Licensed under the MIT License.
- */
-
-/*! ieee754. BSD-3-Clause License. Feross Aboukhadijeh <https://feross.org/opensource> */
-
 /**
  * @license
  * Copyright 2017 Google LLC. All Rights Reserved.

lib/bundle.js.LICENSE.txt

@@ -21,18 +21,16 @@
     "danfojs/types"
   ],
   "dependencies": {
-    "@babel/runtime": "^7.12.13",
     "@tensorflow/tfjs": "2.8.5",
     "mathjs": "7.5.1",
-    "plotly.js-dist": "^1.58.4",
     "table": "^5.4.6",
     "xlsx": "^0.16.7"
   },
   "scripts": {
     "test": "yarn run build && karma start --single-run --browsers ChromeHeadless karma.conf.js",
     "build": "babel ./danfojs/src -d ./dist --no-comments",
     "lint": "eslint ./danfojs/src",
-    "bundle": "webpack",
+    "bundle": "webpack --mode production",
     "coveralls": "cat ./coverage/lcov.info | ./node_modules/.bin/coveralls",
     "coverage": "nyc report --reporter=text-lcov | coveralls && nyc report --reporter=lcov",
     "patch": "npm version patch"
@@ -62,10 +60,8 @@
   "devDependencies": {
     "@babel/cli": "^7.10.5",
     "@babel/core": "^7.10.5",
-    "@babel/plugin-transform-modules-umd": "^7.10.4",
     "@babel/plugin-transform-runtime": "^7.12.10",
     "@babel/preset-env": "^7.10.4",
-    "@babel/register": "^7.10.1",
     "babel-loader": "^8.1.0",
     "browserify": "16.5.2",
     "chai": "^4.2.0",
@@ -76,16 +72,10 @@
     "karma-chai": "^0.1.0",
     "karma-chrome-launcher": "^3.1.0",
     "karma-mocha": "^2.0.1",
-    "mocha": "^7.2.0",
-    "mocha-lcov-reporter": "^1.3.0",
-    "mocha-loader": "^5.1.2",
-    "mocha-webpack": "^1.1.0",
+    "mocha": "^8.3.0",
     "nyc": "^15.1.0",
-    "parcel-bundler": "^1.12.4",
-    "source-map-loader": "^1.0.0",
-    "transform-loader": "^0.2.4",
     "webpack": "5.21.2",
-    "webpack-cli": "4.5.0"
+    "webpack-cli": "^4.5.0"
   },
   "nyc": {
     "reporter": [