[Bioc-devel] Fwd: philr problems reported in the Multiple platform build/check report for BioC 3.17

2023-04-07 Thread Justin Silverman
Hello helpful list. I have repeatedly got the following email over the past few weeks. At first I thought it was a false positive as the error in question was fixed a while back. But I keep receiving this email. I have asked a few colleagues to try to test out the package and run R CMD CHECK an

Re: [Bioc-devel] Fwd: philr problems reported in the Multiple platform build/check report for BioC 3.17

2023-04-07 Thread Oleksii Nikolaienko
Hi, looks like the latest commit to the relevant code in package mia has changed the arguments of makePhyloseqFromTreeSummarizedExperiment (makeP

Re: [Bioc-devel] Fwd: philr problems reported in the Multiple platform build/check report for BioC 3.17

2023-04-07 Thread Robert M. Flight
Hi Justin, I pulled the Bioconductor devel docker image ( https://www.bioconductor.org/help/docker/#intro), fired it up, and installed your github version of philr. And run the vignette. And I can reproduce the error: > Error in philr.data.frame(otu.table, tree = tree, ...) : > > Zero values mus

Re: [Bioc-devel] Fwd: philr problems reported in the Multiple platform build/check report for BioC 3.17

2023-04-07 Thread Justin Silverman
Thank you both so much! @Robert, thank you for introducing me to the docker containers, that was super helpful. @Oleksii, you were spot on in terms of the problem. Thank you! Should be fixed. Best, Justin ___ Bioc-devel@r-project.org mailing list ht