wums 0.1.7__tar.gz → 0.1.9__tar.gz

{wums-0.1.7 → wums-0.1.9}/PKG-INFO

@@ -1,6 +1,6 @@
-Metadata-Version: 2.2
+Metadata-Version: 2.4
 Name: wums
-Version: 0.1.7
+Version: 0.1.9
 Summary: .
 Author-email: David Walter <david.walter@cern.ch>, Josh Bendavid <josh.bendavid@cern.ch>, Kenneth Long <kenneth.long@cern.ch>, Jan Eysermans <jan.eysermans@cern.ch>
 License: MIT

{wums-0.1.7 → wums-0.1.9}/pyproject.toml

@@ -4,7 +4,7 @@ build-backend = "setuptools.build_meta"
 
 [project]
 name = "wums"
-version = "0.1.7"
+version = "0.1.9"
 description = "."
 readme = { file = "README.md", content-type = "text/markdown" }
 license = { text = "MIT" }
@@ -38,4 +38,4 @@ all = ["plotting", "fitting", "pickling"]
 where = ["."]
 
 [tool.setuptools.package-data]
-"wums" = ["Templates/index.php"]
+"wums" = ["Templates/index.php"]

{wums-0.1.7 → wums-0.1.9}/wums/boostHistHelpers.py

@@ -4,6 +4,7 @@ from functools import reduce
 
 import hist
 import numpy as np
+from scipy.interpolate import make_smoothing_spline
 
 from wums import logging
 
@@ -60,7 +61,7 @@ def broadcastSystHist(h1, h2, flow=True, by_ax_name=True):
     # move back to original order
     new_vals = np.moveaxis(new_vals, np.arange(len(moves)), list(moves.keys()))
 
-    if new_vals.shape != h2.values(flow=flow).shape:
+    if new_vals.shape != s2:
         raise ValueError(
             f"Broadcast shape {new_vals.shape} (from h1.shape={h1.values(flow=flow).shape}, axes={h1.axes.name}) "
             f"does not match desired shape {h2.view(flow=flow).shape} (axes={h2.axes.name})"
@@ -193,10 +194,10 @@ def multiplyWithVariance(vals1, vals2, vars1=None, vars2=None):
     return outvals, outvars
 
 
-def multiplyHists(h1, h2, allowBroadcast=True, createNew=True, flow=True):
+def multiplyHists(h1, h2, allowBroadcast=True, createNew=True, flow=True, broadcast_by_ax_name=True):
     if allowBroadcast:
-        h1 = broadcastSystHist(h1, h2, flow=flow)
-        h2 = broadcastSystHist(h2, h1, flow=flow)
+        h1 = broadcastSystHist(h1, h2, flow=flow, by_ax_name=broadcast_by_ax_name)
+        h2 = broadcastSystHist(h2, h1, flow=flow, by_ax_name=broadcast_by_ax_name)
 
     if (
         h1.storage_type == hist.storage.Double
@@ -233,6 +234,7 @@ def concatenateHists(h1, h2, allowBroadcast=True, by_ax_name=True, flow=False):
         h2 = broadcastSystHist(h2, h1, flow=flow, by_ax_name=by_ax_name)
 
     axes = []
+
     for ax1, ax2 in zip(h1.axes, h2.axes):
         if ax1 == ax2:
             axes.append(ax1)
@@ -263,7 +265,7 @@ def concatenateHists(h1, h2, allowBroadcast=True, by_ax_name=True, flow=False):
                 )
             else:
                 raise ValueError(
-                    f"Cannot concatenate hists with inconsistent axes: {ax1.name} and {ax2.name}"
+                    f"Cannot concatenate hists with inconsistent axes: {ax1.name}: ({ax1.edges}) and {ax2.name}: ({ax2.edges})"
                 )
 
     newh = hist.Hist(*axes, storage=h1.storage_type())
@@ -407,6 +409,10 @@ def normalize(h, scale=1e6, createNew=True, flow=True):
     return scaleHist(h, scale, createNew, flow)
 
 
+def renameAxis(h, axis_name, new_name):
+    h.axes[axis_name].__dict__['name'] = new_name
+
+
 def makeAbsHist(h, axis_name, rename=True):
     ax = h.axes[axis_name]
     axidx = list(h.axes).index(ax)
@@ -455,8 +461,8 @@ def compatibleBins(edges1, edges2):
 
 def rebinHistMultiAx(h, axes, edges=[], lows=[], highs=[]):
     # edges: lists of new edges or integers to merge bins, in case new edges are given the lows and highs will be ignored
-    # lows: list of new lower boundaries
-    # highs: list of new upper boundaries
+    # lows: list of new lower boundaries or bins
+    # highs: list of new upper boundaries or bins
 
     sel = {}
     for ax, low, high, rebin in itertools.zip_longest(axes, lows, highs, edges):
@@ -467,14 +473,21 @@ def rebinHistMultiAx(h, axes, edges=[], lows=[], highs=[]):
             h = rebinHist(h, ax, rebin)
         elif low is not None and high is not None:
             # in case high edge is upper edge of last bin we need to manually set the upper limit
-            upper = hist.overflow if high == h.axes[ax].edges[-1] else complex(0, high)
-            logger.info(f"Restricting the axis '{ax}' to range [{low}, {high}]")
+            # distinguish case with pure real or imaginary number (to select index or value, respectively)
+            # in the former case, force casting into integer
+            if isinstance(high, int):
+                upper = hist.overflow if high == h.axes[ax].size else high
+            elif isinstance(high, complex):
+                high_imag = high.imag
+                upper = hist.overflow if high_imag == h.axes[ax].edges[-1] else high
+            logger.info(f"Slicing the axis '{ax}' to [{low}, {upper}]")
             sel[ax] = slice(
-                complex(0, low), upper, hist.rebin(rebin) if rebin else None
+                low, upper, hist.rebin(rebin) if rebin else None
             )
         elif type(rebin) == int and rebin > 1:
             logger.info(f"Rebinning the axis '{ax}' by [{rebin}]")
             sel[ax] = slice(None, None, hist.rebin(rebin))
+
     return h[sel] if len(sel) > 0 else h
 
 
@@ -499,9 +512,9 @@ def mirrorAxes(h, axes, flow=True):
     return h
 
 
-def disableAxisFlow(ax):
+def disableAxisFlow(ax, under=False, over=False):
     if isinstance(ax, hist.axis.Integer):
-        args = [ax.edges[0], ax.edges[-1]]
+        args = [int(ax.edges[0]), int(ax.edges[-1])]
     elif isinstance(ax, hist.axis.Regular):
         args = [ax.size, ax.edges[0], ax.edges[-1]]
     else:
@@ -510,17 +523,24 @@ def disableAxisFlow(ax):
     return type(ax)(
         *args,
         name=ax.name,
-        overflow=False,
-        underflow=False,
+        overflow=over,
+        underflow=under,
         circular=ax.traits.circular,
     )
 
 
-def disableFlow(h, axis_name):
-    # disable the overflow and underflow bins of a single axes, while keeping the flow bins of other axes
+def disableFlow(h, axis_name, under=False, over=False):
+    # axes_name can be either string or a list of strings with the axis name(s) to disable the flow
+    if not isinstance(axis_name, str):
+        for var in axis_name:
+            if var in h.axes.name:
+                h = disableFlow(h, var)
+        return h
+
+    # disable the overflow and underflow bins of a single axis, while keeping the flow bins of other axes
     ax = h.axes[axis_name]
     ax_idx = [a.name for a in h.axes].index(axis_name)
-    new_ax = disableAxisFlow(ax)
+    new_ax = disableAxisFlow(ax, under=under, over=over)
     axes = list(h.axes)
     axes[ax_idx] = new_ax
     hnew = hist.Hist(*axes, name=h.name, storage=h.storage_type())
@@ -528,7 +548,7 @@ def disableFlow(h, axis_name):
         (
             slice(None)
             if i != ax_idx
-            else slice(ax.traits.underflow, new_ax.size + ax.traits.underflow)
+            else slice(ax.traits.underflow * (not under), ax.size + ax.traits.underflow + ax.traits.overflow * over)
         )
         for i in range(len(axes))
     ]
@@ -773,9 +793,9 @@ def unrolledHist(h, obs=None, binwnorm=None, add_flow_bins=False):
 
     if binwnorm:
         edges = (
-            plot_tools.extendEdgesByFlow(hproj) if add_flow_bins else hproj.axes.edges
+            extendEdgesByFlow(hproj) if add_flow_bins else hproj.axes.edges
         )
-        binwidths = np.outer(*[np.diff(e.squeeze()) for e in edges]).flatten()
+        binwidths = np.array(list(itertools.product(*[np.diff(e.squeeze()) for e in edges]))).prod(axis=1)
         scale = binwnorm / binwidths
     else:
         scale = 1
@@ -1133,3 +1153,42 @@ def rssHistsMid(h, syst_axis, scale=1.0):
     hDown = addHists(hnom, hrss[{"downUpVar": -1j}], scale2=-1.0)
 
     return hUp, hDown
+
+def smooth_hist(h, smooth_ax_name, exclude_axes=[], start_bin=0, end_bin=None):
+
+    hnew = h.copy()
+
+    smooth_ax = h.axes[smooth_ax_name]
+    hproj = h.project(smooth_ax_name, *[ax for ax in h.axes.name if ax not in [smooth_ax_name, *exclude_axes]])
+    smoothh = hproj.copy()
+
+    # Reshape before looping over all other bins and smoothing along the relevant axis
+    vals = smoothh.values().reshape(smooth_ax.size, -1)
+    if not end_bin:
+        end_bin = vals.shape[0]
+
+    # Correct for bin width
+    binw = np.diff(smoothh.axes[smooth_ax_name].edges)
+    vals = (vals.T/binw).T
+
+    for b in range(vals.shape[-1]):
+        spl = make_smoothing_spline(smooth_ax.centers[start_bin:end_bin], vals[start_bin:end_bin,b])
+        vals[start_bin:end_bin,b] = spl(smooth_ax.centers[start_bin:end_bin])
+
+    #Recorrect for bin width
+    vals = (vals.T*binw).T
+
+    smoothh.values()[...] = vals.reshape(smoothh.shape)
+
+    if not exclude_axes:
+        return smoothh.project(*hnew.axes.name)
+
+    # If some axis has been excluded, broadcast it back 
+    smoothfac = divideHists(smoothh, hproj).project(*[ax for ax in h.axes.name if ax not in exclude_axes])
+
+    # Broadcast over excluded axes
+    indices = tuple(
+        None if ax in exclude_axes else slice(None) for ax in h.axes.name
+    )
+    hnew.values()[...] = h.values() * smoothfac.values()[indices]
+    return hnew

{wums-0.1.7 → wums-0.1.9}/wums/ioutils.py

@@ -325,8 +325,12 @@ class H5Unpickler(pickle.Unpickler):
             raise pickle.UnpicklingError("unsupported persistent object")
 
 
-def pickle_dump_h5py(name, obj, h5out):
+def pickle_dump_h5py(name, obj, h5out, override=False):
     """Write an object to a new h5py group which will be created within the provided group."""
+    if name in h5out and override:
+        # If the group already exists, delete it first
+        del h5out[name]
+
     obj_group = h5out.create_group(name)
     try:
         obj_group.attrs["narf_h5py_pickle_protocol_version"] = CURRENT_PROTOCOL_VERSION

{wums-0.1.7 → wums-0.1.9}/wums/output_tools.py

@@ -165,6 +165,12 @@ def make_meta_info_dict(
     return meta_data
 
 
+def encode_complex(obj):
+    if isinstance(obj, complex):
+        return {"real": obj.real, "imag": obj.imag}
+    raise TypeError(f"Object of type {obj.__class__.__name__} is not JSON serializable")
+
+
 def write_logfile(
     outpath,
     logname,
@@ -181,7 +187,8 @@ def write_logfile(
             logf.write("\n" + "-" * 80 + "\n")
             if isinstance(v, dict):
                 logf.write(k)
-                logf.write(json.dumps(v, indent=5).replace("\\n", "\n"))
+                # String conversion needed for non-primitive types (e.g., complex number)
+                logf.write(json.dumps(v, default=encode_complex, indent=5).replace("\\n", "\n"))
             else:
                 logf.write(f"{k}: {v}\n")
 
@@ -202,3 +209,19 @@ def write_index_and_log(
 ):
     write_indexfile(outpath, template_dir)
     write_logfile(outpath, logname, args, analysis_meta_info)
+
+def write_lz4_pkl_output(
+    outfile, outfolder, output_dict, basedir, args=None, file_meta_data=None
+):
+    if not outfile.endswith(".pkl.lz4"):
+        outfile += ".pkl.lz4"
+
+    logger.info(f"Write file {outfile}")
+    result_dict = {
+        outfolder : output_dict,
+        "meta_data": make_meta_info_dict(args, wd=basedir),
+    }
+    if file_meta_data is not None:
+        result_dict["file_meta_data"] = file_meta_data
+    with lz4.frame.open(outfile, "wb") as f:
+        pickle.dump(result_dict, f, protocol=pickle.HIGHEST_PROTOCOL)

{wums-0.1.7 → wums-0.1.9}/wums/plot_tools.py

@@ -1,12 +1,7 @@
-import datetime
-import json
+import importlib
+import inspect
 import math
-import pathlib
-import shutil
-import socket
-import sys
 import textwrap
-import importlib
 
 import hist
 import matplotlib as mpl
@@ -22,7 +17,7 @@ from matplotlib.patches import Polygon
 from matplotlib.ticker import StrMethodFormatter
 
 from wums import boostHistHelpers as hh
-from wums import ioutils, logging
+from wums import logging
 
 hep.style.use(hep.style.ROOT)
 
@@ -466,6 +461,7 @@ def addLegend(
     reverse=True,
     labelcolor=None,
     padding_loc="auto",
+    title=None,
 ):
     handles, labels = ax.get_legend_handles_labels()
     if extra_entries_first:
@@ -496,6 +492,7 @@ def addLegend(
     text_size = get_textsize(ax, text_size)
     handler_map = get_custom_handler_map(custom_handlers)
     leg = ax.legend(
+        title=title,
         handles=handles,
         labels=labels,
         prop={"size": text_size},
@@ -507,6 +504,8 @@ def addLegend(
         markerfirst=markerfirst,
         labelcolor=labelcolor,
     )
+    if title is not None:
+        leg.set_title(title, prop={"size": text_size})
 
     if extra_text is not None:
         if extra_text_loc is None:
@@ -609,18 +608,18 @@ def add_decor(
     ax, title, label=None, lumi=None, loc=2, data=True, text_size=None, no_energy=False
 ):
     text_size = get_textsize(ax, text_size)
-
     if title in ["CMS", "ATLAS", "LHCb", "ALICE"]:
         module = getattr(hep, title.lower())
         make_text = module.text
         make_label = module.label
     else:
+
         def make_text(text=None, **kwargs):
             for key, value in dict(hep.rcParams.text._get_kwargs()).items():
                 if (
                     value is not None
                     and key not in kwargs
-                    and key in inspect.getfullargspec(label_base.exp_text).kwonlyargs
+                    and key in inspect.getfullargspec(hep.label.exp_text).kwonlyargs
                 ):
                     kwargs.setdefault(key, value)
             kwargs.setdefault("italic", (False, True, False))
@@ -632,7 +631,7 @@ def add_decor(
                 if (
                     value is not None
                     and key not in kwargs
-                    and key in inspect.getfullargspec(label_base.exp_text).kwonlyargs
+                    and key in inspect.getfullargspec(hep.label.exp_text).kwonlyargs
                 ):
                     kwargs.setdefault(key, value)
             kwargs.setdefault("italic", (False, True, False))
@@ -651,7 +650,7 @@ def add_decor(
             data=data,
             loc=loc,
         )
-    
+
     # else:
     #     if loc==0:
     #         # above frame
@@ -666,7 +665,7 @@ def add_decor(
     #         x = 0.05
     #         y = 0.88
     #     elif loc==2:
-    #         #     
+    #         #
     #     ax.text(
     #         x,
     #         y,
@@ -678,13 +677,12 @@ def add_decor(
     #     if label is not None:
     #         ax.text(0.05, 0.80, label, transform=ax.transAxes, fontstyle="italic")
 
+
 def makeStackPlotWithRatio(
     histInfo,
     stackedProcs,
     histName="nominal",
     unstacked=None,
-    fitresult=None,
-    prefit=False,
     xlabel="",
     ylabel=None,
     rlabel="Data/Pred.",
@@ -727,12 +725,14 @@ def makeStackPlotWithRatio(
     alpha=0.7,
     legPos="upper right",
     leg_padding="auto",
+    lowerLeg=True,
     lowerLegCols=2,
     lowerLegPos="upper right",
     lower_panel_variations=0,
     lower_leg_padding="auto",
     scaleRatioUnstacked=[],
     subplotsizes=[4, 2],
+    x_vertLines_edges=[],
 ):
     add_ratio = not (no_stack or no_ratio)
     if ylabel is None:
@@ -757,11 +757,6 @@ def makeStackPlotWithRatio(
         if xlim:
             h = h[complex(0, xlim[0]) : complex(0, xlim[1])]
 
-        # If plotting from combine, apply the action to the underlying hist.
-        # Don't do this for the generic case, as it screws up the ability to make multiple plots
-        if fitresult:
-            histInfo[k].hists[histName] = h
-
         if k != "Data":
             stack.append(h)
         else:
@@ -803,72 +798,10 @@ def makeStackPlotWithRatio(
         ratio_axes = None
         ax2 = None
 
-    if fitresult:
-        import uproot
-
-        combine_result = uproot.open(fitresult)
-
-        fittype = "prefit" if prefit else "postfit"
-
-        # set histograms to prefit/postfit values
-        for p in to_read:
-
-            hname = f"expproc_{p}_{fittype}" if p != "Data" else "obs"
-            vals = combine_result[hname].to_hist().values()
-            if len(histInfo[p].hists[histName].values()) != len(vals):
-                raise ValueError(
-                    f"The size of the combine histogram ({(vals.shape)}) is not consistent with the xlim or input hist ({histInfo[p].hists[histName].shape})"
-                )
-
-            histInfo[p].hists[histName].values()[...] = vals
-            if p == "Data":
-                histInfo[p].hists[histName].variances()[...] = vals
-
-        # for postfit uncertaity bands
-        axis = histInfo[to_read[0]].hists[histName].axes[0].edges
-
-        # need to divide by bin width
-        binwidth = axis[1:] - axis[:-1]
-        hexp = combine_result[f"expfull_{fittype}"].to_hist()
-        if hexp.storage_type != hist.storage.Weight:
-            raise ValueError(
-                f"Did not find uncertainties in {fittype} hist. Make sure you run combinetf with --computeHistErrors!"
-            )
-        nom = hexp.values() / binwidth
-        std = np.sqrt(hexp.variances()) / binwidth
-
-        hatchstyle = "///"
-        ax1.fill_between(
-            axis,
-            np.append(nom + std, (nom + std)[-1]),
-            np.append(nom - std, (nom - std)[-1]),
-            step="post",
-            facecolor="none",
-            zorder=2,
-            hatch=hatchstyle,
-            edgecolor="k",
-            linewidth=0.0,
-            label="Uncertainty",
-        )
-
-        if add_ratio:
-            ax2.fill_between(
-                axis,
-                np.append((nom + std) / nom, ((nom + std) / nom)[-1]),
-                np.append((nom - std) / nom, ((nom - std) / nom)[-1]),
-                step="post",
-                facecolor="none",
-                zorder=2,
-                hatch=hatchstyle,
-                edgecolor="k",
-                linewidth=0.0,
-            )
-
     opts = dict(stack=not no_stack, flow=flow)
     optsr = opts.copy()  # no binwnorm for ratio axis
-    optsr["density"] = density
     if density:
-        opts["density"] = True
+        opts["density"] = True            
     else:
         opts["binwnorm"] = binwnorm
 
@@ -889,13 +822,14 @@ def makeStackPlotWithRatio(
             for x in (data_hist.sum(), hh.sumHists(stack).sum())
         ]
         scale = vals[0] / vals[1]
-        unc = scale * (varis[0] / vals[0] ** 2 + varis[1] / vals[1] ** 2)**0.5
+        unc = scale * (varis[0] / vals[0] ** 2 + varis[1] / vals[1] ** 2) ** 0.5
         ndigits = -math.floor(math.log10(abs(unc))) + 1
         logger.info(
             f"Rescaling all processes by {round(scale,ndigits)} +/- {round(unc,ndigits)} to match data norm"
         )
         stack = [s * scale for s in stack]
 
+    
     hep.histplot(
         stack,
         histtype="fill" if not no_fill else "step",
@@ -994,8 +928,7 @@ def makeStackPlotWithRatio(
 
         for i, (proc, style) in enumerate(zip(unstacked, linestyles)):
             unstack = histInfo[proc].hists[histName]
-            if not fitresult or proc not in to_read:
-                unstack = action(unstack)[select]
+            unstack = action(unstack)[select]
             if proc != "Data":
                 unstack = unstack * scale
             if len(scaleRatioUnstacked) > i:
@@ -1042,6 +975,11 @@ def makeStackPlotWithRatio(
                 extra_labels.append(histInfo[proc].label)
             if ratio_to_data and proc == "Data" or not add_ratio:
                 continue
+            if xlim:
+                unstack = unstack[complex(0, xlim[0]) : complex(0, xlim[1])]
+            if density:
+                unstack = hh.scaleHist(unstack, np.sum(ratio_ref.values())/np.sum(unstack.values()))
+
             stack_ratio = hh.divideHists(
                 unstack,
                 ratio_ref,
@@ -1068,6 +1006,14 @@ def makeStackPlotWithRatio(
                 **optsr,
             )
 
+    if len(x_vertLines_edges):
+        h_inclusive = hh.sumHists(stack)
+        max_y = 1.05 * np.max(h_inclusive.values() + h_inclusive.variances() ** 0.5)
+        min_y = np.min(h_inclusive.values() - h_inclusive.variances() ** 0.5)
+        for x in x_vertLines_edges:
+            ax1.plot([x, x], [min_y, max_y], linestyle="--", color="black")
+            ax2.plot([x, x], [rrange[0], rrange[1]], linestyle="--", color="black")
+
     addLegend(
         ax1,
         nlegcols,
@@ -1077,7 +1023,7 @@ def makeStackPlotWithRatio(
         text_size=legtext_size,
         padding_loc=leg_padding,
     )
-    if add_ratio:
+    if add_ratio and lowerLeg:
         addLegend(
             ax2,
             lowerLegCols,
@@ -1105,7 +1051,7 @@ def makeStackPlotWithRatio(
 def makePlotWithRatioToRef(
     hists,
     labels,
-    colors,
+    colors=None,
     hists_ratio=None,
     midratio_idxs=None,
     linestyles=[],
@@ -1141,6 +1087,7 @@ def makePlotWithRatioToRef(
     cms_label=None,
     cutoff=1e-6,
     only_ratio=False,
+    ratio_legend=True,
     width_scale=1,
     automatic_scale=True,
     base_size=8,
@@ -1162,9 +1109,19 @@ def makePlotWithRatioToRef(
     elif select is not None:
         hists_ratio = [h[select] for h in hists_ratio]
 
-    if len(hists_ratio) != len(labels) or len(hists_ratio) != len(colors):
+    if colors is None:
+        colors = plt.rcParams["axes.prop_cycle"].by_key()["color"][: len(hists)]
+    if len(colors) < len(hists):
+        colors = (
+            colors
+            + plt.rcParams["axes.prop_cycle"].by_key()["color"][
+                : (len(hists) - len(colors))
+            ]
+        )
+
+    if len(hists_ratio) != len(labels):
         raise ValueError(
-            f"Number of hists ({len(hists_ratio)}), colors ({len(colors)}), and labels ({len(labels)}) must agree!"
+            f"Number of hists ({len(hists_ratio)}) and labels ({len(labels)}) must agree!"
         )
     ratio_hists = [
         hh.divideHists(
@@ -1299,7 +1256,7 @@ def makePlotWithRatioToRef(
             fill_between=fill_between,
             dataIdx=dataIdx,
             baseline=baseline,
-            add_legend=not only_ratio,
+            add_legend=ratio_legend and not only_ratio,
         )
         if midratio_hists:
             plotRatio(
@@ -1580,7 +1537,7 @@ def fix_axes(
     if noSci and not logy:
         redo_axis_ticks(ax1, "y")
     elif not logy:
-        ax1.ticklabel_format(style="sci", useMathText=True, axis="y", scilimits=(0, 0))
+        ax1.ticklabel_format(style="sci", useMathText=True, axis="y", scilimits=(-2, 2))
 
     if ratio_axes is not None:
         if not isinstance(ratio_axes, (list, tuple, np.ndarray)):
@@ -1623,6 +1580,8 @@ def redo_axis_ticks(ax, axlabel, no_labels=False):
     fixedloc = ticker.FixedLocator(
         autoloc.tick_values(*getattr(ax, f"get_{axlabel}lim")())
     )
+    if ax.get_xscale() == 'log':
+        fixedloc = ticker.LogLocator(base=10, numticks=5)
     getattr(ax, f"{axlabel}axis").set_major_locator(fixedloc)
     ticks = getattr(ax, f"get_{axlabel}ticks")()
     labels = [format_axis_num(x, ticks[-1]) for x in ticks] if not no_labels else []
@@ -1865,19 +1824,21 @@ def read_axis_label(x, labels, with_unit=True):
         return x
 
 
-def get_axis_label(config, default_keys=None, label=None, is_bin=False):
+def get_axis_label(config, default_keys=None, label=None, is_bin=False, with_unit=True):
     if label is not None:
         return label
 
     if default_keys is None:
         return "Bin index"
+    elif isinstance(default_keys, str):
+        default_keys = [default_keys]
 
     labels = getattr(config, "axis_labels", {})
 
-    if len(default_keys) == 1:        
+    if len(default_keys) == 1:
         if is_bin:
             return f"{read_axis_label(default_keys[0], labels, False)} bin"
         else:
-            return read_axis_label(default_keys[0], labels)
+            return read_axis_label(default_keys[0], labels, with_unit)
     else:
         return f"({', '.join([read_axis_label(a, labels, False) for a in default_keys])}) bin"

{wums-0.1.7 → wums-0.1.9}/wums.egg-info/PKG-INFO

@@ -1,6 +1,6 @@
-Metadata-Version: 2.2
+Metadata-Version: 2.4
 Name: wums
-Version: 0.1.7
+Version: 0.1.9
 Summary: .
 Author-email: David Walter <david.walter@cern.ch>, Josh Bendavid <josh.bendavid@cern.ch>, Kenneth Long <kenneth.long@cern.ch>, Jan Eysermans <jan.eysermans@cern.ch>
 License: MIT

{wums-0.1.7 → wums-0.1.9}/wums.egg-info/top_level.txt

@@ -1,3 +1,2 @@
 dist
-env
 wums