Carpet plot rebase

lib/carpet.js

@@ -0,0 +1,11 @@
+/**
+* Copyright 2012-2017, 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/carpet');

lib/contourcarpet.js

@@ -0,0 +1,11 @@
+/**
+* Copyright 2012-2017, 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/contourcarpet');

lib/index.js

@@ -36,6 +36,10 @@ Plotly.register([
 
  require('./scattermapbox'),
 
+ require('./carpet'),
+ require('./scattercarpet'),
+ require('./contourcarpet'),
+
  require('./ohlc'),
  require('./candlestick')
 ]);

lib/scattercarpet.js

@@ -0,0 +1,11 @@
+/**
+* Copyright 2012-2017, 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/scattercarpet');

package.json

@@ -55,7 +55,6 @@
  "dependencies": {
  "3d-view": "^2.0.0",
  "alpha-shape": "^1.0.0",
- "arraytools": "^1.0.0",
  "color-rgba": "^1.0.4",
  "convex-hull": "^1.0.3",
  "country-regex": "^1.1.0",

src/components/drawing/index.js

@@ -678,3 +678,13 @@ drawing.setPointGroupScale = function(selection, x, y) {
 
  return scale;
 };
+
+drawing.measureText = function(tester, text, font) {
+ var dummyText = tester.append('text')
+ .text(text)
+ .call(drawing.font, font);
+
+ var bbox = drawing.bBox(dummyText.node());
+ dummyText.remove();
+ return bbox;
+};

src/components/legend/draw.js

@@ -512,6 +512,8 @@ function handleClick(g, gd, numClicks) {
 
  for(i = 0; i < fullData.length; i++) {
  allTraces.push(i);
+ // Allow the legendonly state through for *all* trace types (including
+ // carpet for which it's overridden with true/false in supplyDefaults)
  traceVisibility.push('legendonly');
  }
 
@@ -542,7 +544,11 @@ function handleClick(g, gd, numClicks) {
  if(sameAsLast) {
  traceVisibility = true;
  }
- Plotly.restyle(gd, 'visible', traceVisibility, allTraces);
+ var visibilityUpdates = [];
+ for(i = 0; i < fullData.length; i++) {
+ visibilityUpdates.push(allTraces[i]);
+ }
+ Plotly.restyle(gd, 'visible', traceVisibility, visibilityUpdates);
  }
  }
 }

src/components/legend/style.js

@@ -65,6 +65,11 @@ function styleLines(d) {
  showFill = trace.visible && trace.fill && trace.fill !== 'none',
  showLine = subTypes.hasLines(trace);
 
+ if(trace && trace._module && trace._module.name === 'contourcarpet') {
+ showLine = trace.contours.showlines;
+ showFill = trace.contours.coloring === 'fill';
+ }
+
  var fill = d3.select(this).select('.legendfill').selectAll('path')
  .data(showFill ? [d] : []);
  fill.enter().append('path').classed('js-fill', true);

src/lib/ensure_array.js

@@ -0,0 +1,27 @@
+/**
+* Copyright 2012-2017, 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';
+
+/*
+ * Ensures an array has the right amount of storage space. If it doesn't
+ * exist, it creates an array. If it does exist, it returns it if too
+ * short or truncates it in-place.
+ *
+ * The goal is to just reuse memory to avoid a bit of excessive garbage
+ * collection.
+ */
+module.exports = function ensureArray(out, n) {
+ if(!Array.isArray(out)) out = [];
+
+ // If too long, truncate. (If too short, it will grow
+ // automatically so we don't care about that case)
+ out.length = n;
+
+ return out;
+};

src/lib/index.js

@@ -19,6 +19,7 @@ lib.isArray = require('./is_array');
 lib.mod = require('./mod');
 lib.toLogRange = require('./to_log_range');
 lib.relinkPrivateKeys = require('./relink_private');
+lib.ensureArray = require('./ensure_array');
 
 var coerceModule = require('./coerce');
 lib.valObjects = coerceModule.valObjects;

src/plot_api/plot_api.js

@@ -1294,6 +1294,7 @@ function _restyle(gd, aobj, _traces) {
  'autobinx', 'nbinsx', 'xbins', 'xbins.start', 'xbins.end', 'xbins.size',
  'autobiny', 'nbinsy', 'ybins', 'ybins.start', 'ybins.end', 'ybins.size',
  'autocontour', 'ncontours', 'contours', 'contours.coloring',
+ 'contours.operation', 'contours.value', 'contours.type', 'contours.value[0]', 'contours.value[1]',
  'error_y', 'error_y.visible', 'error_y.value', 'error_y.type',
  'error_y.traceref', 'error_y.array', 'error_y.symmetric',
  'error_y.arrayminus', 'error_y.valueminus', 'error_y.tracerefminus',
@@ -1311,9 +1312,31 @@ function _restyle(gd, aobj, _traces) {
  'line.showscale', 'line.cauto', 'line.autocolorscale', 'line.reversescale',
  'marker.line.showscale', 'marker.line.cauto', 'marker.line.autocolorscale', 'marker.line.reversescale',
  'xcalendar', 'ycalendar',
- 'cumulative', 'cumulative.enabled', 'cumulative.direction', 'cumulative.currentbin'
+ 'cumulative', 'cumulative.enabled', 'cumulative.direction', 'cumulative.currentbin',
+ 'a0', 'da', 'b0', 'db', 'atype', 'btype',
+ 'cheaterslope', 'carpet', 'sum',
  ];
 
+ var carpetAxisAttributes = [
+ 'color', 'smoothing', 'title', 'titlefont', 'titlefont.size', 'titlefont.family',
+ 'titlefont.color', 'titleoffset', 'type', 'autorange', 'rangemode', 'range',
+ 'fixedrange', 'cheatertype', 'tickmode', 'nticks', 'tickvals', 'ticktext',
+ 'ticks', 'mirror', 'ticklen', 'tickwidth', 'tickcolor', 'showticklabels',
+ 'tickfont', 'tickfont.size', 'tickfont.family', 'tickfont.color', 'tickprefix',
+ 'showtickprefix', 'ticksuffix', 'showticksuffix', 'showexponent', 'exponentformat',
+ 'separatethousands', 'tickformat', 'categoryorder', 'categoryarray', 'labelpadding',
+ 'labelprefix', 'labelsuffix', 'labelfont', 'labelfont.family', 'labelfont.size',
+ 'labelfont.color', 'showline', 'linecolor', 'linewidth', 'gridcolor', 'gridwidth',
+ 'showgrid', 'minorgridcount', 'minorgridwidth', 'minorgridcolor', 'startline',
+ 'startlinecolor', 'startlinewidth', 'endline', 'endlinewidth', 'endlinecolor',
+ 'tick0', 'dtick', 'arraytick0', 'arraydtick', 'hoverformat', 'tickangle'
+ ];
+
+ for(i = 0; i < carpetAxisAttributes.length; i++) {
+ recalcAttrs.push('aaxis.' + carpetAxisAttributes[i]);
+ recalcAttrs.push('baxis.' + carpetAxisAttributes[i]);
+ }
+
  for(i = 0; i < traces.length; i++) {
  if(Registry.traceIs(fullData[traces[i]], 'box')) {
  recalcAttrs.push('name');
@@ -1657,9 +1680,14 @@ function _restyle(gd, aobj, _traces) {
  doextra(axlist.map(rangeAttr), [0, 1], 0);
  }
  flags.docalc = true;
+
+ } else if(replotAttrs.indexOf(aiAboveArray) !== -1) {
+ flags.doplot = true;
+ } else if(aiAboveArray.indexOf('aaxis') === 0 || aiAboveArray.indexOf('baxis') === 0) {
+ flags.doplot = true;
+ } else if(autorangeAttrs.indexOf(aiAboveArray) !== -1) {
+ flags.docalcAutorange = true;
  }
- else if(replotAttrs.indexOf(aiAboveArray) !== -1) flags.doplot = true;
- else if(autorangeAttrs.indexOf(aiAboveArray) !== -1) flags.docalcAutorange = true;
  }
 
  // do we need to force a recalc?

src/plots/cartesian/axes.js

@@ -1665,6 +1665,8 @@ axes.doTicks = function(gd, axid, skipTitle) {
  ticksign = ticksign.map(function(v) { return -v; });
  }
 
+ if(!ax.visible) return;
+
  // remove zero lines, grid lines, and inside ticks if they're within
  // 1 pixel of the end
  // The key case here is removing zero lines when the axis bound is zero.

src/plots/cartesian/axis_defaults.js

@@ -47,6 +47,8 @@ module.exports = function handleAxisDefaults(containerIn, containerOut, coerce,
  return Lib.coerce2(containerIn, containerOut, layoutAttributes, attr, dflt);
  }
 
+ var visible = coerce('visible', !options.cheateronly);
+
  var axType = containerOut.type;
 
  if(axType === 'date') {
@@ -56,6 +58,20 @@ module.exports = function handleAxisDefaults(containerIn, containerOut, coerce,
 
  setConvert(containerOut, layoutOut);
 
+ var autoRange = coerce('autorange', !containerOut.isValidRange(containerIn.range));
+
+ if(autoRange) coerce('rangemode');
+
+ coerce('range');
+ containerOut.cleanRange();
+
+ handleCategoryOrderDefaults(containerIn, containerOut, coerce);
+ containerOut._initialCategories = axType === 'category' ?
+ orderedCategories(letter, containerOut.categoryorder, containerOut.categoryarray, options.data) :
+ [];
+
+ if(!visible) return containerOut;
+
  var dfltColor = coerce('color');
  // if axis.color was provided, use it for fonts too; otherwise,
  // inherit from global font color in case that was provided.
@@ -68,17 +84,9 @@ module.exports = function handleAxisDefaults(containerIn, containerOut, coerce,
  color: dfltFontColor
  });
 
- var autoRange = coerce('autorange', !containerOut.isValidRange(containerIn.range));
-
- if(autoRange) coerce('rangemode');
-
- coerce('range');
- containerOut.cleanRange();
-
  handleTickValueDefaults(containerIn, containerOut, coerce, axType);
  handleTickLabelDefaults(containerIn, containerOut, coerce, axType, options);
  handleTickMarkDefaults(containerIn, containerOut, coerce, options);
- handleCategoryOrderDefaults(containerIn, containerOut, coerce);
 
  var lineColor = coerce2('linecolor', dfltColor),
  lineWidth = coerce2('linewidth'),
@@ -109,10 +117,5 @@ module.exports = function handleAxisDefaults(containerIn, containerOut, coerce,
  delete containerOut.zerolinewidth;
  }
 
- // fill in categories
- containerOut._initialCategories = axType === 'category' ?
- orderedCategories(letter, containerOut.categoryorder, containerOut.categoryarray, options.data) :
- [];
-
  return containerOut;
 };

src/plots/cartesian/graph_interact.js

@@ -467,6 +467,11 @@ function hover(gd, evt, subplot) {
  if(!cd || !cd[0] || !cd[0].trace || cd[0].trace.visible !== true) continue;
 
  trace = cd[0].trace;
+
+ // Explicitly bail out for these two. I don't know how to otherwise prevent
+ // the rest of this function from running and failing
+ if(['carpet', 'contourcarpet'].indexOf(trace._module.name) !== -1) continue;
+
  subplotId = getSubplot(trace);
  subploti = subplots.indexOf(subplotId);
 

src/plots/cartesian/index.js

@@ -63,8 +63,11 @@ exports.plot = function(gd, traces, transitionOpts, makeOnCompleteCallback) {
  // Skip trace if whitelist provided and it's not whitelisted:
  // if (Array.isArray(traces) && traces.indexOf(i) === -1) continue;
  if(trace.xaxis + trace.yaxis === subplot) {
+ // XXX: Should trace carpet dependencies. Only replot all carpet plots if the carpet
+ // axis has actually changed:
+ //
  // If this trace is specifically requested, add it to the list:
- if(traces.indexOf(trace.index) !== -1) {
+ if(traces.indexOf(trace.index) !== -1 || trace.carpet) {
  // Okay, so example: traces 0, 1, and 2 have fill = tonext. You animate
  // traces 0 and 2. Trace 1 also needs to be updated, otherwise its fill
  // is outdated. So this retroactively adds the previous trace if the
@@ -300,6 +303,7 @@ function makeSubplotLayer(plotinfo) {
  joinLayer(parent, 'g', 'imagelayer');
  joinLayer(parent, 'g', 'maplayer');
  joinLayer(parent, 'g', 'barlayer');
+ joinLayer(parent, 'g', 'carpetlayer');
  joinLayer(parent, 'g', 'boxlayer');
  joinLayer(parent, 'g', 'scatterlayer');
  }

src/plots/cartesian/layout_attributes.js

@@ -16,6 +16,15 @@ var constants = require('./constants');
 
 
 module.exports = {
+ visible: {
+ valType: 'boolean',
+ role: 'info',
+ description: [
+ 'A single toggle to hide the axis while preserving interaction like dragging.',
+ 'Default is true when a cheater plot is present on the axis, otherwise',
+ 'false'
+ ].join(' ')
+ },
  color: {
  valType: 'color',
  dflt: colorAttrs.defaultLine,

src/plots/cartesian/layout_defaults.js

@@ -29,6 +29,8 @@ module.exports = function supplyLayoutDefaults(layoutIn, layoutOut, fullData) {
  yaListCartesian = [],
  xaListGl2d = [],
  yaListGl2d = [],
+ xaListCheater = [],
+ xaListNonCheater = [],
  outerTicks = {},
  noGrids = {},
  i;
@@ -51,6 +53,21 @@ module.exports = function supplyLayoutDefaults(layoutIn, layoutOut, fullData) {
  var xaName = axisIds.id2name(trace.xaxis),
  yaName = axisIds.id2name(trace.yaxis);
 
+ // Two things trigger axis visibility:
+ // 1. is not carpet
+ // 2. carpet that's not cheater
+ if(!Registry.traceIs(trace, 'carpet') || (trace.type === 'carpet' && !trace._cheater)) {
+ if(xaName) Lib.pushUnique(xaListNonCheater, xaName);
+ }
+
+ // The above check for definitely-not-cheater is not adequate. This
+ // second list tracks which axes *could* be a cheater so that the
+ // full condition triggering hiding is:
+ // *could* be a cheater and *is not definitely visible*
+ if(trace.type === 'carpet' && trace._cheater) {
+ if(xaName) Lib.pushUnique(xaListCheater, xaName);
+ }
+
  // add axes implied by traces
  if(xaName && listX.indexOf(xaName) === -1) listX.push(xaName);
  if(yaName && listY.indexOf(yaName) === -1) listY.push(yaName);
@@ -168,7 +185,8 @@ module.exports = function supplyLayoutDefaults(layoutIn, layoutOut, fullData) {
  showGrid: !noGrids[axName],
  data: fullData,
  bgColor: bgColor,
- calendar: layoutOut.calendar
+ calendar: layoutOut.calendar,
+ cheateronly: axLetter === 'x' && (xaListCheater.indexOf(axName) !== -1 && xaListNonCheater.indexOf(axName) === -1)
  };
 
  handleAxisDefaults(axLayoutIn, axLayoutOut, coerce, defaultOptions, layoutOut);

src/plots/gl3d/layout/axis_attributes.js

@@ -14,6 +14,7 @@ var extendFlat = require('../../../lib/extend').extendFlat;
 
 
 module.exports = {
+ visible: axesAttrs.visible,
  showspikes: {
  valType: 'boolean',
  role: 'info',