I tried running your tests (checked out from github repo at
commit e580d52d7242693b79ec6b8483d68a65d6fb9e02 (HEAD -> master,
origin/master, origin/HEAD)
Author: Matthias Zytnicki <matthias.zytni...@inra.fr>
Date: Mon Mar 6 21:35:41 2023 +0100
version bump
and ran into some serious issues -- a very long stream of
> test_that("Running implemented constructor", {
+ object <- srnadiffExp(bamFiles, sampleInfo, annotReg)
+ expect_is(object, "srnadiffExp")
+ .... [TRUNCATED]
── Warning: Running implemented constructor
────────────────────────────────────────────────────────────────────────────
subscript out of bounds (index 1 >= vector size 1)
Backtrace:
▆
1. └─srnadiff::srnadiffExp(bamFiles, sampleInfo, annotReg) at
test_AllClass.R:21:5
2. └─srnadiff:::computeNormFactors(object@coverages)
3. └─srnadiff:::rcpp_normalization(cvg, librarySize)
── Warning: Running implemented constructor
────────────────────────────────────────────────────────────────────────────
subscript out of bounds (index 2 >= vector size 1)
Backtrace:
▆
1. └─srnadiff::srnadiffExp(bamFiles, sampleInfo, annotReg) at
test_AllClass.R:21:5
2. └─srnadiff:::computeNormFactors(object@coverages)
3. └─srnadiff:::rcpp_normalization(cvg, librarySize)
Admittedly this is not the right environment to reproduce what you are
seeing but
perhaps it is somewhat helpful. Have you tried running the tests in
the bioc BBS container?
see
https://github.com/bioconductor/bioconductor_salt/pkgs/container/bioconductor_salt
R version 4.4.2 (2024-10-31)
Platform: aarch64-apple-darwin20
Running under: macOS Sequoia 15.2
Matrix products: default
BLAS:
/Library/Frameworks/R.framework/Versions/4.4-arm64/Resources/lib/libRblas.0.dylib
LAPACK:
/Library/Frameworks/R.framework/Versions/4.4-arm64/Resources/lib/libRlapack.dylib;
LAPACK version 3.12.0
locale:
[1] en_US.UTF-8/en_US.UTF-8/en_US.UTF-8/C/en_US.UTF-8/en_US.UTF-8
time zone: America/New_York
tzcode source: internal
attached base packages:
[1] stats graphics grDevices utils datasets methods base
other attached packages:
[1] srnadiff_1.19.2 testthat_3.2.2 rmarkdown_2.29
loaded via a namespace (and not attached):
[1] later_1.4.1 BiocIO_1.16.0
[3] bitops_1.0-9 filelock_1.0.3
[5] tibble_3.2.1 XML_3.99-0.17
[7] rpart_4.1.23 startup_0.23.0
On Mon, Dec 2, 2024 at 3:13 PM Matthias Zytnicki
<matthias.zytni...@inrae.fr> wrote:
>
> Dear everyone,
>
>
> I have difficulties trying to decrease the time spent to check my package,
> srnadiff (for reproducibility, you can get the GitHub repo from
> https://github.com/mzytnicki/srnadiff, on the devel branch).
>
>
> I tried to reduce the tests to 2 files (for reproducibility:
> test_annotation.R and test_pvalues.R).
>
>
> When I use devtools::check(), the "checking tests" step takes 29 minutes.
> With all the files, it takes more than one hour. Same goes with R CMD check
> on the built package. On the BioC check platform, it reaches the time limit.
>
>
> When I use devtools::test(), the same tests takes 8 seconds.
>
>
> Now, on the computer of my colleague, trying to reproduce the BioC
> environment variables, the whole devtools::check takes a few minutes.
>
>
> I am puzzled by the discrepancy, and would like to reduce this time.
>
> Does anyone have a suggestion about how I could proceed?
>
>
> I used profvis with the command p <- profvis::profvis({foo <-
> devtools::check()})
>
> It seems that most of the time is spent on ".Call" (with no other
> sub-function called), and I cannot interpret this.
>
>
> I thank you very much for your help.
>
>
> Best regards,
>
>
> Matthias.
>
> [[alternative HTML version deleted]]
>
> _______________________________________________
> Bioc-devel@r-project.org mailing list
> https://stat.ethz.ch/mailman/listinfo/bioc-devel
--
The information in this email is intended only for the p...{{dropped:12}}
_______________________________________________
Bioc-devel@r-project.org mailing list
https://stat.ethz.ch/mailman/listinfo/bioc-devel
