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

upset R package compatibility #18

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

chrisamiller
Copy link

In the R upset package installed in the bioconda/quay repository, the options mainbar.y.label and sets.x.label do not exist, causing a failure:

$ intervene upset --names A,B,C -o . -i $a $b $c

Running UpSet module. Please wait...

Error in upset(fromExpression(expressionInput), nsets = 3, nintersects = 30,  :
  unused arguments (mainbar.y.label = "No. of Intersections", sets.x.label = "Set size")
Execution halted

You are done! Please check your results @ ..
Thank you for using Intervene!

This removes those options, which allows it to proceed

These two params don't exist in the upset R package version that's packaged with intervene in Conda. Removing this fixes the problem.
they don't exist in the R package
rm sets.x.label and mainbar.y.label from R code
@asntech
Copy link
Owner

asntech commented Oct 19, 2019

Hello @chrisamiller! Please accept my apologies for getting back to you very late. Do you remember what version of R and UpSet package you were using? Thanks!

@chrisamiller
Copy link
Author

I don't know offhand, but I was running it from the bioconda container, so the environment should be available in that docker image:
https://bioconda.github.io/recipes/intervene/README.html

Thanks for looking into it!

@asntech
Copy link
Owner

asntech commented Oct 14, 2020

This seems to be due to the older version of UpSetR. Please check #38

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

Successfully merging this pull request may close these issues.

None yet

2 participants