reimplement pointcloud trace type

Author: archmoj

lib/index-gl2d.js

@@ -13,6 +13,7 @@ var Plotly = require('./core');
 Plotly.register([
     require('./scattergl'),
     require('./splom'),
+    require('./pointcloud'),
     require('./heatmapgl'),
     require('./parcoords')
 ]);

lib/index.js

@@ -44,6 +44,7 @@ Plotly.register([
     require('./scattergl'),
     require('./splom'),
 
+    require('./pointcloud'),
     require('./heatmapgl'),
 
     require('./parcoords'),

lib/pointcloud.js

@@ -0,0 +1,11 @@
+/**
+* Copyright 2012-2021, Plotly, Inc.
+* All rights reserved.
+*
+* This source code is licensed under the MIT license found in the
+* LICENSE file in the root directory of this source tree.
+*/
+
+'use strict';
+
+module.exports = require('../src/traces/pointcloud');

package-lock.json

@@ -87,6 +87,7 @@
     "gl-mesh3d": "^2.3.1",
     "gl-plot2d": "^1.4.5",
     "gl-plot3d": "^2.4.7",
+    "gl-pointcloud2d": "^1.0.3",
     "gl-scatter3d": "^1.2.3",
     "gl-select-box": "^1.0.4",
     "gl-spikes2d": "^1.0.2",

package.json

@@ -237,7 +237,7 @@ module.exports = function draw(gd) {
         var subplotObj = fullLayout._plots[subplot];
 
         // filter out overlaid plots (which have their images on the main plot)
-        // and heatmapgl plots (which don't support below images, at least not yet)
+        // and gl2d plots (which don't support below images, at least not yet)
         if(!subplotObj.imagelayer) continue;
 
         var imagesOnSubplot = subplotObj.imagelayer.selectAll('image')

src/components/images/draw.js

@@ -332,15 +332,15 @@ function layoutHeadAttr(fullLayout, head) {
             _module = basePlotModules[i];
             if(_module.attrRegex && _module.attrRegex.test(head)) {
                 // if a module defines overrides, these take precedence
-                // initially this was to allow heatmapgl different editTypes from svg cartesian
+                // initially this is to allow gl2d different editTypes from svg cartesian
                 if(_module.layoutAttrOverrides) return _module.layoutAttrOverrides;
 
                 // otherwise take the first attributes we find
                 if(!out && _module.layoutAttributes) out = _module.layoutAttributes;
             }
 
             // a module can also override the behavior of base (and component) module layout attrs
-            // again see heatmapgl for initial use case
+            // again see gl2d for initial use case
             var baseOverrides = _module.baseLayoutAttrOverrides;
             if(baseOverrides && head in baseOverrides) return baseOverrides[head];
         }

src/plot_api/plot_schema.js

@@ -68,7 +68,7 @@ function lsInner(gd) {
     exports.drawMainTitle(gd);
     ModeBar.manage(gd);
 
-    // _has('cartesian') means SVG specifically, not heatmapgl - but heatmapgl
+    // _has('cartesian') means SVG specifically, not GL2D - but GL2D
     // can still get here because it makes some of the SVG structure
     // for shared features like selections.
     if(!fullLayout._has('cartesian')) {

src/plot_api/subroutines.js

@@ -102,7 +102,7 @@ axes.coerceRef = function(containerIn, containerOut, gd, attr, dflt, extraOption
     if(!extraOption) extraOption = dflt;
     axlist = axlist.concat(axlist.map(function(x) { return x + ' domain'; }));
 
-    // data-ref annotations are not supported in heatmapgl yet
+    // data-ref annotations are not supported in gl2d yet
 
     attrDef[refAttr] = {
         valType: 'enumerated',

src/plots/cartesian/axes.js

@@ -35,7 +35,7 @@ module.exports = function makeIncludeComponents(containerArrayName) {
         var xaList = subplots.xaxis;
         var yaList = subplots.yaxis;
         var cartesianList = subplots.cartesian;
-        var hasCartesianOrHeatmapgl = layoutOut._has('cartesian') || layoutOut._has('gl2d');
+        var hasCartesianOrGL2D = layoutOut._has('cartesian') || layoutOut._has('gl2d');
 
         for(var i = 0; i < array.length; i++) {
             var itemi = array[i];
@@ -49,7 +49,7 @@ module.exports = function makeIncludeComponents(containerArrayName) {
             var hasXref = idRegex.x.test(xref);
             var hasYref = idRegex.y.test(yref);
             if(hasXref || hasYref) {
-                if(!hasCartesianOrHeatmapgl) Lib.pushUnique(layoutOut._basePlotModules, Cartesian);
+                if(!hasCartesianOrGL2D) Lib.pushUnique(layoutOut._basePlotModules, Cartesian);
 
                 var newAxis = false;
                 if(hasXref && xaList.indexOf(xref) === -1) {