Bioconductor / Contributions

Contribute Packages to Bioconductor
134 stars 33 forks source link

decontX #3020

Closed yuan-yin-truly closed 11 months ago

yuan-yin-truly commented 1 year ago

Update the following URL to point to the GitHub repository of the package you wish to submit to Bioconductor

Confirm the following by editing each check box to '[x]'

I am familiar with the essential aspects of Bioconductor software management, including:

For questions/help about the submission process, including questions about the output of the automatic reports generated by the SPB (Single Package Builder), please use the #package-submission channel of our Community Slack. Follow the link on the home page of the Bioconductor website to sign up.

bioc-issue-bot commented 1 year ago

Hi @yuan-yin-truly

Thanks for submitting your package. We are taking a quick look at it and you will hear back from us soon.

The DESCRIPTION file for this package is:

Package: decontX
Title: Decontamination of single cell genomics data
Version: 0.99.0
Authors@R: 
    person("Yuan", "Yin", , "yuan_yin@outlook.com", role = c("aut", "cre"),
 comment = c(ORCID = "0000-0001-9261-6061"))
Description: This package contains implementation of DecontX (Yang et al. 2020),
    a decontamination algorithm for single-cell RNA-seq, and DecontPro (Yin et al. 
    2023), a decontamination algorithm for single cell protein expression data.
License: MIT + file LICENSE
Encoding: UTF-8
Roxygen: list(markdown = TRUE)
RoxygenNote: 7.2.3
Suggests: 
    BiocStyle,
    dplyr,
    knitr,
    rmarkdown,
    scran,
    SingleCellMultiModal,
    TENxPBMCData,
    testthat (>= 3.0.0)
Config/testthat/edition: 3
Imports:
    celda,
    dbscan,
    DelayedArray,
    ggplot2,
    Matrix (>= 1.5.3),
    MCMCprecision,
    methods,
    patchwork,
    plyr,
    Rcpp (>= 0.12.0),
    RcppParallel (>= 5.0.1),
    reshape2,
    rstan (>= 2.18.1),
    rstantools (>= 2.2.0),
    S4Vectors,
    scater,
    Seurat,
    SingleCellExperiment,
    SummarizedExperiment,
    withr
Biarch: true
Depends: 
    R (>= 3.4.0)
LinkingTo: 
    BH (>= 1.66.0),
    Rcpp (>= 0.12.0),
    RcppEigen (>= 0.3.3.3.0),
    RcppParallel (>= 5.0.1),
    rstan (>= 2.18.1),
    StanHeaders (>= 2.18.0)
SystemRequirements: GNU make
VignetteBuilder: knitr
biocViews: SingleCell, Bayesian
vjcitn commented 1 year ago

Do you know what is going on here?:

Quitting from lines 63-69 (decontX.Rmd) 
Error: processing vignette 'decontX.Rmd' failed with diagnostics:
no method or default for coercing "DelayedMatrix" to "SVT_SparseArray"
--- failed re-building ‘decontX.Rmd’
yuan-yin-truly commented 1 year ago

Hi @vjcitn, thank you for your help! I think it has to do with package TENxPBMCData not installed in your system. This package is currently listed under Suggests in the DESCRIPTION file. Could you try installing TENxPBMCData (https://bioconductor.org/packages/release/data/experiment/html/TENxPBMCData.html) and rebuild vignette?

bioc-issue-bot commented 1 year ago

A reviewer has been assigned to your package. Learn what to expect during the review process.

IMPORTANT: Please read this documentation for setting up remotes to push to git.bioconductor.org. It is required to push a version bump to git.bioconductor.org to trigger a new build.

Bioconductor utilized your github ssh-keys for git.bioconductor.org access. To manage keys and future access you may want to active your Bioconductor Git Credentials Account

bioc-issue-bot commented 1 year ago

Dear Package contributor,

This is the automated single package builder at bioconductor.org.

Your package has been built on the Bioconductor Build System.

On one or more platforms, the build results were: "ERROR". This may mean there is a problem with the package that you need to fix. Or it may mean that there is a problem with the build system itself.

Please see the build report for more details.

The following are build products from R CMD build on the Bioconductor Build System: macOS 12.6.5 Monterey: decontX_0.99.0.tar.gz Linux (Ubuntu 22.04.2 LTS): decontX_0.99.0.tar.gz

Links above active for 21 days.

Remember: if you submitted your package after July 7th, 2020, when making changes to your repository push to git@git.bioconductor.org:packages/decontX to trigger a new build. A quick tutorial for setting up remotes and pushing to upstream can be found here.

bioc-issue-bot commented 1 year ago

Received a valid push on git.bioconductor.org; starting a build for commit id: 81342e923b570e6b38295acf58dfecfad1ab5e35

bioc-issue-bot commented 1 year ago

Dear Package contributor,

This is the automated single package builder at bioconductor.org.

Your package has been built on the Bioconductor Build System.

On one or more platforms, the build results were: "WARNINGS". This may mean there is a problem with the package that you need to fix. Or it may mean that there is a problem with the build system itself.

Please see the build report for more details.

The following are build products from R CMD build on the Bioconductor Build System: Linux (Ubuntu 22.04.2 LTS): decontX_0.99.1.tar.gz

Links above active for 21 days.

Remember: if you submitted your package after July 7th, 2020, when making changes to your repository push to git@git.bioconductor.org:packages/decontX to trigger a new build. A quick tutorial for setting up remotes and pushing to upstream can be found here.

bioc-issue-bot commented 1 year ago

Received a valid push on git.bioconductor.org; starting a build for commit id: f8a4e7b1d2f98afc39203319be70ea83f0d729ae

bioc-issue-bot commented 1 year ago

Dear Package contributor,

This is the automated single package builder at bioconductor.org.

Your package has been built on the Bioconductor Build System.

On one or more platforms, the build results were: "WARNINGS". This may mean there is a problem with the package that you need to fix. Or it may mean that there is a problem with the build system itself.

Please see the build report for more details.

The following are build products from R CMD build on the Bioconductor Build System: Linux (Ubuntu 22.04.2 LTS): decontX_0.99.2.tar.gz

Links above active for 21 days.

Remember: if you submitted your package after July 7th, 2020, when making changes to your repository push to git@git.bioconductor.org:packages/decontX to trigger a new build. A quick tutorial for setting up remotes and pushing to upstream can be found here.

bioc-issue-bot commented 1 year ago

Received a valid push on git.bioconductor.org; starting a build for commit id: 32edae5b8c133f64e00e45715909f7e8e1e46dd6

bioc-issue-bot commented 1 year ago

Dear Package contributor,

This is the automated single package builder at bioconductor.org.

Your package has been built on the Bioconductor Build System.

Congratulations! The package built without errors or warnings on all platforms.

Please see the build report for more details.

The following are build products from R CMD build on the Bioconductor Build System: Linux (Ubuntu 22.04.2 LTS): decontX_0.99.3.tar.gz

Links above active for 21 days.

Remember: if you submitted your package after July 7th, 2020, when making changes to your repository push to git@git.bioconductor.org:packages/decontX to trigger a new build. A quick tutorial for setting up remotes and pushing to upstream can be found here.

jianhong commented 1 year ago

Package 'decontX' Review

Thank you for submitting your package to Bioconductor. The package passed check and build. It is in pretty good shape. And it is a complex package. I will leave comments for multiple times. But before my next comments, please try to answer the comments line by line when you are ready for a second review. Code: Note: please consider; Important: must be addressed.

The DESCRIPTION file

R code

yuan-yin-truly commented 12 months ago

Hi @jianhong, thank you for all the feedback! I have created and merged a pull request (https://github.com/campbio/decontX/pull/9) into the devel branch to address your feedback.

Summary:

Please feel free to let me know if you have any questions.

Again thank you very much!

Detailed Response:

Package 'decontX' Review

Thank you for submitting your package to Bioconductor. The package passed check and build. It is in pretty good shape. And it is a complex package. I will leave comments for multiple times. But before my next comments, please try to answer the comments line by line when you are ready for a second review. Code: Note: please consider; Important: must be addressed.

The DESCRIPTION file

  • [x] Important: R version should be no less than 4.3
  • [x] Important: At least three sentences in Description.

The NAMESPACE file

  • [ ] Note: Function names use camelCase or snake_case and do not include ..

    • in line 3 export("decontXcounts<-")
    • in line 13 exportMethods("decontXcounts<-")

General package development

  • [ ] FYI: I am not sure Seurat will fix this issue or not. This is not related to this package. The legacy packages maptools, rgdal, and rgeos, underpinning the sp package, which was just loaded, will retire in October 2023. Please refer to R-spatial evolution reports for details, especially https://r-spatial.org/r/2023/05/15/evolution4.html. It may be desirable to make the sf package available; package maintainers should consider adding sf to Suggests:. The sp package is now running under evolution status 2 (status 2 uses the sf package in place of rgdal)

R code

  • [ ] NOTE: no direct slot access with @ or slot() - accessors implemented and used.

    • In file R/stan_helpers.R:

    • at line 28 found ' val <- stan_vb_output@sim$est'

  • [ ] Important: vapply instead of sapply.

    • In file R/stanmodels.R:

    • at line 10 found 'stanmodels <- sapply(stanmodels, function(model_name) {'

  • [x] Important: No paste in message(), message, stop

    • In file R/celda_functions.R:

    • at line 19 found ' message(paste(..., sep = sep))'

  • [ ] Note: Will it better to use writeLines instead of cat?

    • In file R/celda_functions.R:

    • at line 14 found ' cat(paste(..., "\n", sep = sep),'

  • [ ] NOTE: :: is not suggested in source code unless you can make sure all the packages are imported. Some people think it is better to keep ::. However please note that you need to manully double check the import items when you make any change in the DESCRIPTION file during development. My recommendation is to remove one or two repeats to force the dependency check.
  • [x] NOTE: 1:n is not suggested in source code. Use seq_along or seq.int instead.

    • In file R/plot_decontPro.R:

    • at line 53 found ' breaks = c(1, 5, 10 ^ (1:4))) +'

    • at line 162 found ' breaks = c(1, 5, 10 ^ (1:4))) +'

  • [ ] NOTE: Vectorize: for loops present, try to replace them by *apply funcitons.

    • In file R/celda_functions.R:

    • at line 111 found ' for (i in seq_along(features)) {'

    • In file R/decon.R:

    • at line 184 found ' for (i in batchIndex) {'

    • at line 430 found ' for (bat in batchIndex) {'

    • at line 1340 found ' for (i in seq(K)) {'

    • at line 1343 found ' for (j in setdiff(seq(K), i)) {'

    • at line 1359 found ' for (i in seq(K)) {'

    • In file R/plot_decontPro.R:

    • at line 38 found ' for (i in seq_along(features)) {'

    • at line 140 found ' for (i in seq_along(features)) {'

    • In file R/plot_decontx.R:

    • at line 192 found ' for (i in seq_along(assayName)) {'

    • at line 365 found ' for (i in seq_along(assayName)) {'

    • at line 506 found ' for (i in seq_along(groupClusters)) {'

  • [x] Important: Remove unused code.

    • In file R/decon.R:

    • at line 654 found ' # nG <- nrow(counts)'

    • at line 851 found ' # ll = sum( t(counts) log( (1-conP )geneDist[z,] + conP * conDist[z, ] +'

    • at line 852 found ' # 1e-20 ) ) # when dist_mat are K x G matrices'

    • at line 864 found ' # ll <- sum(t(counts) log(theta t(cellDist) +'

    • at line 865 found ' # (1 - theta) * t(bgDist) + 1e-20))'

  • [ ] Important: Please consider to add drop=FALSE to avoid the reduction of dimension for matrices and arrays. Ignore this if using datatable.

    • In file R/celda_functions.R:

    • at line 97 found ' search <- SummarizedExperiment::rowData(x)[, by]'

    • at line 102 found ' search <- x[, by]'

    • In file R/decon.R:

    • at line 189 found ' tempUMAP[result$runParams$batch == i, ] <- result$estimates[[i]]$UMAP'

    • at line 452 found ' countsBat <- counts[, batch == bat]'

    • at line 453 found ' bgBat <- countsBackground[, batchBackground == bat]'

    • at line 540 found ' estRmat <- estRmat[, !(allCol %in% subCol)]'

    • at line 544 found ' estRmat <- estRmat[, allCol]'

    • at line 853 found ' ll <- sum(Matrix::t(counts) log(theta t(phi)[z, ] +'

    • at line 854 found ' (1 - theta) * t(eta)[z, ] + 1e-20))'

    • at line 866 found ' ll <- sum(t(counts) log(theta t(phi)[cbZ, ] +'

    • at line 867 found ' (1 - theta) * t(eta)[globalZ, ] + 1e-20))'

    • at line 889 found ' logPr <- log(t(phi)[z, ] + 1e-20) + log(theta + 1e-20)'

    • at line 890 found ' logPc <- log(t(eta)[z, ] + 1e-20) + log(1 - theta + 1e-20)'

    • at line 932 found ' logPr <- log(t(phi)[cbZ, ] + 1e-20) + log(theta + 1e-20)'

    • at line 934 found ' log(t(eta)[globalZ, ] + 1e-20) + log(1 - theta + 1e-20)'

    • at line 1342 found ' phi[i, ix] <- max(phi[i, ])'

    • at line 1344 found ' phi[j, ix] <- 0'

    • at line 1351 found ' stats::rmultinom(1, size = rNbyC[i], prob = phi[z[i], ])'

    • at line 1370 found ' stats::rmultinom(1, size = cNbyC[i], prob = eta[, z[i]])'

    • In file R/plot_decontPro.R:

    • at line 42 found ' df <- data.frame(con = counts[feature, ],'

    • at line 43 found ' decon = decontaminated_counts[feature, ])'

    • at line 145 found ' con = counts[feature, ],'

    • at line 146 found ' decon = decontaminated_counts[feature, ],'

    • In file R/plot_decontx.R:

    • at line 60 found ' df <- df[!naIx, ]'

    • at line 212 found ' counts <- x[geneMarkerIndex, ]'

    • at line 383 found ' counts <- x[geneMarkerIndex, ]'

    • at line 468 found ' z <- SummarizedExperiment::colData(x)[, z]'

    • at line 511 found ' x <- x[, !na.ix]'

    • In file R/stan_helpers.R:

    • at line 72 found ' unscaled_rates <- t(r_est[dat$cell_type, ])'

  • [ ] NOTE: Functional programming: code repetition.

    • repetition in .cDCalcEMbgDecontamination and .cDCalcEMDecontamination

    • in .cDCalcEMbgDecontamination

      • line 17: phi <- celda::normalizeCounts(phiUnnormalized, normalize = "proportion",
      • line 18: pseudocountNormalize = 1e-20)
      • line 19: eta <- celda::normalizeCounts(etaUnnormalized, normalize = "proportion",
      • line 20: pseudocountNormalize = 1e-20)
      • line 21: return(list(estRmat = estRmat, theta = theta, phi = phi,
      • line 22: eta = eta, delta = deltaV2))
    • in .cDCalcEMDecontamination

      • line 18: phi <- celda::normalizeCounts(rnGByK, normalize = "proportion",
      • line 19: pseudocountNormalize = 1e-20)
      • line 20: eta <- celda::normalizeCounts(cnGByK, normalize = "proportion",
      • line 21: pseudocountNormalize = 1e-20)
      • line 22: return(list(estRmat = estRmat, theta = theta, phi = phi,
      • line 23: eta = eta, delta = deltaV2))
    • repetition in .colSumByGroupInteger and .colSumByGroupNumeric

    • in .colSumByGroupInteger

      • line 1: group, K)
      • line 2:{
      • line 3: group <- factor(group, levels = seq(K))
      • line 4: res <- .Call("_colSumByGroup", x, group)
      • line 5: return(res)
    • in .colSumByGroupNumeric

      • line 1: group, K)
      • line 2:{
      • line 3: group <- factor(group, levels = seq(K))
      • line 4: res <- .Call("_colSumByGroup_numeric", x, group)
      • line 5: return(res)
    • repetition in plotDecontXMarkerExpression and plotDecontXMarkerPercentage

    • in plotDecontXMarkerExpression

      • line 5: legend <- "none"
      • line 6: temp <- .processPlotDecontXMarkerInupt(x = x, z = z,
      • line 7: markers = markers, groupClusters = groupClusters,
      • line 8: by = by, exactMatch = exactMatch)
      • line 9: x <- temp$x
      • line 10: z <- temp$z
      • line 11: geneMarkerIndex <- temp$geneMarkerIndex
      • line 12: groupClusters <- temp$groupClusters
      • line 13: xlab <- temp$xlab
      • line 14: if (inherits(x, "SingleCellExperiment")) {
      • line 15: df.list <- list()
      • line 16: for (i in seq_along(assayName)) {
      • line 17: counts <- SummarizedExperiment::assay(x[geneMarkerIndex,
      • line 18: ], assayName[i])
      • line 25: if (length(assayName) > 1) {
      • line 26: legend <- "right"
      • line 27: }
      • line 28: }
      • line 29: else {
      • line 30: counts <- x[geneMarkerIndex, ]
    • in plotDecontXMarkerPercentage

      • line 10: }
      • line 11: temp <- .processPlotDecontXMarkerInupt(x = x, z = z,
      • line 12: markers = markers, groupClusters = groupClusters,
      • line 13: by = by, exactMatch = exactMatch)
      • line 14: x <- temp$x
      • line 15: z <- temp$z
      • line 16: geneMarkerIndex <- temp$geneMarkerIndex
      • line 17: geneMarkerCellTypeIndex <- temp$geneMarkerCellTypeIndex
      • line 18: groupClusters <- temp$groupClusters
      • line 19: xlab <- temp$xlab
      • line 20: if (inherits(x, "SingleCellExperiment")) {
      • line 21: df.list <- list()
      • line 22: for (i in seq_along(assayName)) {
      • line 23: counts <- SummarizedExperiment::assay(x[geneMarkerIndex,
      • line 24: ], assayName[i])
      • line 31: if (length(assayName) > 1) {
      • line 32: legend <- "right"
      • line 33: }
      • line 34: }
      • line 35: else {
      • line 36: counts <- x[geneMarkerIndex, ]

Documentation

  • [x] Important: Vignette should have an Installation section.

    • rmd file vignettes/decontPro.Rmd
    • rmd file vignettes/decontX.Rmd
  • [x] Important: Please include Bioconductor installation instructions using BiocManager.

    • rmd file vignettes/decontPro.Rmd
    • rmd file vignettes/decontX.Rmd
  • [ ] Note: Vignette includes motivation for submitting to Bioconductor as part of the abstract/intro of the main vignette.

    • rmd file vignettes/decontPro.Rmd
    • rmd file vignettes/decontX.Rmd
  • [ ] To avoid the '* NOTE: Consider adding runnable examples to man pages that document exported objects.' in BiocCheck, could you please add runnable examples to
    • decontXcounts.Rd
  • [ ] NOTE: Consider adding a NEWS file, so your package news will be included in Bioconductor release announcements.
bioc-issue-bot commented 11 months ago

Received a valid push on git.bioconductor.org; starting a build for commit id: ca82730d8a9715c04526d1919a1f98c5c02ff20f

bioc-issue-bot commented 11 months ago

Dear Package contributor,

This is the automated single package builder at bioconductor.org.

Your package has been built on the Bioconductor Build System.

Congratulations! The package built without errors or warnings on all platforms.

Please see the build report for more details.

The following are build products from R CMD build on the Bioconductor Build System: macOS 12.6.5 Monterey: decontX_0.99.4.tar.gz Linux (Ubuntu 22.04.2 LTS): decontX_0.99.4.tar.gz

Links above active for 21 days.

Remember: if you submitted your package after July 7th, 2020, when making changes to your repository push to git@git.bioconductor.org:packages/decontX to trigger a new build. A quick tutorial for setting up remotes and pushing to upstream can be found here.

jianhong commented 11 months ago

Before I close this review, I just want to say, please make sure to update a NEWS file and make sure the LICENSE and LICENSE.md file are consistent.

bioc-issue-bot commented 11 months ago

Your package has been accepted. It will be added to the Bioconductor nightly builds.

Thank you for contributing to Bioconductor!

Reviewers for Bioconductor packages are volunteers from the Bioconductor community. If you are interested in becoming a Bioconductor package reviewer, please see Reviewers Expectations.

bioc-issue-bot commented 11 months ago

cannot build unless issue is open and has the 'pre-review' label or '2. review in progress' label, or is closed and has the 'TESTING' label.

yuan-yin-truly commented 11 months ago

Before I close this review, I just want to say, please make sure to update a NEWS file and make sure the LICENSE and LICENSE.md file are consistent.

Hi @jianhong Thank you! I just did the following to the local devel branch and origin devel branch:

Was trying to push to upstream (git.bioconductor.org) but was blocked. May I know in the future if there is any changes, I just need to push to origin devel branch?

lshep commented 11 months ago

It wasn't blocked it went through to git.biconductor.org. you just wont get a new build report as once packages are accepted they move to the official Bioconductor daily builder.

jianhong commented 11 months ago

I did not expect you will patch the package today. No need to change it now. Once you package got accepted, the auto build system will not work. Please keep those comments in mind for your next updates.

From: Yuan Yin @.> Date: Thursday, October 19, 2023 at 2:15 PM To: Bioconductor/Contributions @.> Cc: JIANHONG OU @.>, Mention @.> Subject: Re: [Bioconductor/Contributions] decontX (Issue #3020)

Before I close this review, I just want to say, please make sure to update a NEWS file and make sure the LICENSE and LICENSE.md file are consistent.

Hi @jianhonghttps://github.com/jianhong Thank you! I just did the following to the local devel branch and origin devel branch:

Was trying to push to upstream (git.bioconductor.org) but was blocked. May I know in the future if there is any changes, I just need to push to origin devel branch?

— Reply to this email directly, view it on GitHubhttps://github.com/Bioconductor/Contributions/issues/3020#issuecomment-1771488998, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ABLBEA7POK7BDYEJGLR7MKTYAFU55AVCNFSM6AAAAAAXH7JU4GVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTONZRGQ4DQOJZHA. You are receiving this because you were mentioned.Message ID: @.***>

yuan-yin-truly commented 11 months ago

@jianhong To clarify, I should wait until status change from 3a. accepted to another status (which one?), and then try pushing the new changes to git.bioconductor.org again?

lshep commented 11 months ago

The changes are already in git.bioconductor.org. You dont need to do anything further to get them into Bioconductor

yuan-yin-truly commented 11 months ago

Hi @lshep, just to clarify, I made some changes a couple of minutes ago after Jianhong changed the packaged status to 3a. accepted, and when I try to push the changes to git.bioconductor.org, I had the following message on my git command line even though I have bumped version in DESCRIPTION:

remote: Error: Please bump the version again and push. remote: remote: The build did not start as expected. If the issue persists, remote: please reach out at bioc-devel@r-project.org or post on the remote: Github issue where your package is being reviewed. remote: remote: 400 Client Error: Bad Request for url: https://issues.bioconductor.org/start_build remote: To git.bioconductor.org:packages/decontX.git ca82730..03e3e7d devel -> devel

and I had this message in this issue:

cannot build unless issue is open and has the 'pre-review' label or '2. review in progress' label, or is closed and has the 'TESTING' label.

I was wondering if the changes I made were pushed to git.bioconductor.org? Or should I wait until some other status (maybe after the nightly build) and try pushing the changes to upstream (git.bioconductor.org) again? Thank you!

lshep commented 11 months ago

Again -- the changes were pushed to git.bioconductor.org -- the error is that it would not trigger an on demand build by the single package builder. We normally don't expect changes while a package is accepted but hasn't been added to the daily builder yet so its a rare edge case.

yuan-yin-truly commented 11 months ago

Again -- the changes were pushed to git.bioconductor.org -- the error is that it would not trigger an on demand build by the single package builder. We normally don't expect changes while a package is accepted but hasn't been added to the daily builder yet so its a rare edge case.

Got ya! It makes sense. Thank you for clarifying!

lshep commented 11 months ago

The default branch of your GitHub repository has been added to Bioconductor's git repository as branch devel.

To use the git.bioconductor.org repository, we need an 'ssh' key to associate with your github user name. If your GitHub account already has ssh public keys (https://github.com/yuan-yin-truly.keys is not empty), then no further steps are required. Otherwise, do the following:

  1. Add an SSH key to your github account
  2. Submit your SSH key to Bioconductor

See further instructions at

https://bioconductor.org/developers/how-to/git/

for working with this repository. See especially

https://bioconductor.org/developers/how-to/git/new-package-workflow/ https://bioconductor.org/developers/how-to/git/sync-existing-repositories/

to keep your GitHub and Bioconductor repositories in sync.

Your package will be included in the next nigthly 'devel' build (check-out from git at about 6 pm Eastern; build completion around 2pm Eastern the next day) at

https://bioconductor.org/checkResults/

(Builds sometimes fail, so ensure that the date stamps on the main landing page are consistent with the addition of your package). Once the package builds successfully, you package will be available for download in the 'Devel' version of Bioconductor using BiocManager::install("decontX"). The package 'landing page' will be created at

https://bioconductor.org/packages/decontX

If you have any questions, please contact the bioc-devel mailing list (https://stat.ethz.ch/mailman/listinfo/bioc-devel); this issue will not be monitored further.