Closed svteichman closed 8 months ago
Attention: 16 lines
in your changes are missing coverage. Please review.
Comparison is base (
779de75
) 94.12% compared to head (cfc6ba6
) 93.36%.:exclamation: Current head cfc6ba6 differs from pull request most recent head cc47207. Consider uploading reports for the commit cc47207 to get more accurate results
Files | Patch % | Lines |
---|---|---|
R/contrastsTest.R | 35.71% | 9 Missing :warning: |
R/differentialTest.R | 87.50% | 3 Missing :warning: |
R/clean_taxa_names.R | 85.71% | 1 Missing :warning: |
R/convert_phylo.R | 87.50% | 1 Missing :warning: |
R/otu_to_taxonomy.R | 85.71% | 1 Missing :warning: |
R/warn_phyloseq.R | 0.00% | 1 Missing :warning: |
:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
This pull request does several things:
-it adds new data, specifically the sample data, otu tables, and in some cases taxonomy tables, that make up phyloseq objects used for tests and the vignette -it removes all phyloseq objects from the data folder of the package -it updates all examples so that they run without phyloseq and without phyloseq data objects (for
bbdml()
,differentialTest()
, andcontrastsTest()
, there are examples using phyloseq wrapped in\donotrun{}
tags) -it takes all tests that use phyloseq and puts them into a single testing file, where the phyloseq objects are created from the dataframes stored in the package -it creates a new vignette that doesn't use phyloseq (with most of the same analyses as the original vignette that does use phyloseq)