Merge pull request #321 from fidelram/fix_extra_space

fixed #309 (Extra space around multiple scatter plot panels)
scverse · Oct 23, 2018 · 76f8c26 · 76f8c26
2 parents c98fe31 + 55c64fe
commit 76f8c26
Show file tree

Hide file tree

Showing 3 changed files with 1 addition and 1 deletion.
diff --git a/scanpy/plotting/tools/scatterplots.py b/scanpy/plotting/tools/scatterplots.py
@@ -236,7 +236,7 @@ def plot_scatter(adata,
         from matplotlib import gridspec
         # set up the figure
         num_panels = len(color) * len(components_list)
-        n_panels_x = ncols
+        n_panels_x = min(ncols, num_panels)
         n_panels_y = np.ceil(num_panels / n_panels_x).astype(int)
         # each panel will have the size of rcParams['figure.figsize']
         fig = pl.figure(figsize=(n_panels_x * rcParams['figure.figsize'][0] * (1 + wspace),

diff --git a/scanpy/tests/_images/master_umap_gene_expr.png b/scanpy/tests/_images/master_umap_gene_expr.png
diff --git a/scanpy/tests/_images/master_violin_multi_panel.png b/scanpy/tests/_images/master_violin_multi_panel.png