a few small tweaks

Author: etpinard

src/components/calendars/index.js

@@ -240,8 +240,7 @@ module.exports = {
                 zaxis: {calendar: axisAttrs}
             },
             polar: {
-                radialaxis: {calendar: axisAttrs},
-                angularaxis: {calendar: axisAttrs}
+                radialaxis: {calendar: axisAttrs}
             }
         },
         transforms: {

src/lib/coerce.js

@@ -178,8 +178,6 @@ exports.valObjectMeta = {
         description: [
             'A number (in degree) between -180 and 180.'
         ].join(' '),
-        // TODO maybe add option for 0 to 360??
-        // e.g. for polar.sector
         requiredOpts: [],
         otherOpts: ['dflt'],
         coerceFunction: function(v, propOut, dflt) {

src/plots/cartesian/axes.js

@@ -821,7 +821,6 @@ axes.calcTicks = function calcTicks(ax) {
     }
 
     // If same angle over a full circle, the last tick vals is a duplicate.
-    //
     // TODO must do something similar for angular date axes.
     if(ax._id === 'angular' && Math.abs(rng[1] - rng[0]) === 360) {
         vals.pop();
@@ -895,9 +894,7 @@ var roundBase10 = [2, 5, 10],
     // these don't have to be exact, just close enough to round to the right value
     roundLog1 = [-0.046, 0, 0.301, 0.477, 0.602, 0.699, 0.778, 0.845, 0.903, 0.954, 1],
     roundLog2 = [-0.301, 0, 0.301, 0.699, 1],
-    // TODO
-    // maybe [1, 2, 5, 10, 15, 30, 45, 90, 180] would give better results
-    // on thin polar sectors?
+    // N.B. `thetaunit; 'radians' angular axes must be converted to degrees
     roundAngles = [15, 30, 45, 90, 180];
 
 function roundDTick(roughDTick, base, roundingSet) {
@@ -995,7 +992,7 @@ axes.autoTicks = function(ax, roughDTick) {
     }
     else if(ax._id === 'angular') {
         ax.tick0 = 0;
-        base = getBase(1);
+        base = 1;
         ax.dtick = roundDTick(roughDTick, base, roundAngles);
     }
     else {

src/plots/polar/polar.js

@@ -74,11 +74,9 @@ proto.plot = function(polarCalcData, fullLayout) {
     var _this = this;
     var polarLayout = fullLayout[_this.id];
 
-    // TODO maybe move to generalUpdatePerTraceModule ?
     _this._hasClipOnAxisFalse = false;
     for(var i = 0; i < polarCalcData.length; i++) {
         var trace = polarCalcData[i][0].trace;
-
         if(trace.cliponaxis === false) {
             _this._hasClipOnAxisFalse = true;
             break;

src/traces/scatterpolar/attributes.js

@@ -77,7 +77,7 @@ module.exports = {
     }),
     fillcolor: scatterAttrs.fillcolor,
 
-    // TODO (probably not for first push)
+    // TODO error bars
     // https://stackoverflow.com/a/26597487/4068492
     // error_x (error_r, error_theta)
     // error_y