lshep

Results 575 comments of lshep

- [ ] Please include package level man page so a naive users that does `?MultiModalGraphics` would have starting information - [ ] It would be good to provide a...

``` R CMD check MultiModalGraphics_0.99.0.tar.gz * using log directory '/home/lorikern/PkgReview/PreReview/MultiModalGraphics.Rcheck' * using R version 4.4.0 Patched (2024-04-30 r86503) * using platform: x86_64-pc-linux-gnu * R was compiled by gcc (Ubuntu 11.4.0-1ubuntu1~22.04)...

``` > sessionInfo() R version 4.4.0 Patched (2024-04-30 r86503) Platform: x86_64-pc-linux-gnu Running under: Ubuntu 22.04.4 LTS Matrix products: default BLAS: /home/lorikern/R-Installs/bin/R-4-4-branch/lib/libRblas.so LAPACK: /usr/lib/x86_64-linux-gnu/lapack/liblapack.so.3.10.0 locale: [1] LC_CTYPE=en_US.UTF-8 LC_NUMERIC=C [3] LC_TIME=en_US.UTF-8 LC_COLLATE=en_US.UTF-8...

I still don't see any interop with common _Bioconductor_ classes. See [common classes](http://contributions.bioconductor.org/important-bioconductor-package-development-features.html#commonclass). It would be great to show how this could interact with say a SummarizedExperiment?

A few things: You have 2 R scripts in the top level of your repository. Please remove. When I try to R CMD build the package I'm getting an ERROR...

I'm currently unable to process as there is an issue with large data in the package ``` Error in .precheck_filesize(wd) : files larger than 5Mb: /inst/extdata/scatterexample.csv ```

set.seed should not be used within R functions themselves but called outside the function. If there is some strong reasoning for using within R functions minimally it should be controlled...

We are updating our builders; I believe this to be an error on our end that we are investigating

I will kick off a new build for you once we have completed updates