Vince Carey

Results 211 comments of Vince Carey

Package builds to 8MB which is too big. But it can be reviewed.

This is what the first comment was edited with; please create NEW issues to provide your content in response to review. ``` Dear Bioconductor team, Please find below the GitHub...

Does your package build on R 4.2/Bioc 3.16? ``` > start sce

Please read your vignette. You print out an entire dataset in the vignette.

pre-check is passed but think about user effort to do ``` df_comb_minimal[,"distance_int"]

Thanks for this submission. I think ``` head(EML4_ALK_detection(file = H3122_bam, genome = "hg38", mates = 2)) #> pos cigar mpos #> 726 42299657 94M2S 29223691 #> 727 42299657 94M2S 29223375...

be sure to read the FAQ about git pushing issues and consider pushing to https://github.com/bioconductor.org/packages/DNAfusion

Oh, right, I have to change the labels once I look at the revisions. Thanks @LiNk-NY !!

Even when it gets to "pre-check passed" there is a manual ingestion step needed.

Please use BiocCheck on your package. I got ``` * Checking License: for restrictive use... * NOTE: License 'GPL (>= 3)' unknown; licenses cannot restrict use ``` among other concerns;...