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

Support non decreasing sorts in reactome pathway analysis #37

Open
7 of 8 tasks
Nelson-Gon opened this issue Jan 6, 2022 · 0 comments
Open
7 of 8 tasks

Support non decreasing sorts in reactome pathway analysis #37

Nelson-Gon opened this issue Jan 6, 2022 · 0 comments

Comments

@Nelson-Gon
Copy link
Contributor

Prerequisites

  • Have you read Feedback and follow the guide?
    • make sure your are using the latest release version
    • read the documents
    • google your quesion/issue

Describe you issue

  • Make a reproducible example (e.g. 1)
  • your code should contain comments to describe the problem (e.g. what expected and actually happened?)

Hi, I have been exploring this package and once again stumbled on the strict decreasing only sort. Here is how to reproduce:

data("geneList", package="DOSE")
non_decreasing <-sort(geneList, decreasing=FALSE)
ReactomePA::gsePathway(non_decreasing, 
                pvalueCutoff = 0.2,
                pAdjustMethod = "BH", 
                verbose = FALSE)

The above will throw the following error:

Error in GSEA_internal(geneList = geneList, exponent = exponent, minGSSize = minGSSize, :
geneList should be a decreasing sorted vector...

This is related to YuLab-SMU/DOSE#58. I also see that in a comment, there is some suggestion that this was fixed but looking at the code in DOSE, isSorted still only allows for a decreasing sort.

This is also related to PR YuLab-SMU/DOSE#59 where I had attempted to fix this.

Thank you,

NelsonGon

Ask in right place

  • for bugs or feature requests, post here (github issue)
  • for questions, please post to Bioconductor or Biostars with tag ReactomePA
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant