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

SET_TYPEOF #1312

Closed
bbolker opened this issue Jul 1, 2024 · 4 comments · Fixed by #1315
Closed

SET_TYPEOF #1312

bbolker opened this issue Jul 1, 2024 · 4 comments · Fixed by #1315

Comments

@bbolker
Copy link

bbolker commented Jul 1, 2024

With the usual apologies if this is a red herring: the lme4 package is getting

Found non-API call to R: ‘SET_TYPEOF’. Compiled code should not call non-API entry points in R.

NOTEs with r-devel. My best guess is that these are coming upstream from Rcpp, e.g. see here; I don't find any instances of SET_TYPEOF in the lme4, Matrix, or RcppEigen code ...

@eddelbuettel
Copy link
Member

We appear to have three uses of it:

edd@ahmad:~/git/rcpp(master)$ ag -c SET_TYPEOF
inst/include/Rcpp/Pairlist.h:1
inst/include/Rcpp/Language.h:1
inst/include/Rcpp/r_cast.h:1
edd@ahmad:~/git/rcpp(master)$ 

But I do not think I can offer you anything in the short term. Plead with CRAN that it is not you...

@Enchufa2
Copy link
Member

Enchufa2 commented Jul 2, 2024

Yes, it's clear that we will have to address many of these, but of course it takes time. I don't think CRAN can expect to raise those NOTEs and WARNINGs and have them fixed overnight. I hope they can live with them for a few months in downstream packages to give us here time to adapt, address them one at a time and test properly.

@bbolker
Copy link
Author

bbolker commented Jul 2, 2024

Thanks, I wasn't asking for them to be fixed in a hurry, just raising the issue.

@Enchufa2
Copy link
Member

Enchufa2 commented Jul 2, 2024

Yes, I figured that much. :) Just stating the obvious so that you have a reference to plead with CRAN. ;-)

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 a pull request may close this issue.

3 participants