Re: [Rd] winbuilder warning message wrt function pointers

2017-12-29 Thread Therneau, Terry M., Ph.D.
Bill,   That's a very nice solution.  It is both cleaner looking and preferable to track R's .h files. However, some of my routines don't have void * as the return type (two are int *), and Rdynload has    typedef void * (*DL_FUNC)(); Will this untruth mess anything up? Terry T. On

Re: [Rd] winbuilder warning message wrt function pointers

2017-12-29 Thread William Dunlap via R-devel
And remove the cast on the return value of R_GETCCallable. And check that your function is found before using it. #include #include #include void bdsmatrix_prod4(int nrow,int nblock, int *bsize, double *bmat, double *rmat, int nfrail, double

Re: [Rd] winbuilder warning message wrt function pointers

2017-12-29 Thread William Dunlap via R-devel
Try changing static void (*fun)() = NULL; to DL_FUNC fun = NULL; Bill Dunlap TIBCO Software wdunlap tibco.com On Fri, Dec 29, 2017 at 5:14 AM, Therneau, Terry M., Ph.D. < thern...@mayo.edu> wrote: > I've recently updated the coxme package, which calls internal routines > from the bdsmatrix

Re: [R-pkg-devel] File name error

2017-12-29 Thread Uwe Ligges
On 21.12.2017 18:22, Henrik Bengtsson wrote: Uwe/Kurt, is this You mean the new check? This is R. controlled by R itself or by the host system? I suspect one of my R.filesets tests(*) fails (since a few days) because of this, but it might actually be a false positive. I fail to reproduce

Re: [R-pkg-devel] File name error

2017-12-29 Thread Uwe Ligges
On 27.12.2017 18:55, Cathy Lee Gierke wrote: My program puts the output into the folder where it found the input. Please let the user specify the output dir. Otherwise there is the risk to overwrite files that the user want to keep. Must I move the input files to tmpdir() or are they

Re: [Rd] data compression in a package

2017-12-29 Thread Dirk Eddelbuettel
On 29 December 2017 at 07:59, Therneau, Terry M., Ph.D. wrote: | which is already taken care of)." The survival package has about 2M of .rda files and | 2.2M of vignettes so I'm still under the 5M boundary, but R CMD check nags about it. The nags are genuinely irritating, especially as they

Re: [Rd] data compression in a package

2017-12-29 Thread Therneau, Terry M., Ph.D.
On 12/29/2017 07:34 AM, Dirk Eddelbuettel wrote: On 29 December 2017 at 07:23, Therneau, Terry M., Ph.D. wrote: | The submission guide has the following cryptic (to me) sentence: |    "Reasonable compression should be used for data (not just .rda files) " | | The survival pacakge has a

Re: [Rd] Numerical stability in chisq.test

2017-12-29 Thread Jan Motl
Hi, there is also PR#8224, which seems to be relevant. I executed the following code: ## Modify the function chisq.test2 <- edit(chisq.test) # Modify to use increasing order of sorting at line 57 ## PR#8224 (patological contingency table) m <- matrix(c(1,0,7,16),2,2); # Original original <-

Re: [Rd] data compression in a package

2017-12-29 Thread Dirk Eddelbuettel
On 29 December 2017 at 07:23, Therneau, Terry M., Ph.D. wrote: | The submission guide has the following cryptic (to me) sentence: |    "Reasonable compression should be used for data (not just .rda files) " | | The survival pacakge has a fairly large number of data files --- exactly what

[Rd] data compression in a package

2017-12-29 Thread Therneau, Terry M., Ph.D.
The submission guide has the following cryptic (to me) sentence:    "Reasonable compression should be used for data (not just .rda files) " The survival pacakge has a fairly large number of data files --- exactly what should I be doing?   xz compression? Terry T.

[Rd] winbuilder warning message wrt function pointers

2017-12-29 Thread Therneau, Terry M., Ph.D.
I've recently updated the coxme package, which calls internal routines from the bdsmatrix package.  (It is in fact mentioned as an example of this in the Extensions manual.) The call connections are a blocks like this, one for each of the 9 called C routines. void bdsmatrix_prod4(int nrow,