Re: [R-pkg-devel] R CMD check getting stalled

2015-05-28 Thread Hadley Wickham
> Wild guess: It's not the package but the file system or file permissions so > that you cannot determine the size. Do you have any thoughts on where specifically to look? Permissions of the library or of the current dir? I also wondered if it might be caused by a recursive link that caused the s

Re: [R-pkg-devel] Keeping secrets in R packages

2015-06-06 Thread Hadley Wickham
The API key and "secret" aren't actually secrets so you can include in the package. Hadley On Friday, June 5, 2015, Neal Fultz wrote: > I am working on updating the RGoogleDocs package to use OAuth2, since the > old api was recently turned off (at my job we use it as a poor mans > database). >

Re: [R-pkg-devel] how to skip tests on CRAN but NOT on travis-ci?

2015-06-22 Thread Hadley Wickham
On Sun, Jun 21, 2015 at 6:17 PM, Jennifer Bryan wrote: > Thanks for all of the responses! > > What Gabor says below is true and is the path of least resistance in my > particular case. As Dirk suggested, environment variables are the way to go. > > If I put this into .travis.yml: > > env: > - N

Re: [R-pkg-devel] no visible global function definition

2015-06-29 Thread Hadley Wickham
> and available as an RSS feed. (If you prefer Twitter, feel free to > mirror the RSS feed there, if that's possible. I don't use Twitter, so > I won't do it.) It seems to have become known here essentially > immediately, so I'm not sure how an announcement would help. > > For the reasoning: th

Re: [R-pkg-devel] Saving binary files in a package

2015-07-04 Thread Hadley Wickham
On Sat, Jul 4, 2015 at 10:00 AM, Dirk Eddelbuettel wrote: > > On 4 July 2015 at 09:12, Thierry Onkelinx wrote: > | You have no guarantee that the user has write access to those folders. So > | you might want to rethink your strategy. > > Seconded. And I am not aware of a good cross-platform soluti

Re: [R-pkg-devel] query about package design

2015-07-10 Thread Hadley Wickham
On Fri, Jul 10, 2015 at 12:50 PM, ProfJCNash wrote: > I have a general question about appropriate design for a fairly > complicated package, as I am redesigning and rebuilding the optimx > package. It will get a new name as it appears that the calls will change > in order to accommodate new functi

Re: [R-pkg-devel] Help needed to setting custom packages for R 3.1.2

2015-07-20 Thread Hadley Wickham
On Mon, Jul 13, 2015 at 5:38 PM, Duncan Murdoch wrote: > On 13/07/2015 5:36 PM, Seth Wenchel wrote: >> Are you exporting the functions from pkgB and pkgC? It's hard to tell from >> your table below. The easiest way is to add a comment before each of the >> functions in pkgB and pkgC that you want

Re: [R-pkg-devel] need some help to understand package build workflow

2015-07-30 Thread Hadley Wickham
On Thursday, July 30, 2015, Duncan Murdoch wrote: > On 30/07/2015 9:14 PM, Duncan Murdoch wrote: > > On 30/07/2015 8:49 PM, Glenn Schultz wrote: > >> Hi All, > >> > >> I have a package stable and working. Now, I am trying to consolidate > some functions that share similar inputs. Example below.

Re: [R-pkg-devel] need some help to understand package build workflow

2015-07-31 Thread Hadley Wickham
> > The issue occurs when a function and an S4 generic have the same name; > I imagine this is fairly uncommon? Wouldn't the function be masked by > the S4 generic and effectively be invisible after the package was > loaded? > > On Thu, Jul 30, 2015 at 7:53 PM, Hadley Wickha

Re: [R-pkg-devel] need some help to understand package build workflow

2015-07-31 Thread Hadley Wickham
>> It seems arguable that this is actually a roxygen bug - the function >> that's actually exported from the package does not have a type >> argument. > > The package fails R CMD check because of a bad .Rd file. If a user had > written that file, it would be user error, but roxygen2 wrote it, so I

Re: [R-pkg-devel] need some help to understand package build workflow

2015-08-03 Thread Hadley Wickham
On Fri, Jul 31, 2015 at 5:14 PM, Kevin Ushey wrote: > I'm not sure if that's correct -- from what I see, in the generated Rd > documentation: > > - The 'usage' is drawn from the S4 generic, > - The 'arguments' are drawn from the function. > > I think R CMD check is correctly warning about that. A

Re: [R-pkg-devel] Working with connections

2015-08-09 Thread Hadley Wickham
Also it's a little strange to put an RDS file _inside_ a gz, since normally the compression is done internally. And are you sure you should be exposing this data via a function, rather than using the regular package data mechanism? Hadley On Sun, Aug 9, 2015 at 7:04 AM, Joshua Ulrich wrote: > O

Re: [R-pkg-devel] how to make "r-release-osx-x86_64-mavericks" machine support rgl 3D plots

2015-08-17 Thread Hadley Wickham
>> | Check DetailsVersion: 1.8.3 >> | Check: tests >> | Result: ERROR >> | Running the tests in ‘tests/stdUsage.R’ failed. >> | Last 13 lines of output: >> | > plot(ffTest42,col=Col,plot_GOF=TRUE,speed=T) >> | [1] "compute goodness-of-fit with leave-one-out k-nearest >> neighbor(

Re: [R-pkg-devel] Define S4 method for generic in other package

2015-08-31 Thread Hadley Wickham
I'd recommend creating a minimal package that illustrates the problem using an existing S4 generic in a base package (e.g. stats4::AIC). That makes it easier to figure out what's gone wrong. It's unlikely to be a roxygen2 problem since the translation from @importFrom foo bar (etc) to importFrom(f

Re: [R-pkg-devel] Define S4 method for generic in other package

2015-09-01 Thread Hadley Wickham
ata. ~ Roger Brinner > The combination of some data and an aching desire for an answer does > not ensure that a reasonable answer can be extracted from a given body > of data. ~ John Tukey > > > 2015-09-01 0:32 GMT+02:00 Hadley Wickham : >> I'd recommend creating a mini

Re: [R-pkg-devel] Redefinition of generic for plot function breaks plot.formula

2015-09-01 Thread Hadley Wickham
Why don't you just create your own function? Hadley On Tue, Sep 1, 2015 at 8:08 AM, Benjamin Hofner wrote: > Dear Gavin, > > unfortunately, I cannot overwrite plot.data.frame only. If I do this I > get the following warning from R CMD check: > > * checking use of S3 registration ... WARNING > Re

Re: [R-pkg-devel] problem with locale-dependent order()

2015-09-21 Thread Hadley Wickham
Hi Alex, As far as I know, that's the best solution. In roxygen, I've wrapped it up with a couple of helpers: set_collate <- function(locale) { cur <- Sys.getlocale(category = "LC_COLLATE") Sys.setlocale(category = "LC_COLLATE", locale = locale) cur } with_collate <- function(locale, code)

Re: [R-pkg-devel] CRAN checks question

2015-10-22 Thread Hadley Wickham
I've also seen some problems with missing testthat on r-devel-windows-ix86+x86_64 - I'd say it's a temporary problem on the CRAN end, and if your package is ok on win builder, you should be ok to submit. Hadley On Wed, Oct 21, 2015 at 5:15 PM, Derek Ogle wrote: > I was hoping to submit a new ver

Re: [R-pkg-devel] S3 method, useMethod and <-, using Roxygen

2015-10-23 Thread Hadley Wickham
Here's a minimal reprex: out <- roc_proc_text(rd_roclet(), " #' Foo `foo<-` <- function(x, y, value) { UseMethod('set_labels<-') } #' Foo `foo<-.default` <- function(x, y, value) { x } ")[[2]] cat(format(out)) That generates \usage{ \method{foo}{default}(x, y) <- value }

Re: [R-pkg-devel] Strange warnings from build

2015-11-27 Thread Hadley Wickham
Take a look at this line: #' #' Pass through OAS engine Hadley On Sat, Nov 28, 2015 at 11:33 AM, Glenn Schultz wrote: > Hello All, > > When I build my package. I get the following warnings. I think these are > related to LaTex and ElCapitan but are flowing through to my R build. All > the tes

Re: [R-pkg-devel] How to make S3-method for the "format" generic for class inheriting from "AsIs"

2015-12-08 Thread Hadley Wickham
Why not just make your own as.data.frame method? e.g. as.data.frame.pin <- function(x, ...) { structure( list(x), class = "data.frame", row.names = .set_row_names(length(x)) ) } data.frame() calls as.data.frame() on all of its arguments. Hadley On Tue, Dec 8, 2015 at 5:13 AM, Er

Re: [R-pkg-devel] RCMD Note - why did I have to do this?

2015-12-20 Thread Hadley Wickham
You didn't include the NOTE you saw, and this isn't valid roxygen: #' @imports methods::fields So it's a bit hard to tell what the problem is. Hadley On Sun, Dec 20, 2015 at 12:02 PM, Glenn Schultz wrote: > Hello all, > > I need some help understanding namespace and methods. > > I received a

Re: [R-pkg-devel] S3 length method behavior

2016-02-02 Thread Hadley Wickham
I've found that it's a very bad idea to provide length or names methods for just this reason. Hadley On Sat, Jan 30, 2016 at 1:25 PM, Nathan Wendt wrote: > Hello, > > I have run into an issue while developing an R package. Specifically, my > issue relates to what happens when I define an S3 lengt

Re: [R-pkg-devel] Documenting S4 generic does not show usage

2016-03-19 Thread Hadley Wickham
The problem seems to be your use of methods::setMethod instead of setMethod (etc). Roxygen2 doesn't seem to be quite smart enough to realise that these aren't equivalent. You might want to file an issue on github so I look into this the next time I'm working on roxygen2. Hadley On Fri, Mar 18, 20

Re: [R-pkg-devel] Documenting S4 generic does not show usage

2016-03-21 Thread Hadley Wickham
rm :-) > > I would appreciate if you could please answer (very briefly) questions 1 to > 3, so I can learn how to correctly document S4 generics properly in the > future. > > Thanks again. > > 2016-03-18 15:11 GMT+01:00 Hadley Wickham : >> >> The problem seems to be

Re: [R-pkg-devel] Roxygen help documenting a S4 class

2016-04-22 Thread Hadley Wickham
Does this help? code <- c("BeginningBal", "Convexity", "EffConvexity", "EffDuration", "EndingBal", "Formula", "Horizon", "HorizonReturn", "KeyRateConvexity", "KeyRateDuration", "KeyRateTenor", "ModDuration", "Name", "PassThroughInterest", "Period", "PmtDate", "PrepaidPrin", "SMM", "Scheduled

Re: [R-pkg-devel] Possible R CMD check extensions (Was: Possibly mis-spelled words in DESCRIPTION)

2016-05-08 Thread Hadley Wickham
On Sun, May 8, 2016 at 10:15 AM, Duncan Murdoch wrote: > On 08/05/2016 10:47 AM, Dirk Eddelbuettel wrote: >> >> >> On 8 May 2016 at 16:18, Uwe Ligges wrote: >> | On 08.05.2016 16:13, carlos cinelli wrote: >> | > How should I proceed in this case? >> | >> | Submit to CRAN. >> >> The deeper question

Re: [R-pkg-devel] Data for unit testing in packages

2016-05-20 Thread Hadley Wickham
If you're using testthat, check out expect_equal_to_reference. Hadley On Fri, May 20, 2016 at 7:45 PM, carlos cinelli wrote: > Do you guys have any suggestions on how to store data for unit testing? > > I am implementing some methods to estimate matrix entries from the > marginals. The input of

Re: [R-pkg-devel] Getting a package ready for CRAN - some questions

2016-05-31 Thread Hadley Wickham
On Mon, May 30, 2016 at 7:10 PM, Roy Mendelssohn - NOAA Federal wrote: > Hi All: > > I am in the midst of some minor revisions to my xtractomatic package, which > up till now has only been on github. Since ncdf4 for Windows is now > available from CRAN (and many thanks to whomever is responsib

Re: [R-pkg-devel] Submit second version of R package to CRAN

2016-06-06 Thread Hadley Wickham
On Sun, Jun 5, 2016 at 1:53 AM, Uwe Ligges wrote: > I do not know devtools: > If it worked for you for the initial submission and the CRAN team did not > complain, please resubmit in the same way - and keep in mind you want to > check the package in advance using a very recent version of R. And s

Re: [R-pkg-devel] modification of warning messages

2016-06-22 Thread Hadley Wickham
Try warningcall(R_NilValue, "message") Hadley On Wednesday, June 22, 2016, Aaron King wrote: > I am in the midst of trying to improve error and warning messages in a > complex package. I find myself wanting to modify R's default behavior when > printing warnings. Specifically, it frequently h

Re: [R-pkg-devel] Absent variables and tibble

2016-06-27 Thread Hadley Wickham
On Mon, Jun 27, 2016 at 9:03 AM, Duncan Murdoch wrote: > On 27/06/2016 9:22 AM, Lenth, Russell V wrote: >> >> My package 'lsmeans' is now suddenly broken because of a new provision in >> the 'tibble' package (loaded by 'dplyr' 0.5.0), whereby the "[[" and "$" >> methods for 'tbl_df' objects - as d

Re: [R-pkg-devel] Absent variables and tibble

2016-06-27 Thread Hadley Wickham
if x has var in it. > > Thanks > > Russ > >> On Jun 27, 2016, at 9:47 AM, Hadley Wickham wrote: >> >> On Mon, Jun 27, 2016 at 9:03 AM, Duncan Murdoch >> wrote: >>> On 27/06/2016 9:22 AM, Lenth, Russell V wrote: >>>> >>>> My p

Re: [R-pkg-devel] duplicate function during build

2016-07-23 Thread Hadley Wickham
I think this sort of meta problem is best solved with svn/git because you can easily see if the changes you think you made align with the changes you actually made. Learning svn or git is a lot of work, but the payoff is worth it. Hadley On Friday, July 22, 2016, ProfJCNash wrote: > In trying t

Re: [R-pkg-devel] Handling Not-Always-Needed Dependencies?

2016-08-02 Thread Hadley Wickham
>> It is really simple: Having _both_ Suggests: foo _and_ an unconditonal >> call >> to foo::bar() in the code. >> >> Whereas Josh and I argue that it needs to be conditional on requireNames() >> coming back TRUE. > > > I am feeling particularly dense today. What about "Having _both_ Suggests: >

Re: [R-pkg-devel] [Learning] the secret of Win[dows C-backed packages]

2016-08-15 Thread Hadley Wickham
One other option is to include the src for libmagic inside your package, and use that as a fallback if it's not installed on the system. That gives linux users what they want (linking to the system package), and windows users what they want (it just works). It only works for simple libraries, but i

Re: [R-pkg-devel] Understanding notes from win-builder

2016-10-03 Thread Hadley Wickham
On Mon, Oct 3, 2016 at 12:10 PM, Roy Mendelssohn - NOAA Federal wrote: > Hi All: > > I am working on preparing a package for submission to CRAN, so using devtools > i submitted the package to win-builder. I really appreciate that this has > been setup, so that hopefully any final submission wil

Re: [R-pkg-devel] R CMD Check: Tests running infinite

2017-02-07 Thread Hadley Wickham
>> A lot of packages have to work around this: >> https://github.com/search?q=user%3Acran+R_TESTS&type=Code > > I wonder if those are mostly there because of cut'n'paste behavior. It's not something I've ever advocated; I didn't realise there were so many people unsetting it. (devtools does it for

Re: [R-pkg-devel] .bib file in vignette not found during R CMD check

2017-02-28 Thread Hadley Wickham
I wonder if it would be useful for R CMD build to list the files it ignores? A misspecified .Rbuildignore seems to be reasonable for a high proportion of weird errors where you have no idea what's going wrong. Hadley On Tue, Feb 28, 2017 at 11:57 AM, Duncan Murdoch wrote: > On 28/02/2017 10:44 A

Re: [R-pkg-devel] Referring to data in R/sysdata.rda

2017-04-15 Thread Hadley Wickham
Just ignore it. Hadley On Friday, April 14, 2017, Roy Mendelssohn - NOAA Federal < roy.mendelss...@noaa.gov> wrote: > In my package I have a dataframe that I use but want to hide from the > user. In Hadley's book on R Packages, he says: > > > • If you want to store parsed data, but not ma

Re: [R-pkg-devel] tibbles are not data frames

2017-09-26 Thread Hadley Wickham
On Tue, Sep 26, 2017 at 8:28 AM, Jeroen Ooms wrote: > On Tue, Sep 26, 2017 at 11:56 AM, Gábor Csárdi wrote: >> >> On Tue, Sep 26, 2017 at 10:35 AM, Joris Meys wrote: >> > I don't like the dropping of dimensions either. That doesn't change the >> > fact that a tibble reacts different from a data.

Re: [R-pkg-devel] tibbles are not data frames

2017-09-26 Thread Hadley Wickham
On Tue, Sep 26, 2017 at 2:30 AM, Göran Broström wrote: > I am beginning to get complaints from users of my CRAN packages (especially > 'eha') to the effect that they get error messages like "Error: Unsupported > use of matrix or array for column indexing". > > It turns out that they are sticking i

Re: [R-pkg-devel] tibbles are not data frames

2017-09-26 Thread Hadley Wickham
On Tue, Sep 26, 2017 at 8:35 AM, Joris Meys wrote: > Where its parent class _sometimes_ returns an atomic vector and >> >> _sometimes_ returns a data frame. > > Indeed. And a tibble doesn't, so there's a conflict. Nobody said data.frame > works better than tibble. Actually, we all agree that the l

Re: [R-pkg-devel] tibbles are not data frames

2017-09-26 Thread Hadley Wickham
On Tue, Sep 26, 2017 at 8:51 AM, Pedro J. Aphalo wrote: > What I think is troublesome is that data.frame is part of the definition > of the R language, and the expectation based on R's normal behaviour is > that testing with is.data.frame() should be enough to ensure that an > object can be treate

Re: [R-pkg-devel] tibbles are not data frames

2017-09-26 Thread Hadley Wickham
On Tue, Sep 26, 2017 at 9:22 AM, Patrick Perry wrote: > Would it be possible to change tibbles so that > > x[,1,drop=TRUE] > > returns a vector, not a data frame? I certainly find it surprising that > tibbles ignore > the drop argument. If tibbles respeced the drop argument, then package > develop

Re: [R-pkg-devel] tibbles are not data frames

2017-09-26 Thread Hadley Wickham
> I for one am happy this discussion pops up, because it's a piece of > information I give to my students as well: convert to a data.frame when you > start your analysis just to play safe. And this discussion shows why that is > -for the time being!- a good advice. The moment tibbles become the def

Re: [R-pkg-devel] tibbles are not data frames

2017-09-26 Thread Hadley Wickham
On Tue, Sep 26, 2017 at 10:40 AM, Joris Meys wrote: > On Tue, Sep 26, 2017 at 5:33 PM, Hadley Wickham wrote: >> >> > I for one am happy this discussion pops up, because it's a piece of >> > information I give to my students as well: convert to a data.frame when &

Re: [R-pkg-devel] tibbles are not data frames

2017-09-26 Thread Hadley Wickham
On Tue, Sep 26, 2017 at 12:15 PM, Patrick Perry wrote: > Pro ignoring x[,1,drop=TRUE]: > (1) it forces users to write consistent code for extracting a vector from a > data frame > > Con: > (1) functions that accept both matrices and data frames might break > (x[[j]][i] doesn't work for a matrix)

Re: [R-pkg-devel] tibbles are not data frames

2017-09-26 Thread Hadley Wickham
> If that is right -- and I tend to believe it is right -- this change had > better been done in R core and not on package level. I think the root of > this evil is design inconsistencies of the language together with the lack > of removing these inconsistencies. The longer we hesitated, the more >

Re: [R-pkg-devel] Conditionally register method with generic in other package

2017-12-07 Thread Hadley Wickham
On Thu, Dec 7, 2017 at 2:44 AM, Bill Denney wrote: > >> On Dec 6, 2017, at 07:45, Joshua Ulrich wrote: >> >> To avoid excessive dependencies, I would like to only register >> foo.bar() if package A is installed at the time package B is >> installed. If package A is installed after package B, the

Re: [R-pkg-devel] Conditionally register method with generic in other package

2017-12-07 Thread Hadley Wickham
We use this technique in a couple of places: https://github.com/tidyverse/dbplyr/blob/6be777d8b23d588f19c98de52f4e58f16c2ef67e/R/zzz.R Basic idea is to call registerS3method() manually, when needed - it's just a little tricky because the suggested package may be loaded either before or after your

Re: [R-pkg-devel] Fwd: R CMD check and strange ## Not run strings

2017-12-19 Thread Hadley Wickham
> This actually is not about Rd format. Indeed, you are using 'roxygen' > syntax. This is unrelated to roxygen. \dontrun{} is Rd formatting. Hadley -- http://hadley.nz __ R-package-devel@r-project.org mailing list https://stat.ethz.ch/mailman/list

Re: [R-pkg-devel] winUCRT failures

2021-04-25 Thread Hadley Wickham
> One additional thought: > > If the testing package (i.e. testthat in this case) had been available > but other suggested packages were not, it would be worth running tests > with just testthat present: that might be why you called the decision > defensible. I'd agree with that. > > However, it'

Re: [R-pkg-devel] winUCRT failures

2021-04-25 Thread Hadley Wickham
Who is responsible for the winUCRT checks? Perhaps that person could provide us with a list of root causes behind the testthat failures, and we could look into resolving them. Hadley On Sun, Apr 25, 2021 at 7:50 AM Duncan Murdoch wrote: > > The current CRAN release of rgl fails on winUCRT becaus

Re: [R-pkg-devel] best LICENSE practices: AGPL-3 + LaTeX Project Public License

2021-07-15 Thread Hadley Wickham
On Wed, Jul 14, 2021 at 9:11 AM Ben Bolker wrote: > > >In the process of trying to get a package to build successfully on > r-hub's Fedora platform, I had to add a whole bunch of LaTeX .sty files > to the vignette directory. One of these was collectbox.sty, which > triggers the NOTE > > --- >

Re: [R-pkg-devel] [External] Re: What is a "retired"package?

2021-09-21 Thread Hadley Wickham
> But for the broader question, Jeff is saying that there really are 700 > packages that are in potential trouble! I think that's rather an overstatement of the problem — there's nothing wrong with plyr; it's just no longer under active development. If anything, plyr is one of the safest packages

Re: [R-pkg-devel] [External] Re: What is a "retired"package?

2021-09-21 Thread Hadley Wickham
d base::apply instead of plyr::aaply. > > -Original Message- > From: Hadley Wickham > Sent: Tuesday, September 21, 2021 11:48 AM > To: Lenth, Russell V > Cc: Jeff Newmiller ; r-package-devel@r-project.org > Subject: Re: [R-pkg-devel] [External] Re: What is a "reti

Re: [R-pkg-devel] Non-ASCII and CRAN Checks

2022-09-20 Thread Hadley Wickham
In my experience this NOTE does not interfere with CRAN submission and you can ignore it. Hadley On Monday, September 19, 2022, Igor L wrote: > Hello everybody, > > I'm testing my package with the devtools::check() function and I got a > warning about found non-ASCII strings. > > These characte

Re: [R-pkg-devel] CRAN package isoband and its reverse dependencies

2022-10-05 Thread Hadley Wickham
Yes, we will make sure that this is fixed ASAP. There is no need to worry. Hadley On Wed, Oct 5, 2022 at 7:32 AM John Harrold wrote: > > Howdy Folks, > > I got a message from CRAN today telling me that I have a strong reverse > dependency on the isoband package. But I'm not alone! It look like m

Re: [R-pkg-devel] Issue handling datetimes: possible differences between computers

2022-10-10 Thread Hadley Wickham
On Sun, Oct 9, 2022 at 9:31 PM Jeff Newmiller wrote: > > ... which is why tidyverse functions and Python datetime handling irk me so > much. > > Is tidyverse time handling intrinsically broken? They have a standard > practice of reading time as UTC and then using force_tz to fix the "mistake".

Re: [R-pkg-devel] Best way forward on a CRAN archived package

2022-10-16 Thread Hadley Wickham
I’d suggest resubmitting, after ensuring that R CMD check runs without any notes, warnings, or errors. Hadley On Monday, October 10, 2022, Diego Hernangómez Herrero < diego.hernangomezherr...@gmail.com> wrote: > Hi: > > I have some doubts on how to proceed in this case. I am the developer of >

Re: [R-pkg-devel] Changing R Package Maintainer

2023-04-07 Thread Hadley Wickham
Just submit your package, and you'll get an automated email to the old address. Hadley On Sat, Apr 8, 2023 at 7:14 AM Andrew Simmons wrote: > > Hi, > > > I'm changing my name and my email address. I've got an update I'd like to > submit to CRAN, I've changed my name and email in my DESCRIPTION.

Re: [R-pkg-devel] Acknowledging small functions from another package

2023-05-04 Thread Hadley Wickham
IMO those functions are so small that you don't need to call them out in your DESCRIPTION. Just note in a nearby comment where they came from. Hadley On Thu, May 4, 2023 at 3:21 AM David Hugh-Jones wrote: > > Hi, > > One of my packages copy-pasted some small functions (stuff like `%||%` for > is

Re: [R-pkg-devel] Unfortunate function name generic.something

2023-05-08 Thread Hadley Wickham
If it's internal only, you could change the name to levels_no()? Hadley On Mon, May 8, 2023 at 7:28 AM Ulrike Groemping wrote: > > Thanks, Duncan. I appreciate the view that levels.no acts as an S3 > method for the generic levels, if an object of class "no" is handed to > it. However, as the func

Re: [R-pkg-devel] Feedback on "Using Rust in CRAN packages"

2023-07-13 Thread Hadley Wickham
> > it is not expected to use cargo to resolve them from random (possibly > inaccessible) places > > Yes, I agree with you. So, I suggested the possibility of forbidding the > Git dependency. Or, do you call crates.io, Rust's official repository, > "random places"? If CRAN cannot trust even the off

Re: [R-pkg-devel] Feedback on "Using Rust in CRAN packages"

2023-07-13 Thread Hadley Wickham
> > If CRAN cannot trust even the official one of Rust, why does CRAN have Rust > > at all? > > > > I don't see the connection - if you downloaded something in the past it > doesn't mean you will be able to do so in the future. And CRAN has Rust > because it sounded like a good idea to allow pac

Re: [R-pkg-devel] LICENSE file in an R package for CRAN submission

2023-08-09 Thread Hadley Wickham
If you're using one of the licenses supported by usethis (https://usethis.r-lib.org/reference/licenses.html), you can just call the appropriate function and it will do all the setup required to be both CRAN and GitHub compatible. Hadley On Wed, Aug 9, 2023 at 10:10 AM Emanuele Cordano wrote: > >

Re: [R-pkg-devel] How to fix Archived Package Rpdb?

2023-09-08 Thread Hadley Wickham
On Fri, Sep 8, 2023 at 6:02 AM Leonard Mada via R-package-devel wrote: > > Dear Members, > > I would like to reanimate the archived package Rpdb: > https://cran.r-project.org/web/packages/Rpdb/index.html > > 1.) I have tried to contact the original author by email, but got no > response. > > 2.) N

Re: [R-pkg-devel] Discrepancy between R CMD check results and usethis::use_cran_comments

2023-09-26 Thread Hadley Wickham
On Tue, Sep 26, 2023 at 2:01 AM Leonard Mada via R-package-devel wrote: > > Dear List-Members, > > There are no errors/warnings/notes when I run the check: > > ── R CMD check results > Rpdb 2.3.3 > Duration: 2m 50.1s > > 0 errors ✔ | 0

Re: [R-pkg-devel] how to use pkgdown::build_site() with a project using S7 with a specialized plot()?

2024-01-03 Thread Hadley Wickham
This bug is fixed in the dev version (I don’t remember off the top of my head in which of pkgdown and roxygen2 you need but it might be both). I’m planning CRAN updates for both in the near future. Hadley On Thursday, January 4, 2024, Daniel Kelley wrote: > # Question > > Is there an online exa

Re: [R-pkg-devel] how to use pkgdown::build_site() with a project using S7 with a specialized plot()?

2024-01-03 Thread Hadley Wickham
nstalled(package, "as it's used in the reference > index.") > 35. base::stop(cnd) > 36. (function (cnd) … > 37. cli::cli_abort(message, location = i, name = name, parent = cnd, … > 38. | rlang::abort(message, ..., call = call, use_cli_format = TRUE, … > 39. | rlang:

Re: [R-pkg-devel] @doctype is deprecated. need help for r package documentation

2024-03-07 Thread Hadley Wickham
Do you have a pointer to the roxygen2 comments that you're using? Hadley On Thu, Mar 7, 2024 at 5:38 AM Ruff, Sergej wrote: > Hello, > > I need help with a package I am currently developing called bootGSEA. > I noticed that when I try ‘?bootGSEA’ it goes to the help page in R > itself but not t

Re: [R-pkg-devel] cpp11 and "non-API calls to R"

2024-07-09 Thread Hadley Wickham
Hi Mark, The cpp11 package is now maintained by Davis Vaughan and we will release a new version once the C API status is a little cleaner. This warnings generated from cpp11 will not interfere with your ability to submit your package; just let CRAN know that the warnings are coming from cpp11. Ha

Re: [R-pkg-devel] Windows binaries

2018-02-14 Thread Hadley Wickham
devtools::build_win() has an unfortunate name - it's actually more about checking your package on windows than building a package (and in the development version we've renamed to check_win()). However, fortunately, once your package has been accepted on CRAN, you don't need to worry about building

Re: [R-pkg-devel] R package CRAN check error

2018-02-16 Thread Hadley Wickham
Or even better, devtools::release(): it walks you through a checklist of activities designed to make your submission as successful as possible. Hadley On Fri, Feb 16, 2018 at 2:46 PM, Georgi Boshnakov wrote: > You get a source package suitable for submission by running `R CMD check > namepackag

Re: [R-pkg-devel] GPL + other licence

2018-04-10 Thread Hadley Wickham
On Tue, Apr 10, 2018 at 2:45 AM, Kasper Daniel Hansen wrote: > There are 3 solutions. (1) You (get permission) to change the library to > GPL. (2) You get permission to change the license of the R code to > whatever license the library is released under. (3) you split the package. For complete

Re: [R-pkg-devel] CRAN pretest archived because of 2 NOTEs

2018-04-18 Thread Hadley Wickham
For the purposes of CRAN submission, you should basically treat every NOTE as an ERROR. Hadley On Wed, Apr 18, 2018 at 3:36 AM, Gertjan van den Burg wrote: > While waiting to get this message posted to the list, I've solved the > problem by copying the stdlib rand() and srand() functions into my

Re: [R-pkg-devel] vignette via devtools: sometimes found, sometimes not (same package)

2018-04-24 Thread Hadley Wickham
On Fri, Apr 20, 2018 at 12:59 AM, Martin Maechler wrote: >> Thierry Onkelinx >> on Fri, 20 Apr 2018 09:14:44 +0200 writes: > > > Dear Alex, Another idea is to use pkgdown > > (http://pkgdown.r-lib.org) to convert all the > > documentation of your package (include the vigne

Re: [R-pkg-devel] Errors from Suggests or Enhances not in mainstream repositories

2018-07-03 Thread Hadley Wickham
On Tue, Jul 3, 2018 at 5:57 AM, Duncan Murdoch wrote: > On 02/07/2018 6:13 PM, Ben Bolker wrote: >> >> I got something similar. I have a few thoughts: >> >> (1) you should use "if (require(citrus)) { ... }" in your examples; >> "Suggests" and "Enhances" packages are supposed to be *optional*, i.

Re: [R-pkg-devel] Weird error on CRAN linux check

2018-07-04 Thread Hadley Wickham
I don't think it's related to the error, but you shouldn't be exporting this: export("align<-.huxtable") You should generally only export the method. Hadley On Wed, Jul 4, 2018 at 9:00 AM, David Hugh-Jones wrote: > Hi all, > > The following shows an error for my package: > https://www.r-projec

Re: [R-pkg-devel] CRAN incoming queue closed from Sep 1 to Sep 9

2018-08-14 Thread Hadley Wickham
Does this include automatically (bot) accepted submissions? Hadley On Tue, Aug 14, 2018 at 8:07 AM Uwe Ligges wrote: > > Dear package developers, > > the CRAN incoming queue will be closed from Sep 1 to Sep 9. Hence > package submissions are only possible before and after that period. > > Best, >

Re: [R-pkg-devel] Package vignette index

2018-08-14 Thread Hadley Wickham
I'm pretty certain that seeing this warning indicates that your `.Rbuildignore` is ignoring something that it shouldn't. You certainly shouldn't be creating this file by hand. Hadley On Mon, Aug 13, 2018 at 12:44 PM Paul Johnson wrote: > > Can you advise me about this warning in package check: >

Re: [R-pkg-devel] Submission to CRAN when package needs personal data (API key)

2018-09-06 Thread Hadley Wickham
On Wed, Sep 5, 2018 at 3:03 PM Duncan Murdoch wrote: > > On 05/09/2018 2:20 PM, Henrik Bengtsson wrote: > > I take a complementary approach; I condition on, my home-made, > > R_TEST_ALL variable. Effectively, I do: > > > > if (as.logical(Sys.getenv("R_TEST_ALL", "FALSE"))) { > > ... > > } > >

Re: [R-pkg-devel] Submission to CRAN when package needs personal data (API key)

2018-09-07 Thread Hadley Wickham
On Fri, Sep 7, 2018 at 9:13 AM Iñaki Ucar wrote: > > El vie., 7 sept. 2018 a las 16:03, Ralf Stubner > () escribió: > > > > On 07.09.2018 15:52, Iñaki Ucar wrote: > > > For the record, this is what the testthat paper in the R Journal says: > > > > > > "[...] I recommend storing your tests in inst/

Re: [R-pkg-devel] stringi update

2018-09-15 Thread Hadley Wickham
Looking at the primary CRAN site: https://cran.r-project.org/web/packages/stringi/index.html, you can see that the windows binary is still at 1.1.7, suggesting that there's some build failure. You can see exactly what that is on the CRAN check page: https://cran.r-project.org/web/checks/check_resul

Re: [R-pkg-devel] Extending/adding to an R6 class from another package: qns

2018-10-19 Thread Hadley Wickham
> AzureRMR: the "base" package, provides a number of R6 classes > AzureVM: a "child" package that extends classes from AzureRMR with extra > functionality related to virtual machines > AzureStor: another child package that extends classes from AzureRMR, this > time for storage accounts > Etc. > >

Re: [R-pkg-devel] R CMD check halts on @example in shiny::runApp()

2018-11-19 Thread Hadley Wickham
On Mon, Nov 19, 2018 at 8:14 AM L Rutter wrote: > > Hello all: > > R CMD check does not complete for my package and halts at the line "* > checking examples ..." I suspect the halting is due to @examples > hanging in certain R/foo.R files, where shiny::runApp() is called for > a shiny app located

Re: [R-pkg-devel] creating a link to a vignette in a .Rd file

2018-11-19 Thread Hadley Wickham
On Mon, Nov 19, 2018 at 4:49 PM Fox, John wrote: > > Dear r-package-devel list members, > > I'd like to create a link to a package vignette from a help file in the same > package, for example to the "partial-residuals" vignette in the effects > package from effect.Rd. I'm able to generate a URL

Re: [R-pkg-devel] creating a link to a vignette in a .Rd file

2018-11-20 Thread Hadley Wickham
> None of these solutions seem perfect to me. I think that my suggestion is the > most natural, but as you point out it won’t work in all contexts. Perhaps the > safest approach is to give the vignette() command in the text of the help > file, one of your suggestions. If you do that, and you us

[R-pkg-devel] New CRAN internet policy

2018-12-06 Thread Hadley Wickham
Hi all, I'd love to get some clarification on what the new internet policy means for packages like httr: > Packages which use Internet resources should fail gracefully with an > informative > message if the resource is not available (and not give a check warning nor > error). It's not clear wh

Re: [R-pkg-devel] New CRAN internet policy

2018-12-06 Thread Hadley Wickham
point me to where all the policies such as this one are posted. > This may affect a package I have, and one problem I have is different people > have differing ideas of what defines a "graceful" exit. > > Thanks, > > Roy > > > > On Dec 6, 2018, at 8:22 AM

Re: [R-pkg-devel] Suggested package relies on recent R

2018-12-08 Thread Hadley Wickham
Can you just set _R_CHECK_FORCE_SUGGESTS_=false? env: global: # don't treat missing suggested packages as error - _R_CHECK_FORCE_SUGGESTS_=false I am reasonably certain that is what CRAN uses. Hadley On Fri, Dec 7, 2018 at 9:11 AM David Hugh-Jones wrote: > > Hi, > > My package Suggests a

Re: [R-pkg-devel] Package update submission to CRAN fails on pretest

2018-12-08 Thread Hadley Wickham
You might try reinstalling devtools and dependencies - there was unfortunately a brief combination of versions that lead to build() failing to overwrite existing files. Hadley On Fri, Dec 7, 2018 at 10:46 AM Wolfgang Lenhard wrote: > > Many thanks for the remark. It seems, it has something to do

Re: [R-pkg-devel] New CRAN internet policy

2018-12-21 Thread Hadley Wickham
On Fri, Dec 7, 2018 at 2:48 AM Martin Maechler wrote: > > >>>>> Hadley Wickham > >>>>> on Thu, 6 Dec 2018 10:22:47 -0600 writes: > > > Hi all, > > I'd love to get some clarification on what the new internet policy >

Re: [R-pkg-devel] submitting to github

2019-01-25 Thread Hadley Wickham
No one else has mentioned it on the thread, so I'd highly recommend https://happygitwithr.com — it's a guide to git + github specifically written for R users, and covers many of the common problems people have when getting set up. Hadley On Fri, Jan 25, 2019 at 2:51 AM Troels Ring wrote: > > Dea

Re: [R-pkg-devel] Checking for future file timestamps - warning with worldclockapi HTTP status 403 Site Disabled

2019-03-07 Thread Hadley Wickham
It appears that the code was added by BDR on 2 Sep 2018: https://github.com/wch/r-source/commit/d839b1e04e173f90b51ad809ef0bdb18095abe6f I assume we are seeing failing R CMD check results because http://worldclockapi.com/api/json/utc/now has recently died. It would be appreciated if someone from

Re: [R-pkg-devel] Checking for future file timestamps - warning with worldclockapi HTTP status 403 Site Disabled

2019-03-07 Thread Hadley Wickham
As of ~7 hours ago, the warning is suppressed: https://github.com/wch/r-source/commit/31ee14c620eb1b939acd322f3b5617f998aab8e8 (But the service still doesn't work) Hadley On Thu, Mar 7, 2019 at 11:03 AM Hadley Wickham wrote: > > It appears that the code was added by BDR on 2 Sep 20

[R-pkg-devel] CRAN student assistants

2019-05-14 Thread Hadley Wickham
Hi all, Several people on my team have received responses to their CRAN submissions from new members of the CRAN team who appear to be student assistants (judging from their job titles: "Studentischer administrativer Mitarbeiter"). From the outside, they appear to be exercising editorial control[^

Re: [R-pkg-devel] CRAN student assistants

2019-05-16 Thread Hadley Wickham
how many there are, and what backgrounds they have. Similarly, I don't want to know how much they are paid, just whether or not they are volunteers or employees. Hadley On Tue, May 14, 2019 at 10:23 AM Hadley Wickham wrote: > > Hi all, > > Several people on my team have received response

Re: [R-pkg-devel] (Not) Reporting minimum R version in DESCRIPTION Depends Field

2019-07-26 Thread Hadley Wickham
I no longer believe this to be good advice - I think you should only declare a specific dependency if you want to strongly assert that your package works with those versions. For example, all tidyverse versions depend on R 3.2 and later, because we test on all those versions. Hadley On Friday, Ju

  1   2   >