Skip to content
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

Lasso & rectangular selections #154

Merged
merged 35 commits into from
Jan 6, 2016
Merged
Show file tree
Hide file tree
Changes from 31 commits
Commits
Show all changes
35 commits
Select commit Hold shift + click to select a range
d6f983e
agignore dist
alexcjohnson Dec 27, 2015
e536a91
point in polygon routine, with tests
alexcjohnson Dec 27, 2015
5cad4ff
lasso and selectbox skeleton
alexcjohnson Dec 28, 2015
ca60e63
propagate dragbox cursor to coverSlip while dragging
alexcjohnson Dec 29, 2015
76926d7
fix modebar logic and tests for select dragmodes
alexcjohnson Dec 29, 2015
78d2867
change polygon and its tests to multi-exports form
alexcjohnson Dec 29, 2015
f0e6cfb
polygon filtering algorithm
alexcjohnson Dec 29, 2015
c0a94f7
polygon filtering test
alexcjohnson Dec 29, 2015
12b43c6
special case of polygon for rectangles
alexcjohnson Dec 30, 2015
07e5cef
selection on scatter points
alexcjohnson Dec 31, 2015
b1a1c24
clear hover when drag starts
alexcjohnson Dec 31, 2015
1958475
didn't end up using a separate lasso handler
alexcjohnson Dec 31, 2015
3c40c41
crosshair it is
alexcjohnson Jan 4, 2016
188a0db
inline rectFirstEdgeTest
alexcjohnson Jan 4, 2016
8d0afb5
Merge branch 'master' into lasso
alexcjohnson Jan 4, 2016
1c5f325
lasso like it's 2016 baby!
alexcjohnson Jan 4, 2016
d0203c8
scatter.selectPoints uses scatter.hasMarkers
alexcjohnson Jan 4, 2016
5be72de
:cow2:
alexcjohnson Jan 4, 2016
057e4ac
prep for adding selection bounds to event data
alexcjohnson Jan 5, 2016
556cb83
split out scatter.selectPoints to a new file
alexcjohnson Jan 5, 2016
7b07a25
support selecting scatter text
alexcjohnson Jan 5, 2016
2a970fc
fx constants file
alexcjohnson Jan 5, 2016
d7abd18
more fx constants
alexcjohnson Jan 6, 2016
7addcec
BENDPX into constants
alexcjohnson Jan 6, 2016
d02d612
horizontal and vertical select boxes
alexcjohnson Jan 6, 2016
698376e
off-by-one error in select outline
alexcjohnson Jan 6, 2016
89c1655
clear selection on zoom/pan
alexcjohnson Jan 6, 2016
d8ed7a7
selection range event data
alexcjohnson Jan 6, 2016
7d897a4
show select icons only when they apply
alexcjohnson Jan 6, 2016
96e01b6
update modebar tests for new select icon logic
alexcjohnson Jan 6, 2016
bc04a15
desciption attribute value delimiters
alexcjohnson Jan 6, 2016
710791d
fix nonlinear axes in select
alexcjohnson Jan 6, 2016
342f991
:cow2:
alexcjohnson Jan 6, 2016
b5c090f
MINSELECT bigger than MINDRAG
alexcjohnson Jan 6, 2016
1d7745f
add lasso and selectbox icon
etpinard Jan 6, 2016
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
1 change: 1 addition & 0 deletions .agignore
Original file line number Diff line number Diff line change
@@ -0,0 +1 @@
dist
Copy link
Contributor

Choose a reason for hiding this comment

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

@alexcjohnson your ack days are over? Can I ask why?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

I think ack disappeared from my system in an OS upgrade or something... and I tried ag after remembering I think @theengineear mentioned it a while back. I was never really bothered by ack but ag does seem to live up to its claim of speed. Otherwise it seems pretty similar but I'm pretty basic in how I use these tools.

27 changes: 26 additions & 1 deletion src/components/modebar/buttons.js
Original file line number Diff line number Diff line change
Expand Up @@ -121,6 +121,24 @@ modeBarButtons.pan2d = {
click: handleCartesian
};

modeBarButtons.select2d = {
name: 'select2d',
title: 'Box Select',
attr: 'dragmode',
val: 'select',
icon: Icons.question, // TODO
click: handleCartesian
};

modeBarButtons.lasso2d = {
name: 'lasso2d',
title: 'Lasso Select',
attr: 'dragmode',
val: 'lasso',
icon: Icons.question, // TODO
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

@delekru here (and right above) are where the new icons get referenced, once they're part of the ploticon font.

click: handleCartesian
};

modeBarButtons.zoomIn2d = {
name: 'zoomIn2d',
title: 'Zoom in',
Expand Down Expand Up @@ -179,6 +197,13 @@ modeBarButtons.hoverCompareCartesian = {
click: handleCartesian
};

var DRAGCURSORS = {
pan: 'move',
zoom: 'crosshair',
select: 'crosshair',
lasso: 'crosshair'
};

function handleCartesian(gd, ev) {
var button = ev.currentTarget,
astr = button.getAttribute('data-attr'),
Expand Down Expand Up @@ -230,7 +255,7 @@ function handleCartesian(gd, ev) {
if(fullLayout._hasCartesian) {
Plotly.Fx.setCursor(
fullLayout._paper.select('.nsewdrag'),
{pan:'move', zoom:'crosshair'}[val]
DRAGCURSORS[val]
);
}
Plotly.Fx.supplyLayoutDefaults(gd.layout, fullLayout, gd._fullData);
Expand Down
47 changes: 41 additions & 6 deletions src/components/modebar/manage.js
Original file line number Diff line number Diff line change
Expand Up @@ -10,6 +10,7 @@
'use strict';

var Plotly = require('../../plotly');
var scatterSubTypes = require('../../traces/scatter/subtypes');

var createModeBar = require('./');
var modeBarButtons = require('./buttons');
Expand Down Expand Up @@ -57,7 +58,7 @@ module.exports = function manageModeBar(gd) {
}
else {
buttonGroups = getButtonGroups(
fullLayout,
gd,
context.modeBarButtonsToRemove,
context.modeBarButtonsToAdd
);
Expand All @@ -68,8 +69,12 @@ module.exports = function manageModeBar(gd) {
};

// logic behind which buttons are displayed by default
function getButtonGroups(fullLayout, buttonsToRemove, buttonsToAdd) {
var groups = [];
function getButtonGroups(gd, buttonsToRemove, buttonsToAdd) {
var fullLayout = gd._fullLayout,
fullData = gd._fullData,
groups = [],
i,
trace;

function addGroup(newGroup) {
var out = [];
Expand Down Expand Up @@ -99,10 +104,40 @@ function getButtonGroups(fullLayout, buttonsToRemove, buttonsToAdd) {

var hasCartesian = fullLayout._hasCartesian,
hasGL2D = fullLayout._hasGL2D,
allAxesFixed = areAllAxesFixed(fullLayout);
allAxesFixed = areAllAxesFixed(fullLayout),
dragModeGroup = [];

if((hasCartesian || hasGL2D) && !allAxesFixed) {
dragModeGroup = ['zoom2d', 'pan2d'];
}
if(hasCartesian) {
// look for traces that support selection
// to be updated as we add more selectPoints handlers
var selectable = false;
for(i = 0; i < fullData.length; i++) {
if(selectable) break;
trace = fullData[i];
if(!trace._module || !trace._module.selectPoints) continue;

if(trace.type === 'scatter') {
if(scatterSubTypes.hasMarkers(trace) || scatterSubTypes.hasText(trace)) {
selectable = true;
}
}
// assume that in general if the trace module has selectPoints,
// then it's selectable. Scatter is an exception to this because it must
// have markers or text, not just be a scatter type.
else selectable = true;
}

if(selectable) {
dragModeGroup.push('select2d');
dragModeGroup.push('lasso2d');
}
}
if(dragModeGroup.length) addGroup(dragModeGroup);

if((hasCartesian || hasGL2D) && !allAxesFixed) {
addGroup(['zoom2d', 'pan2d']);
addGroup(['zoomIn2d', 'zoomOut2d', 'autoScale2d', 'resetScale2d']);
}

Choose a reason for hiding this comment

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

this if statement could be moved up here, right?

Copy link
Contributor

Choose a reason for hiding this comment

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

@cpsievert that would change the order which the buttons are displayed

Expand All @@ -119,7 +154,7 @@ function getButtonGroups(fullLayout, buttonsToRemove, buttonsToAdd) {
// append buttonsToAdd to the groups
if(buttonsToAdd.length) {
if(Array.isArray(buttonsToAdd[0])) {
for(var i = 0; i < buttonsToAdd.length; i++) {
for(i = 0; i < buttonsToAdd.length; i++) {
groups.push(buttonsToAdd[i]);
}
}
Expand Down
12 changes: 12 additions & 0 deletions src/css/_drag.scss
Original file line number Diff line number Diff line change
@@ -0,0 +1,12 @@
.select-outline {
fill: none;
stroke-width: 1;
shape-rendering: crispEdges;
}
.select-outline-1 {
stroke: white;
}
.select-outline-2 {
stroke: black;
stroke-dasharray: 2px 2px;
}
1 change: 1 addition & 0 deletions src/css/style.scss
Original file line number Diff line number Diff line change
Expand Up @@ -6,5 +6,6 @@
@import "cursor.scss";
@import "modebar.scss";
@import "tooltip.scss";
@import "drag.scss";
}
@import "notifier.scss";
238 changes: 238 additions & 0 deletions src/lib/polygon.js
Original file line number Diff line number Diff line change
@@ -0,0 +1,238 @@
/**
* Copyright 2012-2016, 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';

var dot = require('./matrix').dot;

var polygon = module.exports = {};

/**
* Turn an array of [x, y] pairs into a polygon object
* that can test if points are inside it
*
* @param ptsIn Array of [x, y] pairs
*
* @returns polygon Object {xmin, xmax, ymin, ymax, pts, contains}
* (x|y)(min|max) are the bounding rect of the polygon
* pts is the original array, with the first pair repeated at the end
* contains is a function: (pt, omitFirstEdge)
* pt is the [x, y] pair to test
* omitFirstEdge truthy means points exactly on the first edge don't
* count. This is for use adding one polygon to another so we
* don't double-count the edge where they meet.
* returns boolean: is pt inside the polygon (including on its edges)
*/
Copy link
Contributor

Choose a reason for hiding this comment

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

🐄 put the jsDoc block above

polygon.tester = function tester(ptsIn) {

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

oh right... that's where it started, when this was the only export 🙈

polygon.tester = function tester(ptsIn) {
var pts = ptsIn.slice(),
xmin = pts[0][0],
xmax = xmin,
ymin = pts[0][1],
ymax = ymin;

pts.push(pts[0]);
for(var i = 1; i < pts.length; i++) {
xmin = Math.min(xmin, pts[i][0]);
xmax = Math.max(xmax, pts[i][0]);
ymin = Math.min(ymin, pts[i][1]);
ymax = Math.max(ymax, pts[i][1]);
}

// do we have a rectangle? Handle this here, so we can use the same
// tester for the rectangular case without sacrificing speed
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Initially I thought I'd make separate testers for lasso and rectangle, but it seemed easier to just branch in here and attach a simplified contains function. There still may be cases in other trace types where you could improve rectangle performance by handling it explicitly - like for heatmap/contour - in that case the selectPoints routine can look at poly.isRect and poly.xmin etc.

Copy link
Contributor

Choose a reason for hiding this comment

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

👍 looks clean


var isRect = false,
rectFirstEdgeTest;

if(pts.length === 5) {
if(pts[0][0] === pts[1][0]) { // vert, horz, vert, horz
if(pts[2][0] === pts[3][0] &&
pts[0][1] === pts[3][1] &&
pts[1][1] === pts[2][1]) {
isRect = true;
rectFirstEdgeTest = function(pt) { return pt[0] === pts[0][0]; };
}
}
else if(pts[0][1] === pts[1][1]) { // horz, vert, horz, vert
if(pts[2][1] === pts[3][1] &&
pts[0][0] === pts[3][0] &&
pts[1][0] === pts[2][0]) {
isRect = true;
rectFirstEdgeTest = function(pt) { return pt[1] === pts[0][1]; };
}
}
}

function rectContains(pt, omitFirstEdge) {
var x = pt[0],
y = pt[1];

if(x < xmin || x > xmax || y < ymin || y > ymax) {
// pt is outside the bounding box of polygon
return false;
}
if(omitFirstEdge && rectFirstEdgeTest(pt)) return false;

return true;
}

function contains(pt, omitFirstEdge) {
var x = pt[0],
y = pt[1];

if(x < xmin || x > xmax || y < ymin || y > ymax) {
// pt is outside the bounding box of polygon
return false;
}

var imax = pts.length,
x1 = pts[0][0],
y1 = pts[0][1],
crossings = 0,
i,
x0,
y0,
xmini,
ycross;

for(i = 1; i < imax; i++) {
// find all crossings of a vertical line upward from pt with
// polygon segments
// crossings exactly at xmax don't count, unless the point is
// exactly on the segment, then it counts as inside.
x0 = x1;
y0 = y1;
x1 = pts[i][0];
y1 = pts[i][1];
xmini = Math.min(x0, x1);

// outside the bounding box of this segment, it's only a crossing
// if it's below the box.
if(x < xmini || x > Math.max(x0, x1) || y > Math.max(y0, y1)) {
continue;
}
else if(y < Math.min(y0, y1)) {
// don't count the left-most point of the segment as a crossing
// because we don't want to double-count adjacent crossings
// UNLESS the polygon turns past vertical at exactly this x
// Note that this is repeated below, but we can't factor it out
// because
if(x !== xmini) crossings++;
}
// inside the bounding box, check the actual line intercept
else {
// vertical segment - we know already that the point is exactly
// on the segment, so mark the crossing as exactly at the point.
if(x1 === x0) ycross = y;
// any other angle
else ycross = y0 + (x - x0) * (y1 - y0) / (x1 - x0);

// exactly on the edge: counts as inside the polygon, unless it's the
// first edge and we're omitting it.
if(y === ycross) {
if(i === 1 && omitFirstEdge) return false;
return true;
}

if(y <= ycross && x !== xmini) crossings++;
}
}

// if we've gotten this far, odd crossings means inside, even is outside
return crossings % 2 === 1;
}

return {
xmin: xmin,
xmax: xmax,
ymin: ymin,
ymax: ymax,
pts: pts,
contains: isRect ? rectContains : contains,
isRect: isRect
};
};

/**
* Test if a segment of a points array is bent or straight
*
* @param pts Array of [x, y] pairs
* @param start the index of the proposed start of the straight section
* @param end the index of the proposed end point
* @param tolerance the max distance off the line connecting start and end
* before the line counts as bent
* @returns boolean: true means this segment is bent, false means straight
*/
var isBent = polygon.isSegmentBent = function isBent(pts, start, end, tolerance) {
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Used to simplify the lasso polygon. This is pretty similar to the scatter line decimation algorithm, so not super 🌴 but line decimation has a few other quirks that made it seem annoying to combine.

Copy link
Contributor

Choose a reason for hiding this comment

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

polygon.js is ~150 lines of code. Nothing to worry here.

var startPt = pts[start],
segment = [pts[end][0] - startPt[0], pts[end][1] - startPt[1]],
segmentSquared = dot(segment, segment),
segmentLen = Math.sqrt(segmentSquared),
unitPerp = [-segment[1] / segmentLen, segment[0] / segmentLen],
i,
part,
partParallel;

for(i = start + 1; i < end; i++) {
part = [pts[i][0] - startPt[0], pts[i][1] - startPt[1]];
partParallel = dot(part, segment);

if(partParallel < 0 || partParallel > segmentSquared ||
Math.abs(dot(part, unitPerp)) > tolerance) return true;
}
return false;
};

/**
* Make a filtering polygon, to minimize the number of segments
*
* @param pts Array of [x, y] pairs (must start with at least 1 pair)
* @param tolerance the maximum deviation from straight allowed for
* removing points to simplify the polygon
*
* @returns Object {addPt, raw, filtered}
* addPt is a function(pt: [x, y] pair) to add a raw point and
* continue filtering
* raw is all the input points
* filtered is the resulting filtered Array of [x, y] pairs
*/
polygon.filter = function filter(pts, tolerance) {
var ptsFiltered = [pts[0]],
doneRawIndex = 0,
doneFilteredIndex = 0;

function addPt(pt) {
pts.push(pt);
var prevFilterLen = ptsFiltered.length,
iLast = doneRawIndex;
ptsFiltered.splice(doneFilteredIndex + 1);

for(var i = iLast + 1; i < pts.length; i++) {
if(i === pts.length - 1 || isBent(pts, iLast, i + 1, tolerance)) {
ptsFiltered.push(pts[i]);
if(ptsFiltered.length < prevFilterLen - 2) {
doneRawIndex = i;
doneFilteredIndex = ptsFiltered.length - 1;
}
iLast = i;
}
}
}

if(pts.length > 1) {
var lastPt = pts.pop();
addPt(lastPt);
}

return {
addPt: addPt,
raw: pts,
filtered: ptsFiltered
};
};
Loading