ODC-184 support split and duplicated dims in gs_add_gating_method().#262
ODC-184 support split and duplicated dims in gs_add_gating_method().#262
Conversation
| } | ||
|
|
||
| # format split dims & drop duplicated dims | ||
| dims <- paste(unique(dims), collapse = ",") |
There was a problem hiding this comment.
Is it OK if dims is more than 2-D?
There was a problem hiding this comment.
openCyto only supports 1D or 2D gating, but yes it will work with more than two dims.
There was a problem hiding this comment.
openCyto/R/csvTemplate-parser.R
Lines 291 to 296 in 323430e
There was a problem hiding this comment.
I would love for us to support multiple dims (CytoExploreR uses a hacky way to get around this) but the issue is that the gating functions within openCyto don't perform a dims check so we'd need to update all of them to ensure that appropriate dimensions are supplied for each of the supported gating functions.
There was a problem hiding this comment.
We could possibly check the dims against the list of supported gating methods but that will have to be updated every time we add a new gating method.
@amcdavid adding PR here to implement features or fixes required by CytoExploreR please see https://ozette.atlassian.net/jira/software/c/projects/ODC/issues/ODC-184
This PR just makes it easier for users to interact with the
gs_add_gating_method()API which currently expectsdimsto be supplied in a concatenated comma-separated string (i.e. "FSC-A,SSC-A") rather than the conventionalc("FSC-A", "SSC-A")- both inputs will be supported after this change.