Skip to content

sankey: implement node grouping via mouse selection #3712

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 8 commits into from
Apr 8, 2019
11 changes: 11 additions & 0 deletions src/components/modebar/buttons.js
Original file line number Diff line number Diff line change
Expand Up @@ -513,6 +513,17 @@ function toggleHover(gd) {
Registry.call('_guiRelayout', gd, 'hovermode', newHover);
}

modeBarButtons.resetSankeyGroup = {
name: 'resetSankeyGroup',
title: function(gd) { return _(gd, 'Ungroup all nodes'); },
icon: Icons.home,
click: function(gd) {
Registry.call('restyle', gd, {
'node.groups': [[]],
Copy link
Contributor

@etpinard etpinard Apr 4, 2019

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

As we discussed in person, it might be better to restyle back to the "initial" view rather than to just remove all the groups.

To do so:

  • stash the 'initial' node.(x|y) and groups in fullData[i] (see mapbox example)
  • call restyle with the values in that stash each in the modebar button handler (see example)

Copy link
Contributor

@etpinard etpinard Apr 8, 2019

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@antoinerg let me know if you need help implementing this 😄

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

call restyle with the values in that stash each in the modebar button handler (see example)

I don't think I can reuse resetView because it does a relayout whereas we need a restyle.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Right, no need to use resetView() found in modebar/buttons.js, but you should try to replicate the same pattern.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Done in commit 506da4c! Let me know if it is satisfactory.

I will now write Jasmine tests for the reset button. Is there anything else you would like me to add?

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I will now write Jasmine tests for the reset button. Is there anything else you would like me to add?

Nop, once that jasmine test is in, this one will be good to go!

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Done in f8a4073

});
}
};

// buttons when more then one plot types are present

modeBarButtons.toggleHover = {
Expand Down
4 changes: 4 additions & 0 deletions src/components/modebar/manage.js
Original file line number Diff line number Diff line change
Expand Up @@ -86,6 +86,7 @@ function getButtonGroups(gd, buttonsToRemove, buttonsToAdd, showSendToCloud) {
var hasTernary = fullLayout._has('ternary');
var hasMapbox = fullLayout._has('mapbox');
var hasPolar = fullLayout._has('polar');
var hasSankey = fullLayout._has('sankey');
var allAxesFixed = areAllAxesFixed(fullLayout);

var groups = [];
Expand Down Expand Up @@ -139,6 +140,9 @@ function getButtonGroups(gd, buttonsToRemove, buttonsToAdd, showSendToCloud) {
else if(hasPie) {
hoverGroup = ['hoverClosestPie'];
}
else if(hasSankey) {
hoverGroup = ['resetSankeyGroup'];
}
else { // hasPolar, hasTernary
// always show at least one hover icon.
hoverGroup = ['toggleHover'];
Expand Down
9 changes: 9 additions & 0 deletions src/plots/cartesian/select.js
Original file line number Diff line number Diff line change
Expand Up @@ -48,6 +48,8 @@ function prepSelect(e, startX, startY, dragOptions, mode) {
var allAxes = dragOptions.xaxes.concat(dragOptions.yaxes);
var subtract = e.altKey;

var doneFnCompleted = dragOptions.doneFnCompleted;

var filterPoly, selectionTester, mergedPolygons, currentPolygon;
var i, searchInfo, eventData;

Expand Down Expand Up @@ -284,6 +286,8 @@ function prepSelect(e, startX, startY, dragOptions, mode) {
dragOptions.mergedPolygons.length = 0;
[].push.apply(dragOptions.mergedPolygons, mergedPolygons);
}

doneFnCompleted(selection);
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I like this solution a lot. Very clean 🥇

});
};
}
Expand Down Expand Up @@ -519,6 +523,11 @@ function determineSearchTraces(gd, xAxes, yAxes, subplot) {
var info = createSearchInfo(trace._module, cd, xAxes[0], yAxes[0]);
info.scene = gd._fullLayout._splomScenes[trace.uid];
searchTraces.push(info);
} else if(
trace.type === 'sankey'
) {
var sankeyInfo = createSearchInfo(trace._module, cd, xAxes[0], yAxes[0]);
searchTraces.push(sankeyInfo);
} else {
if(xAxisIds.indexOf(trace.xaxis) === -1) continue;
if(yAxisIds.indexOf(trace.yaxis) === -1) continue;
Expand Down
101 changes: 101 additions & 0 deletions src/traces/sankey/base_plot.js
Original file line number Diff line number Diff line change
Expand Up @@ -13,6 +13,12 @@ var getModuleCalcData = require('../../plots/get_data').getModuleCalcData;
var plot = require('./plot');
var fxAttrs = require('../../components/fx/layout_attributes');

var setCursor = require('../../lib/setcursor');
var dragElement = require('../../components/dragelement');
var prepSelect = require('../../plots/cartesian/select').prepSelect;
var Lib = require('../../lib');
var Registry = require('../../registry');

var SANKEY = 'sankey';

exports.name = SANKEY;
Expand All @@ -24,6 +30,7 @@ exports.baseLayoutAttrOverrides = overrideAll({
exports.plot = function(gd) {
var calcData = getModuleCalcData(gd.calcdata, SANKEY)[0];
plot(gd, calcData);
exports.updateFx(gd);
};

exports.clean = function(newFullData, newFullLayout, oldFullData, oldFullLayout) {
Expand All @@ -32,5 +39,99 @@ exports.clean = function(newFullData, newFullLayout, oldFullData, oldFullLayout)

if(hadPlot && !hasPlot) {
oldFullLayout._paperdiv.selectAll('.sankey').remove();
oldFullLayout._paperdiv.selectAll('.bgsankey').remove();
}
};

exports.updateFx = function(gd) {
for(var i = 0; i < gd._fullData.length; i++) {
subplotUpdateFx(gd, i);
}
};

function subplotUpdateFx(gd, index) {
var trace = gd._fullData[index];
var fullLayout = gd._fullLayout;

var dragMode = fullLayout.dragmode;
var cursor = fullLayout.dragmode === 'pan' ? 'move' : 'crosshair';
var bgRect = trace._bgRect;

if(dragMode === 'pan' || dragMode === 'zoom') return;

setCursor(bgRect, cursor);

var xaxis = {
_id: 'x',
c2p: Lib.identity,
_offset: trace._sankey.translateX,
_length: trace._sankey.width
};
var yaxis = {
_id: 'y',
c2p: Lib.identity,
_offset: trace._sankey.translateY,
_length: trace._sankey.height
};

// Note: dragOptions is needed to be declared for all dragmodes because
// it's the object that holds persistent selection state.
var dragOptions = {
gd: gd,
element: bgRect.node(),
plotinfo: {
id: index,
xaxis: xaxis,
yaxis: yaxis,
fillRangeItems: Lib.noop
},
subplot: index,
// create mock x/y axes for hover routine
xaxes: [xaxis],
yaxes: [yaxis],
doneFnCompleted: function(selection) {
var traceNow = gd._fullData[index];
var newGroups;
var oldGroups = traceNow.node.groups.slice();
var newGroup = [];

function findNode(pt) {
var nodes = traceNow._sankey.graph.nodes;
for(var i = 0; i < nodes.length; i++) {
if(nodes[i].pointNumber === pt) return nodes[i];
}
}

for(var j = 0; j < selection.length; j++) {
var node = findNode(selection[j].pointNumber);
if(!node) continue;

// If the node represents a group
if(node.group) {
// Add all its children to the current selection
for(var k = 0; k < node.childrenNodes.length; k++) {
newGroup.push(node.childrenNodes[k].pointNumber);
}
// Flag group for removal from existing list of groups
oldGroups[node.pointNumber - traceNow.node._count] = false;
} else {
newGroup.push(node.pointNumber);
}
}

newGroups = oldGroups
.filter(Boolean)
.concat([newGroup]);

Registry.call('_guiRestyle', gd, {
'node.groups': [ newGroups ]
}, index);
}
};

dragOptions.prepFn = function(e, startX, startY) {
prepSelect(e, startX, startY, dragOptions, dragMode);
};

dragElement.init(dragOptions);
}
1 change: 1 addition & 0 deletions src/traces/sankey/calc.js
Original file line number Diff line number Diff line change
Expand Up @@ -40,6 +40,7 @@ function convertToD3Sankey(trace) {
if(linkSpec.target[i] > maxNodeId) maxNodeId = linkSpec.target[i];
}
var nodeCount = maxNodeId + 1;
trace.node._count = nodeCount;

// Group nodes
var j;
Expand Down
1 change: 1 addition & 0 deletions src/traces/sankey/index.js
Original file line number Diff line number Diff line change
Expand Up @@ -18,6 +18,7 @@ Plot.plot = require('./plot');
Plot.moduleType = 'trace';
Plot.name = 'sankey';
Plot.basePlotModule = require('./base_plot');
Plot.selectPoints = require('./select.js');
Plot.categories = ['noOpacity'];
Plot.meta = {
description: [
Expand Down
22 changes: 21 additions & 1 deletion src/traces/sankey/render.js
Original file line number Diff line number Diff line change
Expand Up @@ -836,6 +836,25 @@ module.exports = function(gd, svg, calcData, layout, callbacks) {
.style('pointer-events', 'auto')
.attr('transform', sankeyTransform);

sankey.each(function(d, i) {
gd._fullData[i]._sankey = d;
// Create dragbox if missing
var dragboxClassName = 'bgsankey-' + d.trace.uid + '-' + i;
Lib.ensureSingle(gd._fullLayout._draggers, 'rect', dragboxClassName);

gd._fullData[i]._bgRect = d3.select('.' + dragboxClassName);

// Style dragbox
gd._fullData[i]._bgRect
.style('pointer-events', 'all')
.attr('width', d.width)
.attr('height', d.height)
.attr('x', d.translateX)
.attr('y', d.translateY)
.classed('bgsankey', true)
.style({fill: 'transparent', 'stroke-width': 0});
});

sankey.transition()
.ease(c.ease).duration(c.duration)
.attr('transform', sankeyTransform);
Expand Down Expand Up @@ -925,7 +944,8 @@ module.exports = function(gd, svg, calcData, layout, callbacks) {
.call(attachPointerEvents, sankey, callbacks.nodeEvents)
.call(attachDragHandler, sankeyLink, callbacks, gd); // has to be here as it binds sankeyLink

sankeyNode.transition()
sankeyNode
.transition()
.ease(c.ease).duration(c.duration)
.call(updateNodePositions)
.style('opacity', function(n) { return n.partOfGroup ? 0 : 1;});
Expand Down
36 changes: 36 additions & 0 deletions src/traces/sankey/select.js
Original file line number Diff line number Diff line change
@@ -0,0 +1,36 @@
/**
* Copyright 2012-2019, 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 = function selectPoints(searchInfo, selectionTester) {
var cd = searchInfo.cd;
var selection = [];
var fullData = cd[0].trace;

var nodes = fullData._sankey.graph.nodes;

for(var i = 0; i < nodes.length; i++) {
var node = nodes[i];
if(node.partOfGroup) continue; // Those are invisible

// Position of node's centroid
var pos = [(node.x0 + node.x1) / 2, (node.y0 + node.y1) / 2];
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Voting 👍 on centroid.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ok 👍 I 🔪 the TODO in 20d8c92

Maybe we should add a visual cue to let the user know a given node is now part of a selection 🤔 This could also be added later on.


// Swap x and y if trace is vertical
if(fullData.orientation === 'v') pos.reverse();

if(selectionTester && selectionTester.contains(pos, false, i, searchInfo)) {
selection.push({
pointNumber: node.pointNumber
// TODO: add eventData
});
}
}
return selection;
};
14 changes: 14 additions & 0 deletions test/jasmine/tests/sankey_test.js
Original file line number Diff line number Diff line change
Expand Up @@ -379,6 +379,20 @@ describe('sankey tests', function() {
});
});

it('Plotly.deleteTraces removes draggers', function(done) {
var mockCopy = Lib.extendDeep({}, mock);
Plotly.plot(gd, mockCopy)
.then(function() {
expect(document.getElementsByClassName('bgsankey').length).toBe(1);
return Plotly.deleteTraces(gd, [0]);
})
.then(function() {
expect(document.getElementsByClassName('bgsankey').length).toBe(0);
})
.catch(failTest)
.then(done);
});

it('Plotly.plot does not show Sankey if \'visible\' is false', function(done) {
var mockCopy = Lib.extendDeep({}, mock);

Expand Down
60 changes: 60 additions & 0 deletions test/jasmine/tests/select_test.js
Original file line number Diff line number Diff line change
Expand Up @@ -13,6 +13,8 @@ var mouseEvent = require('../assets/mouse_event');
var touchEvent = require('../assets/touch_event');

var LONG_TIMEOUT_INTERVAL = 5 * jasmine.DEFAULT_TIMEOUT_INTERVAL;
var delay = require('../assets/delay');
var sankeyConstants = require('@src/traces/sankey/constants');

function drag(path, options) {
var len = path.length;
Expand Down Expand Up @@ -2666,6 +2668,64 @@ describe('Test select box and lasso per trace:', function() {
.catch(failTest)
.then(done);
});

describe('should work on sankey traces', function() {
var waitingTime = sankeyConstants.duration * 2;

it('@flaky select', function(done) {
var fig = Lib.extendDeep({}, require('@mocks/sankey_circular.json'));
fig.layout.dragmode = 'select';
var dblClickPos = [250, 400];

Plotly.plot(gd, fig)
.then(function() {
// No groups initially
expect(gd._fullData[0].node.groups).toEqual([]);
})
.then(function() {
// Grouping the two nodes on the top right
return _run(
[[640, 130], [400, 450]],
function() {
expect(gd._fullData[0].node.groups).toEqual([[2, 3]], 'failed to group #2 + #3');
},
dblClickPos, BOXEVENTS, 'for top right nodes #2 and #3'
);
})
.then(delay(waitingTime))
.then(function() {
// Grouping node #4 and the previous group
drag([[715, 400], [300, 110]]);
})
.then(delay(waitingTime))
.then(function() {
expect(gd._fullData[0].node.groups).toEqual([[4, 3, 2]], 'failed to group #4 + existing group of #2 and #3');
})
.catch(failTest)
.then(done);
});

it('@flaky should not work when dragmode is undefined', function(done) {
var fig = Lib.extendDeep({}, require('@mocks/sankey_circular.json'));
fig.layout.dragmode = undefined;

Plotly.plot(gd, fig)
.then(function() {
// No groups initially
expect(gd._fullData[0].node.groups).toEqual([]);
})
.then(function() {
// Grouping the two nodes on the top right
drag([[640, 130], [400, 450]]);
})
.then(delay(waitingTime))
.then(function() {
expect(gd._fullData[0].node.groups).toEqual([]);
})
.catch(failTest)
.then(done);
});
});
});

describe('Test that selections persist:', function() {
Expand Down