-
Notifications
You must be signed in to change notification settings - Fork 41
Rcppheaderonly #2
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
Conversation
…e for the rank calculation.
…ays of doing this but I can't work out the syntax.
…rcing; add a suppressSVD argument to do_bench().
…s that will appear in Eigen 3.0.2
…ith some Eigen classes.
…ode duplication) and tests of same.
…the extension mechanism. Should make these legitimate iterators.
…tests to avoid the .eval() methods; bump Rcpp version dependence
…x classes and tests of same
Hmm. I need to fix |
ok so there was a corner case because |
travis is happy now. @dmbates I'll leave it to you to merge this if you like. |
@romainfrancois Which version of Rcpp should I be testing this with? I have installed the current master branch (9c5babc) and consistently get failures of the tests of the form
|
Remove Rcpp from the list of Imports: and add a version to it in LinkingTo:
I'll have a look. I should probably review in which namespace was each function before and put it in the same place. but at least it works with released Rcpp (in travis it built). |
I corrected the |
…into Rcppheaderonly
So:
That type of message would appear if the I'm trying using |
This closes the enterRNGScope problem, but has opened another set of issues; |
Probably irrelevant now. |
Yes. RcppEigen is among the 157 out of 166 [ numbers as of now ] packages who work 'as-is'. See the rcpp-logs repository for details. |
requiring Rcpp 0.10.6.2 so that you get the last fix that meant undoing Rmath macros early enough.
I've also changed functions so that they use attributes (
[[Rcpp::export]]
)