Bioconductor / Contributions

Contribute Packages to Bioconductor
135 stars 33 forks source link

terapadog #3615

Open Gionmattia opened 1 month ago

Gionmattia commented 1 month 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 month ago

Hi @Gionmattia

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: terapadog
Type: Package
Title: Translational Efficiency Regulation Analysis using the PADOG Method
Version: 0.99.0
Authors@R: person("Gionmattia", "Carancini",
  email = "gionmattia@gmail.com",
  role = c("cre", "aut"),
  comment = c(ORCID = "0000-0001-7936-4883"))
Description: This package performs a Gene Set Analysis with the approach adopted 
    by PADOG on the genes that are reported as translationally regulated 
    (ie. exhibit a significant change in TE) by the DeltaTE package.
    It can be used on its own to see the impact of translation regulation on 
    gene sets, but it is also integrated as an additional analysis method 
    within ReactomeGSA, where results are further contextualised in terms of
    pathways and directionality of the change.
License: GPL-2
Encoding: UTF-8
LazyData: TRUE
Imports: DESeq2, KEGGREST, stats, utils, dplyr, plotly, htmlwidgets, biomaRt, methods
Suggests: apeglm, BiocStyle, knitr, rmakrkdown
Collate: preprocessing_helpers.R id_converter.R prepareTerapadogData.R terapadog.R get_FCs.R assign_Regmode.R plotDTA.R 
biocViews: RiboSeq, Transcriptomics, GeneSetEnrichment, GeneRegulation, 
    Reactome, Software
RoxygenNote: 7.3.2
VignetteBuilder: knitr
lshep commented 1 week ago

We recommend changing the name of vignette.Rmd to something more informative as there can be naming conflicts depending on package load order.

id_converter should not automatically write to the home directory. You should have an argument option on where this file should be written that should default to a tempdir(). Similar for plotDTA

You mention how its integration with ReactomeGSA is where it truly shines but I don't see that demonstrated in the vignette? currently I don't see the interoperability?