Re: [R-pkg-devel] WARNING: A complete check needs the 'checkbashisms' script.

2020-08-10 Thread Rodrigo Tobar
Hi Brian, The checkbashisms script is invoked by the R checks automatically: https://github.com/wch/r-source/blob/trunk/src/library/tools/R/check.R#L1216-L1267 Thus there shouldn't be any need for you to invoke it manually in your configure script, even less so to worry about its presence.

[R-pkg-devel] WARNING: A complete check needs the 'checkbashisms' script.

2020-08-10 Thread brian knaus
Hi R-pkg-devel, My package https://github.com/knausb/vcfR has been archived on CRAN because I was asked by CRAN to address issues that I was unable to address before the deadline I was given. The issue I'm struggling with is that when I use ``` R CMD check --as-cran vcfR_1.12.0.tar.gz ``` I

[Rd] M[cbind()] <- assignment with Matrix object loses attributes

2020-08-10 Thread Ben Bolker
  Does this constitute a bug, or is there something I'm missing? assigning sub-elements of a sparse Matrix via M[X]<-..., where X is a 2-column matrix, appears to drop user-assigned attributes. I dug around in the R code for Matrix trying to find the relevant machinery but my brain started to

Re: [Bioc-devel] EXTERNAL: Author/maintainer field no longer accepted.

2020-08-10 Thread Interdonato, Kayla
Thank you for bringing this to our attention. The package guidelines should now reflect this change. Best, Kayla On 8/10/20, 5:06 AM, "Bioc-devel on behalf of Manders-2, F.M." wrote: Hi All, R BiocCheck no longer seems to accept a Author/Maintainer field.

Re: [Rd] lm() takes weights from formula environment

2020-08-10 Thread John Mount
Forgot the url: https://win-vector.com/2014/05/30/trimming-the-fat-from-glm-models-in-r/ On Aug 10, 2020, at 11:50 AM, John Mount mailto:jmo...@win-vector.com>> wrote: Thank you for your suggestion. I do know how to work around the issue. I usually build a fresh environment as a child of

Re: [Rd] lm() takes weights from formula environment

2020-08-10 Thread John Mount
Thank you for your suggestion. I do know how to work around the issue. I usually build a fresh environment as a child of base-environment and then insurt the weights there. I was just trying to provide an example of the issue. emptyenv() can not be used, as it is needed for the eval (errors

Re: [Rd] lm() takes weights from formula environment

2020-08-10 Thread Duncan Murdoch
On 10/08/2020 1:42 p.m., John Mount wrote: I wish I had started with "I am disappointed that lm() doesn't continue its search for weights into the calling environment" or "the fact that lm() looks only in the formula environment and data frame for weights doesn't seem consistent with how other

Re: [Rd] lm() takes weights from formula environment

2020-08-10 Thread William Dunlap via R-devel
I assume you are concerned about this because the formula is defined in one environment and the model fitting with weights occurs in a separate function. If that is the case then the model fitting function can create a new environment, a child of the formula's environment, add the weights

Re: [Rd] lm() takes weights from formula environment

2020-08-10 Thread John Mount
I wish I had started with "I am disappointed that lm() doesn't continue its search for weights into the calling environment" or "the fact that lm() looks only in the formula environment and data frame for weights doesn't seem consistent with how other values are treated." But I did not. So I

Re: [R-pkg-devel] Package Anchored Links with R-Dev

2020-08-10 Thread Henrik Bengtsson
These recently introduced 'R CMD check' WARNINGs were disabled again in R-devel on July 8, 2020: r78793: https://github.com/wch/r-source/commit/ccd903e47ab42e1c181396256aea56454a2e70c9 r78794: https://github.com/wch/r-source/commit/b70f90ae11dd516c1c954ed15eb5a7c2a304b37f This is because

Re: [R-pkg-devel] Package Anchored Links with R-Dev

2020-08-10 Thread John Mount
To all: I'd like to apologize for my negative tone and imprecision in my points. Thanks for the discussion and the effort you put into these systems. > On Aug 9, 2020, at 12:35 PM, Duncan Murdoch wrote: > > On 09/08/2020 3:15 p.m., Ben Bolker wrote: >> On 8/9/20 3:08 PM, Duncan Murdoch wrote:

Re: [Bioc-devel] msa: call for help/support

2020-08-10 Thread Martin Morgan
Hi Ulrich -- Certainly msa provides very useful functionality not present in other Bioconductor packages. It's not possible for the Bioconductor core team to take on additional maintenance responsibilities; the best bet is to find someone else in the community willing and able to maintain

[Bioc-devel] Author/maintainer field no longer accepted.

2020-08-10 Thread Manders-2, F.M.
Hi All, R BiocCheck no longer seems to accept a Author/Maintainer field. (https://github.com/Bioconductor/BiocCheck/commit/e0176f6591c771774f6cb05d4ff8bc1b145ad10e) * Checking for valid maintainer... * ERROR: Remove Maintainer field. Use Authors@R [cre] designation. However, the package

Re: [Rd] qnbinom with small size is slow

2020-08-10 Thread Constantin Ahlmann-Eltze via R-devel
Thanks Ben for verifying the issue. It is always reassuring to hear when others can reproduce the problem. I wrote a small patch that fixes the issue (https://github.com/r-devel/r-svn/pull/11): diff --git a/src/nmath/qnbinom.c b/src/nmath/qnbinom.c index b313ce56b2..d2e8d98759 100644 ---

[Rd] Typos in file.path documentation.

2020-08-10 Thread Rui Barradas
Hello, R 4.0.2 on Ubuntu 20.04, sessionInfo() below. I believe there are two typos in ?file.path, section Value, 2nd paragraph. 1. There is a close parenthesis missing after Encoding, as it is reading is a bit confusing, I had to backtrack and repeat. 2. I'm not a native language speaker but