Re: [Rd] [External] exists, get and get0 accept silently inputs of length > 1

2020-11-16 Thread Gabriel Becker
Hi Luke et al., Apologies. I knew there was no NSE before but incorrectly inferred from the previous message that some had been added. Should have looked at the commit myself before chiming in. Sorry for the noise. ~G On Mon, Nov 16, 2020 at 8:39 PM wrote: > Come on, folks. There is no NSE

Re: [Rd] [External] exists, get and get0 accept silently inputs of length > 1

2020-11-16 Thread luke-tierney
Come on, folks. There is no NSE involved in calls to get(): it's standard evaluation all the way into the C code. Prior to the change a first argument that is anything other than a character vector would produce an error. After the change, passing in a symbol will do the obvious thing. Code that

Re: [Rd] [External] exists, get and get0 accept silently inputs of length > 1

2020-11-16 Thread Gabriel Becker
Hi all, I have used variable values in get() as well, and including, I think, in package code (though pretty infrequently). Perhaps a character.only argument similar to library? ~G On Mon, Nov 16, 2020 at 5:31 PM Hugh Parsonage wrote: > I noticed the recent commit to R-dev (r79434). Is this

Re: [Rd] [External] exists, get and get0 accept silently inputs of length > 1

2020-11-16 Thread Hugh Parsonage
I noticed the recent commit to R-dev (r79434). Is this wise? I've often used get() in constructions like for (j in ls()) if (is.numeric(x <- get(j))) ... (and often interactively, rather than in a package) Am I to understand that get(j) will now be equivalent to `j` even if j is a string

[R-pkg-devel] Debugging library.dynam to include a manual shared object

2020-11-16 Thread Jon Davidson
Hello, I am trying to build an R package with a manually compiled .so file. To do so, I have included a call to useDynLib() in my NAMESPACE file. Building this works fine in RStudio with devtools::load_all('.'), but when I try to run R CMD INSTALL, I get the error: package or namespace load

[Bioc-devel] Bioconductor Developers Forum - 19th November

2020-11-16 Thread Mike Smith
Dear all, The next Developers Forum will take place this coming Thursday 19th Nov at 09:00 PST / 12:00 EST / 18:00 CET - You can find a calendar invite at *https://tinyurl.com/BiocDevel-2020-09-19 * We will be using BlueJeans and the meeting can be

Re: [Bioc-devel] How to Push a Bug Fix Live - MSstatsTMTPTM

2020-11-16 Thread Kern, Lori
There are a few issues. I see the pushed changes but only on the master branch. The email you received indicated it also was failing on the Release 3.12 (RELEASE_3_12) branch. Please see the instructions for pushing to both release and devel branches of Bioconductor:

[Bioc-devel] How to Push a Bug Fix Live - MSstatsTMTPTM

2020-11-16 Thread Devon Kohler
Hello, My package had a build error when it moved from development to live version of Bioconductor. I identified the issue and pushed a fix to the Bioconductor github repo. This fix went through on the development version and seemed to work but it has not been pushed live. Is there a way I can

Re: [R-pkg-devel] Having shiny as an optional dependency

2020-11-16 Thread Kamil Stachowski
It works! Thank you so much! Best wishes, Kamil Stachowski On Mon, 16 Nov 2020 at 11:53, Duncan Murdoch wrote: > On 16/11/2020 4:55 a.m., Kamil Stachowski wrote: > > I did briefly wonder why your reply to me cited an email from Akshit > Achara > > who asked about third party C++ API. I assumed

Re: [R-pkg-devel] Having shiny as an optional dependency

2020-11-16 Thread Duncan Murdoch
On 16/11/2020 4:55 a.m., Kamil Stachowski wrote: I did briefly wonder why your reply to me cited an email from Akshit Achara who asked about third party C++ API. I assumed it was because the solution to his problem and mine was the same. In the meantime, I got a reply from Kurt Hornik who said

Re: [R-pkg-devel] Having shiny as an optional dependency

2020-11-16 Thread Kamil Stachowski
I did briefly wonder why your reply to me cited an email from Akshit Achara who asked about third party C++ API. I assumed it was because the solution to his problem and mine was the same. In the meantime, I got a reply from Kurt Hornik who said I have a top-level call soundcorrsGUI <-

[R-pkg-devel] Publishing Java Wrappers The Right Way

2020-11-16 Thread Peter Meissner
Hey Everyone, I have an R-package that wraps some Java code. The dependencies needed are around 10MB. Thus I split the package into 2 packages. One providing only the dependencies: https://github.com/petermeissner/kafkaesquejars ... and one providing the actual code: