Re: [R-pkg-devel] A function in one of my package is now a method in base R

2024-08-03 Thread Shu Fai Cheung
uot; and "y" in base::sort_by() and so the matching makes sense. Regards, Shu Fai Cheung On Sat, Aug 3, 2024 at 7:13 PM Deepayan Sarkar wrote: > > I haven't thought about this carefully, but shouldn't this mostly work? > > sort_by.est_table <- function(x, y = c(

[R-pkg-devel] A function in one of my package is now a method in base R

2024-08-02 Thread Shu Fai Cheung
not unusual across packages and so users need to learn how to solve this problem anyway. Nevertheless, if possible, I would like to solve the conflict internally such that users do not need to do anything. Regards, Shu Fai Cheung __ R-package-devel@r-project.org mailing list https://stat.ethz.ch/mailman/listinfo/r-package-devel

[R-pkg-devel] callr and CRAN policy on the max number of cores

2024-05-22 Thread Shu Fai Cheung
Hil, I am exploring the use of callr in a package. I know that for packages that do parallel computing, we should not use more than two cores in examples, tests, etc. when being checked on CRAN. I believe I can use callr in tests and examples. How should I use callr in compliance with CRAN policy

Re: [R-pkg-devel] PkgA imports PkgB, and PkgB suggests PkgA?

2023-11-04 Thread Shu Fai Cheung
Many many thanks for the clarification, which is very clear! The case of testthat is a very good example, as many packages suggest it. Regards, Shu Fai On Sat, Nov 4, 2023 at 4:41 PM Iñaki Ucar wrote: > > > > El sáb., 4 nov. 2023 5:43, Shu Fai Cheung escribió: >> >>

[R-pkg-devel] PkgA imports PkgB, and PkgB suggests PkgA?

2023-11-03 Thread Shu Fai Cheung
Hi All, I vaguely recall that, on CRAN, if PkgA imports PkgB, then PkgB cannot import PkgA. (Please correct me if I am wrong.) How about this? PkgA imports PkgB (because PkgA has some helper functions for using PkgB) PkgB suggests PkgA (because some vignettes or examples in PkgB use those helper

Re: [R-pkg-devel] Suppressing long-running vignette code in CRAN submission

2023-10-17 Thread Shu Fai Cheung
Please pardon me if I suggest something unrelated below. Many experts have made suggestions that I would also like to consider because I also have a similar issue with some packages. This is an approach I found, for Rmarkdown vignettes: https://www.kloppenborg.ca/2021/06/long-running-vignettes/

Re: [R-pkg-devel] Checking the number of cores used

2023-09-20 Thread Shu Fai Cheung
like to see if I overlooked anything. Regards, Shu Fai On Tue, Sep 19, 2023 at 5:02 PM Uwe Ligges wrote: > > > > On 18.09.2023 16:10, Shu Fai Cheung wrote: > > Hi All, > > > > I know we should not use more than 2 cores in tests, vignettes, etc. I > > encountered and s

Re: [R-pkg-devel] Checking the number of cores used

2023-09-20 Thread Shu Fai Cheung
7;CheckExEnv'),"\n") Time elapsed: 10.963 0.161 13.589 0.302 0.081 Regards, Shu Fai On Tue, Sep 19, 2023 at 5:59 PM Duncan Murdoch wrote: > > On 18/09/2023 10:10 a.m., Shu Fai Cheung wrote: > > Hi All, > > > > I know we should not use more than 2 cores i

[R-pkg-devel] Checking the number of cores used

2023-09-18 Thread Shu Fai Cheung
Hi All, I know we should not use more than 2 cores in tests, vignettes, etc. I encountered and solved this issue before. However, I still committed this mistake in a new package and would like find out where the cause is. I have a package that already has parallel processing disabled by default a