Bioconductor / Contributions

Contribute Packages to Bioconductor
134 stars 33 forks source link

QHScrnomo #2418

Closed jixccf closed 2 years ago

jixccf commented 2 years 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 2 years ago

Dear @jixccf,

The package version number, '2.99.0', does not start with 0. Expecting format: '0.99.z' for new packages. Starting with non-zero x of 'x.y.z' format is generally only allowed if the package has been pre-released.

We recommend fixing the version number. See Bioconductor version numbers Please also consider running BiocCheck::BiocCheck() on your package to look for other Bioconductor package requirements.

bioc-issue-bot commented 2 years ago

Hi @jixccf

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: QHScrnomo
Type: Package
Title: Construct nomograms for competing risk models
Version: 2.99.0
biocViews: Regression, TimeCourse, Survival
Date: 2021-10-27
Authors@R: c(
    person("Michael", "Kattan", email = "kattanm@ccf", role = "aut"),
    person("Changhong", "Yu", email = "yuc@ccf.org", role = "aut"),
    person("Xinge", "Ji", email="jix@ccf.org", role="cre"))
Depends: R (>= 4.1.0),Hmisc,rms,cmprsk
Suggests: 
    rmarkdown,
    knitr,
    survival,
    testthat (>= 3.0.0)
Description: Nomograms are constructed to predict cumulative incidence
    rate, which is calculated by adjusting for competing causes other
    than that of interest. K-fold cross-validation function is implemented
    to validate the performance. Competing risk version of concordance index
    is calculated.
License: GPL-2
NeedsCompilation: yes
Config/testthat/edition: 3
RoxygenNote: 7.1.2
VignetteBuilder: knitr
vjcitn commented 2 years ago

I don't see how email = "kattanm@ccf" in the DESCRIPTION could be valid.

Is it possible to supply a genomics-oriented application? I think this is an interesting package and would like to see it moving forward here, but Bioconductor doesn't typically review/distribute packages with no example of application in genomic data. Your vignette says you are using artificial prostate cancer data. Can you use data from curatedTCGAData to do something more realistic?

jixccf commented 2 years ago

Thanks for your review. I will close this issue for now and update it.