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

update venn_list to use rpy2 instead of rpy #12

Open
remimarenco opened this issue Nov 4, 2015 · 6 comments
Open

update venn_list to use rpy2 instead of rpy #12

remimarenco opened this issue Nov 4, 2015 · 6 comments

Comments

@remimarenco
Copy link

Hi Peter,

I managed to install the tool by hand, but it seems to need rpy...do we need the old rpy to resolve this?

Fatal error: Exit code 1 () Requires the Python library rpy (to call R)

Thanks!

@peterjc
Copy link
Owner

peterjc commented Nov 4, 2015

If you are referring to https://github.com/peterjc/pico_galaxy/tree/master/tools/venn_list yes, it needs rpy v1 not rpy2. Like the rest of Galaxy, it really ought to be updated since rpy v1 has been superseded.

Have I misunderstood, or can I retitle this bug for updating venn_list to use rpy2?

@peterjc
Copy link
Owner

peterjc commented Nov 4, 2015

Or did you mean to comment on #11 instead?

@remimarenco
Copy link
Author

Yes that would be nice to use rpy2! As it is not really an issue, I guess we can rename it, thanks!

@peterjc peterjc changed the title Fatal error: Exit code 1 () Requires the Python library rpy (to call R) update venn_list to use rpy2 instead of rpy Nov 4, 2015
@bgruening
Copy link
Contributor

@remimarenco is this one not installing for you properly?
I have changed the requirements in 370b7cd and it should work now.

@peterjc
Copy link
Owner

peterjc commented Nov 5, 2015

I think you're discussing the error on #11, but I thought the main and Test Tool Shed already had @bgruening's update 370b7cd included:

@remimarenco
Copy link
Author

Hmmm, I will try the manual installation and get the requirements through the toolshed!

Thanks for the explanation :). (I did not check the date, thought it was a now-update :p )

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

3 participants