y axis tick label fixes + distribution labels

Author: mathematicalmichael

src/infertypes.ts

@@ -387,7 +387,7 @@ export function infertypes(
       }
     }
     if (info.type == ParamType.NUMERICLOG) {
-      info.ticks_format = d3.format(".1e");
+      info.ticks_format = ".1e";
     }
     return info;
   }

src/plotxy.tsx

@@ -229,21 +229,37 @@ export class PlotXY extends React.Component<PlotXYProps, PlotXYState> {
       if (
         !paramDef ||
         paramDef.type !== ParamType.NUMERICLOG ||
-        typeof scale.ticks !== "function"
+        typeof scale.domain !== "function"
       ) {
         return null;
       }
-      const ticks = scale.ticks(approxTickCount);
-      if (!Array.isArray(ticks)) {
+      const domain = scale.domain();
+      if (!Array.isArray(domain) || domain.length < 2) {
+        return null;
+      }
+      const dMin = Number(domain[0]);
+      const dMax = Number(domain[domain.length - 1]);
+      if (!Number.isFinite(dMin) || !Number.isFinite(dMax) || dMin <= 0 || dMax <= 0) {
         return null;
       }
-      const majorTicks = ticks.filter((tick: number) => {
-        if (!Number.isFinite(tick) || tick <= 0) {
-          return false;
+      // Generate powers of 10 spanning the domain, including boundary powers
+      // so that e.g. domain [1.05, 99.3] produces ticks [1, 10, 100]
+      const minExp = Math.floor(Math.log10(dMin));
+      const maxExp = Math.ceil(Math.log10(dMax));
+      const majorTicks: number[] = [];
+      for (let exp = minExp; exp <= maxExp; exp++) {
+        majorTicks.push(Math.pow(10, exp));
+      }
+      // Thin out if there are too many powers of 10 for the available space
+      if (majorTicks.length > approxTickCount && majorTicks.length > 2) {
+        const step = Math.ceil(majorTicks.length / approxTickCount);
+        const thinned: number[] = [majorTicks[0]];
+        for (let i = step; i < majorTicks.length - 1; i += step) {
+          thinned.push(majorTicks[i]);
         }
-        const lg = Math.log10(tick);
-        return Math.abs(lg - Math.round(lg)) < 1e-10;
-      });
+        thinned.push(majorTicks[majorTicks.length - 1]);
+        return thinned;
+      }
       return majorTicks.length > 1 ? majorTicks : null;
     }
     function redraw_axis() {