review comments - 🔪 commented-out code, comment on cleanData

Author: alexcjohnson

src/plot_api/helpers.js

@@ -196,8 +196,14 @@ function cleanAxRef(container, attr) {
     }
 }
 
-// Make a few changes to the data right away
-// before it gets used for anything
+/*
+ * cleanData: Make a few changes to the data right away
+ * before it gets used for anything
+ * Mostly for backward compatibility, modifies the data traces users provide.
+ *
+ * Important: if you're going to add something here that modifies a data array,
+ * update it in place so the new array === the old one.
+ */
 exports.cleanData = function(data, existingData) {
     // Enforce unique IDs
     var suids = [], // seen uids --- so we can weed out incoming repeats

src/plots/gl3d/layout/tick_marks.js

@@ -78,7 +78,6 @@ function computeTickMarks(scene) {
 
 
             axes.tickmode = tickModeCached;
-            // axes.range = rangeCached;
         }
     }