Vince Carey

I am seeing that you have a tarball exceeding 5MB with a lot of large nice gifs under inst. We have to keep our package tarball sizes low. If you...

Thanks for dealing with the images. @lshep this can be moderated. Garth it usually takes some time to get into git.

I am looking at 0.99.4. The comments from Pete above seem to hold ... can't find example data readily when starting the app. I'd also add a few points -...

@lshep will have the best advice here. As long as you make proper attributions I think there is no problem with you using EH for data that is integral to...

I built the vignette and noted a lot of sim_tries and other files created in the folder. You should follow http://contributions.bioconductor.org/r-code.html?q=BiocFileCache#web-querying-and-file-caching and do i/o in the temp folder

This is what I am seeing on current checkout ``` * checking Rd cross-references ... WARNING Unknown package ‘RBowtie2’ in Rd xrefs Missing link or links in documentation object 'calc_coverage.Rd':...

Looks like you should have used the related packages protocol. @lshep

https://github.com/Bioconductor/Contributions/blob/master/CONTRIBUTING.md#submitting-related-packages which is linked from 1.5 of the general guidelines

This package lacks a vignette. Please read the guidelines.

Still no vignette. Should we close this issue and wait for you to produce a vignette?