Merge pull request #52 from chanzuckerberg/bkmartinjr-cleanup

small cleanup requests from recent PR

src/actions/index.js

@@ -90,19 +90,17 @@ const initialize = () => {
 function cleanupExpressionResponse(data) {
   const s = store.getState();
   const metadata = s.controls.currentCellSelectionMap;
-  let errorOccured = false;
+  let errorFound = false;
-  const newcells = _.filter(data.data.cells, cell => {
+  data.data.cells = _.filter(data.data.cells, cell => {
-    const found = metadata[cell.cellname];
+    if (!errorFound && !metadata[cell.cellname]) {
-    errorOccured = errorOccured || !found;
+      errorFound = true;
-    return found;
+      console.error(
+        "Warning: /expression REST API returned unexpected cell names -- discarding surprises."
+      );
+    }
+    return metadata[cell.cellname];
   });
 
-  if (errorOccured) {
-    console.error(
-      "Warning: /expression REST API returned unexpected cell names -- discarding surprises."
-    );
-    data.data.cells = newcells;
-  }
   return data;
 }
 

src/util/scaleLinear.js

@@ -9,10 +9,10 @@
 // this is is equivalent to d3.scaleLinear().domain([0,1]).range([-1,1])
 
 export const scaleLinear = (domain, range) => {
-  const offsetD = domain[0];
+  const domainStart = domain[0];
   const scale = (range[1] - range[0]) / (domain[1] - domain[0]);
-  const offsetR = range[0];
+  const rangeStart = range[0];
-  return function(v) {
+  return function(value) {
-    return (v - offsetD) * scale + offsetR;
+    return (value - domainStart) * scale + rangeStart;
   };
 };