MAINT: simplify get_heatmap_df()

tylerjereddy · tylerjereddy · commit 69377a639047 · 2022-08-16T15:27:56.000-06:00
* remove an extraneous step from `get_heatmap_df()`;
there should never be a fractional bin occupancy greater
than 1 achieved by multiplying fractions
diff --git a/darshan-util/pydarshan/darshan/experimental/plots/heatmap_handling.py b/darshan-util/pydarshan/darshan/experimental/plots/heatmap_handling.py
@@ -350,7 +350,6 @@ def get_heatmap_df(agg_df: pd.DataFrame, xbins: int, nprocs: int) -> pd.DataFram
     combo = start_fraction_bin_occupancy.mul(
         end_fraction_bin_occupancy, fill_value=1, axis=0
     )
-    combo.mask(combo > 1, 1, inplace=True)
     # add the start/end dummy variable dataframes
     # and replace any zeros with NaN's
     cats = cats_start.add(cats_end, fill_value=0)

Original file line number	Diff line number	Diff line change
`@@ -350,7 +350,6 @@ def get_heatmap_df(agg_df: pd.DataFrame, xbins: int, nprocs: int) -> pd.DataFram`
`350`	`350`	`combo = start_fraction_bin_occupancy.mul(`
`351`	`351`	`end_fraction_bin_occupancy, fill_value=1, axis=0`
`352`	`352`	`)`
`353`		`- combo.mask(combo > 1, 1, inplace=True)`
`354`	`353`	`# add the start/end dummy variable dataframes`
`355`	`354`	`# and replace any zeros with NaN's`
`356`	`355`	`cats = cats_start.add(cats_end, fill_value=0)`