Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

swfdr #213

Closed
SiminaB opened this issue Dec 9, 2016 · 30 comments
Closed

swfdr #213

SiminaB opened this issue Dec 9, 2016 · 30 comments
Assignees
Labels
3a. accepted will be ingested into Bioconductor daily builder for distribution

Comments

@SiminaB
Copy link

SiminaB commented Dec 9, 2016

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]'

  • [X ] I understand that by submitting my package to Bioconductor,
    the package source and all review commentary are visible to the
    general public.

  • [ X] I have read the Bioconductor Package Submission
    instructions. My package is consistent with the Bioconductor
    Package Guidelines.

  • [ X] I understand that a minimum requirement for package acceptance
    is to pass R CMD check and R CMD BiocCheck with no ERROR or WARNINGS.
    Passing these checks does not result in automatic acceptance. The
    package will then undergo a formal review and recommendations for
    acceptance regarding other Bioconductor standards will be addressed.

  • [ X] My package addresses statistical or bioinformatic issues related
    to the analysis and comprehension of high throughput genomic data.

  • [ X] I am committed to the long-term maintenance of my package. This
    includes monitoring the support site for issues that users may
    have, subscribing to the bioc-devel mailing list to stay aware
    of developments in the Bioconductor community, responding promptly
    to requests for updates from the Core team in response to changes in
    R or underlying software.

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

  • [ ]X The 'devel' branch for new packages and features.
  • [ X] The stable 'release' branch, made available every six
    months, for bug fixes.
  • [ X] Bioconductor version control using Subversion
    (optionally via GitHub).

For help with submitting your package, please subscribe and post questions
to the bioc-devel mailing list.

@bioc-issue-bot
Copy link
Collaborator

Hi @SiminaB

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: swfdr
Title: Science-wise false discovery rate estimation
Version: 0.99.0
Author: Jeffrey T. Leek, Simina M. Boca
Maintainer: Simina M. Boca <smb310@georgetown.edu>, Jeffrey T. Leek <jtleek@gmail.com>
Description: This package allows users to estimate the science-wise false discovery rate from Jager and Leek, 
    "Empirical estimates suggest most published medical research is true," 2013, Biostatistics, using an EM approach
    due to the presence of rounding and censoring. It also allows users to estimate the proportion of true null
    hypotheses in the presence of covariates, using a regression framework, as per Boca and Leek, "A regression
    framework for the proportion of true null hypotheses," 2015, bioRxiv preprint.
Depends:
    R (>= 3.3.0)
Imports:
    MASS, splines, stats4, genefilter, lme4, reshape2
License: GPL (>= 3)
Encoding: UTF-8
LazyData: true
RoxygenNote: 5.0.1
Suggests: knitr,
    rmarkdown
VignetteBuilder: knitr

@bioc-issue-bot
Copy link
Collaborator

Your package has been approved for building. Your package is
now submitted to our queue.

IMPORTANT: Please read the instructions for setting
up a push hook on your repository, or further changes to your
repository will NOT trigger a new build.

@bioc-issue-bot bioc-issue-bot added 2. review in progress assign a reviewer and a more thorough review of package code and documentation taking place and removed 1. awaiting moderation labels Dec 9, 2016
@bioc-issue-bot
Copy link
Collaborator

Dear Package contributor,

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

Your package has been built on Linux, Mac, and Windows.

On one or more platforms, the build results were: "ERROR, 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 following build report for more details:

http://bioconductor.org/spb_reports/swfdr_buildreport_20161209151659.html

@bioc-issue-bot
Copy link
Collaborator

We only start builds when the Version field in the DESCRIPTION
file is incremented. For example, by changing

Version: 0.99.0

to

Version 0.99.1

If you did not intend to start a build, you don't need to
do anything. If you did want to start a build, increment
the Version: field and try again.

2 similar comments
@bioc-issue-bot
Copy link
Collaborator

We only start builds when the Version field in the DESCRIPTION
file is incremented. For example, by changing

Version: 0.99.0

to

Version 0.99.1

If you did not intend to start a build, you don't need to
do anything. If you did want to start a build, increment
the Version: field and try again.

@bioc-issue-bot
Copy link
Collaborator

We only start builds when the Version field in the DESCRIPTION
file is incremented. For example, by changing

Version: 0.99.0

to

Version 0.99.1

If you did not intend to start a build, you don't need to
do anything. If you did want to start a build, increment
the Version: field and try again.

@bioc-issue-bot
Copy link
Collaborator

Received a valid push; starting a build. Commits are:

b0e5d70 Incremented version to 0.99.1

@bioc-issue-bot
Copy link
Collaborator

Dear Package contributor,

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

Your package has been built on Linux, Mac, and Windows.

On one or more platforms, the build results were: "skipped, ERROR, 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 following build report for more details:

http://bioconductor.org/spb_reports/swfdr_buildreport_20161209230745.html

@bioc-issue-bot
Copy link
Collaborator

Received a valid push; starting a build. Commits are:

0077a2f Fixed some errors, incremented version to 0.99.2

@bioc-issue-bot
Copy link
Collaborator

Dear Package contributor,

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

Your package has been built on Linux, Mac, and Windows.

On one or more platforms, the build results were: "skipped, ERROR, 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 following build report for more details:

http://bioconductor.org/spb_reports/swfdr_buildreport_20161210004014.html

@bioc-issue-bot
Copy link
Collaborator

Received a valid push; starting a build. Commits are:

38a712d Fixed some warnings, incremented to 0.99.3

@bioc-issue-bot
Copy link
Collaborator

Dear Package contributor,

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

Your package has been built on Linux, Mac, and Windows.

On one or more platforms, the build results were: "skipped, ERROR, 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 following build report for more details:

http://bioconductor.org/spb_reports/swfdr_buildreport_20161210201234.html

@bioc-issue-bot
Copy link
Collaborator

Received a valid push; starting a build. Commits are:

74f36ad Removed files from inst/doc and upgraded to 0.99.4

@bioc-issue-bot
Copy link
Collaborator

Dear Package contributor,

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

Your package has been built on Linux, Mac, and Windows.

On one or more platforms, the build results were: "WARNINGS, skipped, 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 following build report for more details:

http://bioconductor.org/spb_reports/swfdr_buildreport_20161210205522.html

@bioc-issue-bot
Copy link
Collaborator

Received a valid push; starting a build. Commits are:

bce8f54 Changes to vignette header, incremented to 0.99.5

@bioc-issue-bot
Copy link
Collaborator

Dear Package contributor,

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

Your package has been built on Linux, Mac, and Windows.

On one or more platforms, the build results were: "skipped, ERROR, 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 following build report for more details:

http://bioconductor.org/spb_reports/swfdr_buildreport_20161211000638.html

@bioc-issue-bot
Copy link
Collaborator

Received a valid push; starting a build. Commits are:

b3107cd Changed vignette to html, incremented to 0.99.6

@bioc-issue-bot
Copy link
Collaborator

Dear Package contributor,

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

Your package has been built on Linux, Mac, and Windows.

On one or more platforms, the build results were: "ERROR, 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 following build report for more details:

http://bioconductor.org/spb_reports/swfdr_buildreport_20161211003244.html

@bioc-issue-bot
Copy link
Collaborator

Received a valid push; starting a build. Commits are:

817b1cd Compressed data file, incremented to 0.99.7

@bioc-issue-bot
Copy link
Collaborator

Dear Package contributor,

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

Your package has been built on Linux, Mac, and Windows.

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 following build report for more details:

http://bioconductor.org/spb_reports/swfdr_buildreport_20161212011808.html

@bioc-issue-bot
Copy link
Collaborator

Received a valid push; starting a build. Commits are:

206c77c Removed 2 variables, incremented to 0.99.8 Remove...

@bioc-issue-bot
Copy link
Collaborator

Dear Package contributor,

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

Your package has been built on Linux, Mac, and Windows.

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 following build report for more details:

http://bioconductor.org/spb_reports/swfdr_buildreport_20161213162644.html

@SiminaB
Copy link
Author

SiminaB commented Dec 14, 2016

Hi Herve @hpages ,
It looks like the ERROR issue with the MAC build on oaxaca is common to other folks as well, for example here http://bioconductor.org/spb_reports/ChIPexoQual_buildreport_20161207193202.html #79
I have a Windows machine and I cannot reproduce the error - it looks like pdflatex is just not installed on oaxaca though.

Thanks so much!
Best,
Simina

@bioc-issue-bot
Copy link
Collaborator

Received a valid push; starting a build. Commits are:

12b532b Incremented version to 0.99.9 to test Mac build

@bioc-issue-bot
Copy link
Collaborator

Dear Package contributor,

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

Your package has been built on Linux, Mac, and Windows.

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

Please see the following build report for more details:

http://bioconductor.org/spb_reports/swfdr_buildreport_20161221233157.html

@bioc-issue-bot bioc-issue-bot added OK and removed ERROR labels Dec 22, 2016
@mtmorgan mtmorgan assigned mtmorgan and unassigned hpages Jan 13, 2017
@nturaga nturaga self-assigned this Jan 13, 2017
@mtmorgan
Copy link
Contributor

I'm really sorry for the delay in getting this review to you. The package looks in great shape, and I have only one suggestion

  • include the script for generating journals_pVals.Rd as a script,
    inst/script/getPvalues.R instead of referencing a github url.

Please respond and if you change your package include a version bump, and I'll be happy to accept the revision.

@bioc-issue-bot
Copy link
Collaborator

Received a valid push; starting a build. Commits are:

4c41bd6 Small changes in DESCRIPTION, add inst/script

@bioc-issue-bot
Copy link
Collaborator

Dear Package contributor,

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

Your package has been built on Linux, Mac, and Windows.

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

Please see the following build report for more details:

http://bioconductor.org/spb_reports/swfdr_buildreport_20170127111624.html

@SiminaB
Copy link
Author

SiminaB commented Jan 27, 2017

Thank you so much @mtmorgan ! I just made the change you suggested.

@mtmorgan
Copy link
Contributor

Great, I've marked your package as accepted. It will be added to the Bioconductor SVN repository and additional information forwarded over the next couple of days.

@mtmorgan mtmorgan added 3a. accepted will be ingested into Bioconductor daily builder for distribution and removed 2. review in progress assign a reviewer and a more thorough review of package code and documentation taking place OK labels Jan 28, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
3a. accepted will be ingested into Bioconductor daily builder for distribution
Projects
None yet
Development

No branches or pull requests

5 participants