Skip to content
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

Import from limma-trend approach #26

Closed
lgeistlinger opened this issue Sep 1, 2020 · 2 comments
Closed

Import from limma-trend approach #26

lgeistlinger opened this issue Sep 1, 2020 · 2 comments

Comments

@lgeistlinger
Copy link
Owner

Alright here is an update for you @lgeistlinger.

I've been using import and I have a few suggestions. For importing limma data I think there needs to be a way for using data from the limma-trend approach. Check page 72 on the limma user guide.

Instead of using voom:

#' # (3) import from voom/limma (RNA-seq count data)
#' # (3a) create the expression data object
#' library(limma)
#' keep <- filterByExpr(counts, rdesign)
#' el <- voom(counts[keep,], rdesign)
#'
#' # (3b) obtain differential expression results
#' fit <- lmFit(el, rdesign)
#' fit <- eBayes(fit, robust = TRUE)
#' res <- topTable(fit, coef = 2, number = nrow(counts), sort.by = "none")
#'
#' # (3c) import
#' se <- import(el, res)

You would use the limma-trend method:

#'   # (5) import from trend/limma (RNA-seq count data)
#'   # (5a) create the expression data object
#'   library(limma)
#'   keep <- filterByExpr(counts, rdesign)
#'   
> logCPM <- edgeR::cpm(counts[keep,], log=TRUE, prior.count=3)
#'   # (5b) obtain differential expression results 
> fit <- lmFit(logCPM, design)
> fit <- eBayes(fit, trend=TRUE)
> res <- topTable(fit, coef=ncol(design))

#'
#'   # (5c) import???
#'   se <- import(el, res)

Originally posted by @grabearummc in #23 (comment)

@lgeistlinger lgeistlinger changed the title Import from limma-trend approachImport from limma-trend approach Import from limma-trend approach Sep 1, 2020
@lgeistlinger
Copy link
Owner Author

Thanks @grabearummc ! I'll be following up on that.

@lgeistlinger
Copy link
Owner Author

That is now possible (f884102).

#' # (4) import from limma-trend (RNA-seq count data)

Thanks for pointing that out!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant