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

compareSingleCell #1113

Closed
8 tasks done
LTLA opened this issue May 4, 2019 · 21 comments
Closed
8 tasks done

compareSingleCell #1113

LTLA opened this issue May 4, 2019 · 21 comments
Assignees
Labels

Comments

@LTLA
Copy link

LTLA commented May 4, 2019

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 understand that by submitting my package to Bioconductor,
    the package source and all review commentary are visible to the
    general public.

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

  • 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.

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

  • 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:

  • The 'devel' branch for new packages and features.
  • The stable 'release' branch, made available every six
    months, for bug fixes.
  • Bioconductor version control using Git
    (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 @LTLA

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: compareSingleCell 
Title: Comparative Analyses of Single-Cell RNA-seq Data with Bioconductor
Version: 0.99.0
Date: 2019-05-04
Authors@R: c(
    person(role=c("aut", "cre"), "Aaron", "Lun", email = "infinite.monkeys.with.keyboards@gmail.com"),
    person(role=c("aut"), "Jonathan", "Griffiths"))
Description: Describes a comparative analysis of scRNA-seq data using scran, scater and other Bioconductor packages.
    This uses a multi-condition replicated data set from a study of early mammalian development. It demonstrates how 
    merge sample together to test for differential abundance and differential expression between conditions.
Imports: BiocStyle, callr, rmarkdown
Suggests: knitr,
        SingleCellExperiment,
        scater,
        scran,
        DropletUtils,
        batchelor,
        BiocSingular,
        BiocFileCache,
        TxDb.Mmusculus.UCSC.mm10.ensGene,
        pheatmap,
        igraph,
        edgeR,
        Matrix
License: Artistic-2.0
Encoding: UTF-8
LazyData: true
VignetteBuilder: knitr
biocViews: Workflow, SingleCellWorkflow
NeedsCompilation: no
URL: https://www.bioconductor.org/help/workflows/compareSingleCell/ 
RoxygenNote: 6.1.1

Add SSH keys to your GitHub account. SSH keys
will are used to control access to accepted Bioconductor
packages. See these instructions to add SSH keys to
your GitHub account.

@bioc-issue-bot
Copy link
Collaborator

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

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 May 4, 2019
@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, TIMEOUT".
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.

@bioc-issue-bot
Copy link
Collaborator

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

f1982de Bumped version, date.

@LTLA
Copy link
Author

LTLA commented May 10, 2019

Bumping in the hope that some of the resources got cached after the first TIMEOUT, so they don't have to get re-downloaded. Downloading seems to be by far the longest, at least locally.

@LTLA
Copy link
Author

LTLA commented May 10, 2019

Incidentally, I can't help but feel @hpages may be more interested in reviewing #1114, while @lshep may be more interested in reviewing this one!

@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, TIMEOUT".
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.

@LTLA
Copy link
Author

LTLA commented May 10, 2019

Not quite sure what's going on here... looks like it's not getting detected as a workflow.

@lshep
Copy link
Contributor

lshep commented May 10, 2019

Workflow: True in the DESCRIPTION

@bioc-issue-bot
Copy link
Collaborator

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

b28942e Specify workflow for longer BioC compilation.

@LTLA
Copy link
Author

LTLA commented May 10, 2019

Thanks @lshep. I think this is the third time I've asked this question and you've given this answer! Looks like my strategy of copying a DESCRIPTION from a previous workflow was insufficient - simpleSingleCell doesn't even have a Workflow: section.

@lshep lshep assigned lshep and unassigned hpages May 13, 2019
@lshep
Copy link
Contributor

lshep commented May 14, 2019

Can you check the build time locally for the package? I kicked off this manually thinking it stalled but there are two triggers that never finished building the package - I'm wondering if I need to check the timeout flag on the build process or if something more is going on...

@LTLA
Copy link
Author

LTLA commented May 15, 2019

The downloads are very slow. Took me hours to download them - which doesn't really make sense to me, the files are not particularly big. The servers are in the UK, but still... maybe it's Brexit.

Once cached, the actual computation should be fast - no more than half an hour, but I'd have to go and time it properly. Will get back to you with some more concrete numbers.

@bioc-issue-bot
Copy link
Collaborator

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

74cbeb9 Use global cache for downloads instead.

@LTLA
Copy link
Author

LTLA commented May 15, 2019

I realized I was caching the downloads locally - ostensibly to check that they could be re-downloaded at every build - but any locally cached results would be killed when the SPB times out and deletes the working directory of the build-in-progress. So, I've switched to the global cache, assuming that it persists on the SPB. My suspicion is that it will still TIMEOUT, though; each download is just very slow.

Might be worth considering EHub packages for this data - I'll prod a student on my end.

@lshep
Copy link
Contributor

lshep commented May 28, 2019

Just wanted to follow up on this - is there a better solution for downloading the data ? How many different files are we talking about - I could always download them manually on the builders so they are there but if files are updated that would probably require a manual redownload then - This also doesnt help the user who may suffer through long download times. Is it an issue that the files are large or the site they are being downloaded from? It made it seem like these weren't particularly big but could be a server issue? If thats the case maybe EH versions would be the way to go?

@LTLA
Copy link
Author

LTLA commented May 29, 2019

The files are somewhat big (500 MB, give or take) and the server is a lab-specific resource that isn't really powered up; I suspect that these two factors combined are probably causing the slow download. I've been nagging someone on the UK end to make an EHub package for this data, but they're taking too long to do this, so I think I will just make one myself over the weekend.

@lshep
Copy link
Contributor

lshep commented Jun 12, 2019

Should I close this issue for now or do you think there will be an accompanied data package soon?

@LTLA
Copy link
Author

LTLA commented Jun 13, 2019

We're almost ready to go with the data package, just a few more checks and then we can submit. Can I re-use the credentials you sent me earlier for this new package?

@lshep
Copy link
Contributor

lshep commented Jun 13, 2019

Yes those credentials have not expired yet and it should be okay to use. If there is an issue let me know

@LTLA
Copy link
Author

LTLA commented Jul 4, 2019

I'm going to close this for the time being, as I might absorb the content here into Rob's book instead of having a separate series of workflows.

@LTLA LTLA closed this as completed Jul 4, 2019
@lshep lshep removed the 2. review in progress assign a reviewer and a more thorough review of package code and documentation taking place label Mar 22, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

4 participants