amc-heme / SCUBA

Plotting functions for SingleCellExperiment objects
MIT License
4 stars 0 forks source link

split_by variable not properly processed via plot_violin #68

Closed wish1832 closed 1 year ago

wish1832 commented 1 year ago

The split by behavior of plot_violin is inconsistent with Seurat::VlnPlot().

Expected behavior:

Seurat::VlnPlot(
  object = AML_Seurat,
  features = rownames(AML_Seurat)[3],
  group.by = "Batch",
  split.by = "seurat_clusters"
)

image

Observed behavior:

SCUBA::plot_violin(
  object = AML_Seurat,
  features = rownames(AML_Seurat)[3],
  group_by = "Batch",
  split_by = "seurat_clusters"
)

image

The behavior appears equivalent to plots produced with plot_violin without a split_by argument.

SCUBA::plot_violin(
  object = AML_Seurat,
  features = rownames(AML_Seurat)[3],
  group_by = "Batch"
)

image