Re: [Rd] Is this a bug in `[`?

2018-08-28 Thread Rui Barradas
Hello, Thanks for the pointer. Inline. On 29/08/2018 04:17, Henrik Bengtsson wrote: FYI, this behavior is documented in Section 3.4.1 'Indexing by vectors' of 'R Language Definition' (accessible for instance via help.start()): "*Integer* [...] A special case is the zero index, which has null

Re: [Rd] Is this a bug in `[`?

2018-08-28 Thread Henrik Bengtsson
FYI, this behavior is documented in Section 3.4.1 'Indexing by vectors' of 'R Language Definition' (accessible for instance via help.start()): "*Integer* [...] A special case is the zero index, which has null effects: x[0] is an empty vector and otherwise including zeros among positive or

[Rd] ROBUSTNESS: x || y and x && y to give warning/error if length(x) != 1 or length(y) != 1

2018-08-28 Thread Henrik Bengtsson
# Issue 'x || y' performs 'x[1] || y' for length(x) > 1. For instance (here using R 3.5.1), > c(TRUE, TRUE) || FALSE [1] TRUE > c(TRUE, FALSE) || FALSE [1] TRUE > c(TRUE, NA) || FALSE [1] TRUE > c(FALSE, TRUE) || FALSE [1] FALSE This property is symmetric in LHS and RHS (i.e. 'y || x' behaves

Re: [R-pkg-devel] New test in R-devel causes existing packages to fail: "Error: connections left open"

2018-08-28 Thread David B. Dahl
Okay, Uwe, I will close the connection between examples. Thanks Henrik, Duncan, Gabor, and Uwe for participating in the discussion. -- David On Mon, Aug 27, 2018 at 4:47 PM Uwe Ligges wrote: > > I still do not undertsand why you cannot stop scala and related > connections at the end of each

Re: [R-pkg-devel] Trying to work around missing functionality

2018-08-28 Thread J C Nash
Thanks for this. Also Duncan's description of how code in the R directory is executed. I've more or less figured out a workaround. Unfortunately Georgi's solution doesn't quite do the trick. Here is my current understanding and solution. Issue: I want to get root of a function of 1 parameter

Re: [Bioc-devel] Help with class lost after subsetting.

2018-08-28 Thread Hervé Pagès
On 08/28/2018 11:46 AM, Hervé Pagès wrote: On 08/27/2018 11:01 PM, Martin Morgan wrote: On 08/28/2018 12:19 AM, Charles Plessy wrote: Dear Bioconductor developers, In the CAGEr package, I created a "CAGEexp" class that extends "MultiAssayExperiment" without adding new slots, in order to

[Rd] "utils::file.edit" does not understand "editor" with additional arguments

2018-08-28 Thread Randy Lai
I am using Sublime Text as my editor. If I run `subl -n .Rprofile` in bash, a file would be opened in a new window. Back in R, if I run this > file.edit(".Rprofile", editor="'subl -n'") sh: 'subl -n': command not found Warning message: error in running command However, the interesting bit

Re: [Bioc-devel] Help with class lost after subsetting.

2018-08-28 Thread Hervé Pagès
On 08/27/2018 11:01 PM, Martin Morgan wrote: On 08/28/2018 12:19 AM, Charles Plessy wrote: Dear Bioconductor developers, In the CAGEr package, I created a "CAGEexp" class that extends "MultiAssayExperiment" without adding new slots, in order to define generic functions that require

Re: [Bioc-devel] Logolas package broken for R (3.4) users

2018-08-28 Thread Obenchain, Valerie
You could use an AMI with R 3.4 installed. See the 'AMI IDs' section: https://www.bioconductor.org/help/bioconductor-cloud-ami/#ami_ids Valerie On 08/27/2018 11:48 AM, kushal kumar dey wrote: Hi, I just found based on user reports that the old version of my package Logolas for R 3.4 and Bioc

Re: [Bioc-devel] updating development version package

2018-08-28 Thread Shepherd, Lori
You need to push any associated changes to the Bioconductor git server. See the help pages found here: http://bioconductor.org/developers/how-to/git/ Lori Shepherd Bioconductor Core Team Roswell Park Cancer Institute Department of Biostatistics & Bioinformatics Elm & Carlton Streets

[Bioc-devel] updating development version package

2018-08-28 Thread zzrickli
Hi There, I need to make a few minor change to my INDEED package, which has already been accepted. I want to know if I need to do anything other than modify the codes and bump up the version number. Thanks ZL [[alternative HTML version deleted]]

Re: [R-pkg-devel] Trying to work around missing functionality

2018-08-28 Thread Duncan Murdoch
On 28/08/2018 9:17 AM, J C Nash wrote: Indeed, it appears that globalVariables must be outside the function. However, I had quite a bit of fiddle to get things to work without errors or warnings or notes. While I now have a package that does not complain with R CMD check, I am far from

Re: [R-pkg-devel] Trying to work around missing functionality

2018-08-28 Thread Georgi Boshnakov
If you don't insist on putting the variable in the global environment, variations of the following give a cleaner solution: TraceSetup_1 <- local({ ifn = 0 igr = 0 ftrace = FALSE fn = NA gr = NA function(ifn=0, igr=0, ftrace=FALSE, fn=NA, gr=NA){ ifn<<- ifn

Re: [Bioc-devel] Don't have access to new package

2018-08-28 Thread Obenchain, Valerie
Hi John, The error was on our end. I've cleaned this up and you should now have access to multiHiCcompare and HiCcompare. Valerie On 08/27/2018 11:13 PM, John Stansfield wrote: I recently had a new package accepted to Bioconductor, however when I try to push changes upstream I get the

Re: [R-pkg-devel] Trying to work around missing functionality

2018-08-28 Thread J C Nash
Indeed, it appears that globalVariables must be outside the function. However, I had quite a bit of fiddle to get things to work without errors or warnings or notes. While I now have a package that does not complain with R CMD check, I am far from satisfied that I can give a prescription. I had

[Bioc-devel] Don't have access to new package

2018-08-28 Thread John Stansfield
I recently had a new package accepted to Bioconductor, however when I try to push changes upstream I get the following error: FATAL: W any packages/multiHiCcompare j.stansfield DENIED by fallthru (or you mis-spelled the reponame) I also checked on the bioconductor git credentials and under

Re: [Bioc-devel] Help with class lost after subsetting.

2018-08-28 Thread Martin Morgan
On 08/28/2018 12:19 AM, Charles Plessy wrote: Dear Bioconductor developers, In the CAGEr package, I created a "CAGEexp" class that extends "MultiAssayExperiment" without adding new slots, in order to define generic functions that require CAGEr-specific contents in the colData slot.