-
Notifications
You must be signed in to change notification settings - Fork 12
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Allow to select a slot (e.g. scaled, raw etc..) in join_features() #69
Conversation
Make get_abundance_sc_long accept assay and slot selection
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks! Can you please add unit tests, similar to join_features wide?
R/utilities.R
Outdated
#' | ||
#' @return A Seurat object | ||
#' | ||
#' @export | ||
get_abundance_sc_long = function(.data, features = NULL, all = FALSE, exclude_zeros = FALSE){ | ||
get_abundance_sc_long = function(.data, features = NULL, all = FALSE, exclude_zeros = FALSE, assay = .data@active.assay, slot = "data"){ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
assay here is not needed as we will return all assays (inside the function is clear).
R/utilities.R
Outdated
|
||
DefaultAssay(.data) = assay | ||
for(i in Assays(.data) %>% setdiff(assay)) { | ||
.data[[i]] = NULL | ||
} | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is not needed as we return all assays
Thank you very much for your prompt review. I apologize for the misinterpretation of the feature. I have removed the |
Condrats! 🎉 |
@stemangiola
I have made the following changes to
join_features()
and checked thatR CMD CHECK
does not produce any warnings or errors. I would be grateful if you could review the PR.assay
slot
andprefix
inget_abundance_sc_wide()
get_abundance_sc_long()
handleassay
andslot
specificationassay
slot
inget_abundance_sc_long()
assay
slot
and...
injoin_features()