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

R and rpy2 #1440

Merged
merged 9 commits into from Jul 6, 2015
Merged

R and rpy2 #1440

merged 9 commits into from Jul 6, 2015

Conversation

@sankhesh
Copy link
Contributor

@sankhesh sankhesh commented Jul 3, 2015

Enable R and RPY2 support

sankhesh and others added 9 commits Jun 26, 2015
This change fixes issue #1405
When RPY2 is built against R that does not have readline support,
importing rpy2 results in error.
readline is listed as a dependency in RPY2's documentation.

This change addresses issue #1405
added aqua for mac
added openmp for parallel builds
disabled -j1 for install step
@sankhesh
Copy link
Contributor Author

@sankhesh sankhesh commented Jul 3, 2015

Taken over from #1437

@aashish24 @doutriaux1 Please review

@sankhesh sankhesh mentioned this pull request Jul 3, 2015
@@ -0,0 +1,16 @@
set( SINGLEDISPATCH_MAJOR 3 )
Copy link
Contributor

@aashish24 aashish24 Jul 6, 2015

why do we need this package?

Copy link
Contributor

@aashish24 aashish24 Jul 6, 2015

Can we not add this to the list of requirements?

Copy link
Contributor

@doutriaux1 doutriaux1 Jul 6, 2015

ubuntu needs it, and it's python based so can't be in requirements

Copy link
Contributor

@aashish24 aashish24 Jul 6, 2015

@doutriaux1 can you explain bit more on why Ubuntu needs it?

Copy link
Contributor

@doutriaux1 doutriaux1 Jul 6, 2015

setup.py on ubuntu lists it as a dependeny. So inside our firewall it died because it couldn't fetch it. It's yet another example of the "cool/sneaky" features of pip...

Copy link
Contributor

@aashish24 aashish24 Jul 6, 2015

Coo! thanks for the information. Very useful.

  • Aashish

doutriaux1 added a commit that referenced this issue Jul 6, 2015
@doutriaux1 doutriaux1 merged commit 7d0a662 into master Jul 6, 2015
3 checks passed
@doutriaux1 doutriaux1 deleted the r_and_rpy2 branch Jul 6, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

3 participants