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

topDirs function isn't compatible with hic_glm when multiple contrasts are passed through #8

Open
serine opened this issue Sep 8, 2020 · 1 comment
Labels
help wanted Extra attention is needed

Comments

@serine
Copy link

serine commented Sep 8, 2020

Hi @mdozmorov ,

I'm not 100%, but it looks like topDirs function isn't compatible with hic_glm if I pass multiple contrasts.

design <- model.matrix(~0+groups)
colnames(design) <- gsub("groups", "", colnames(design))
rownames(design) <- meta$samples

contrasts <- limma::makeContrasts(mock1_vs_mock2=mock2-mock1,
                                  mock3_vs_mock4=mock4-mock3,
                                  levels = design)

hic_fit <- hic_glm(hic_exp,
                   design = design,
                   contrast = contrasts,
                   method = "QLFTest",
                   p.method = "fdr",
                   parallel = TRUE)

when I call results(hic_fit) I get nice table with all contrasts tested, but the columns labeled like this logFC.mock1_vs_mock2 and logFC.mock3_vs_mock4. From what I can tell topDirs expects logFC column label here

It isn't an issue for me, I can workaround by just using results function, but I thought it would be worth raising for future improvements.

Thanks,

@mdozmorov
Copy link
Contributor

Indeed, working with multiple contrasts would be good to implement. We haven't tested it that thoroughly. I'll keep this issue open, or @serine, please, make a pull request if you fix it.

@mdozmorov mdozmorov added the help wanted Extra attention is needed label Jul 10, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
help wanted Extra attention is needed
Projects
None yet
Development

No branches or pull requests

2 participants