From debb209da28dfbd46c2808840666130c9385265f Mon Sep 17 00:00:00 2001 From: Andrew Duberstein Date: Tue, 17 Sep 2019 14:54:12 -0700 Subject: [PATCH] Fix bug that makes undefined key appear in aggregate layer pickingInfo --- modules/aggregation-layers/src/utils/cpu-aggregator.js | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/modules/aggregation-layers/src/utils/cpu-aggregator.js b/modules/aggregation-layers/src/utils/cpu-aggregator.js index 1d7a89cbd8e..def298ddf9c 100644 --- a/modules/aggregation-layers/src/utils/cpu-aggregator.js +++ b/modules/aggregation-layers/src/utils/cpu-aggregator.js @@ -133,7 +133,6 @@ export default class CPUAggregator { this._getCellSize = opts.getCellSize || defaultGetCellSize; this._getAggregator = opts.getAggregator; - this._addDimension(opts.dimensions || defaultDimensions); } @@ -246,10 +245,11 @@ export default class CPUAggregator { }); } - getDimensionUpdaters({key, accessor, getBins, getDomain, getScaleFunc, nullValue}) { + getDimensionUpdaters({key, accessor, pickingInfo, getBins, getDomain, getScaleFunc, nullValue}) { return { key, accessor, + pickingInfo, getBins: Object.assign({updater: this.getDimensionSortedBins}, getBins), getDomain: Object.assign({updater: this.getDimensionValueDomain}, getDomain), getScaleFunc: Object.assign({updater: this.getDimensionScale}, getScaleFunc),