Hi Herve,

I must retract my previous statement about 'yieldSize' and 'which'. As of Rsamtools 1.15.0, scanBam() (and functions that build on it) does handle the case where both are supplied. This is true for the non-mate and mate-pairing code.

From ?BamFile:

     yieldSize: Number of records to yield each time the file is read
          from using ‘scanBam’ or, when ‘length(bamWhich()) != 0’, a
          threshold which yields records in complete ranges whose sum
          first exceeds ‘yieldSize’. Setting ‘yieldSize’ on a
          ‘BamFileList’ does not alter existing yield sizes set on the
          individual ‘BamFile’ instances.


fl <- system.file("extdata", "ex1.bam", package="Rsamtools")
bf <- BamFile(fl, yieldSize=1000)
which <- tileGenome(seqlengths(bf),
     tilewidth=500, cut.last.tile.in.chrom=TRUE)
param <- ScanBamParam(which=which, what="qname")
FUN <- function(elt) length(elt[[1]])

Here we have both 'yieldSize' and a 'which' in the param. We ask for a yield of 1000 records. The first range only has 394, the second has 570 and from the third we get 570. As explained in the man page snippit above, records are yielded in complete ranges whose sum first exceeds 'yieldSize'. In range 3 we exceed the 1000 so we get all of range 3 then stop.

sapply(scanBam(bf, param=param), FUN)
sapply(scanBam(bf, param=param), FUN)
    seq1:1-500  seq1:501-1000 seq1:1001-1500 seq1:1501-1575     seq2:1-500
           394            570            570              0              0
 seq2:501-1000 seq2:1001-1500 seq2:1501-1584
             0              0              0

We can open the file and yield through all records:

bf <- open(BamFile(fl, yieldSize=1000))
sapply(scanBam(bf, param=param), FUN)

sapply(scanBam(bf, param=param), FUN)
    seq1:1-500  seq1:501-1000 seq1:1001-1500 seq1:1501-1575     seq2:1-500
           394            570            570              0              0
 seq2:501-1000 seq2:1001-1500 seq2:1501-1584
             0              0              0
sapply(scanBam(bf, param=param), FUN)
    seq1:1-500  seq1:501-1000 seq1:1001-1500 seq1:1501-1575     seq2:1-500
             0              0              0             82            562
 seq2:501-1000 seq2:1001-1500 seq2:1501-1584
           709              0              0
sapply(scanBam(bf, param=param), FUN)
    seq1:1-500  seq1:501-1000 seq1:1001-1500 seq1:1501-1575     seq2:1-500
             0              0              0              0              0
 seq2:501-1000 seq2:1001-1500 seq2:1501-1584
             0            597             60
sapply(scanBam(bf, param=param), FUN)
    seq1:1-500  seq1:501-1000 seq1:1001-1500 seq1:1501-1575     seq2:1-500
             0              0              0              0              0
 seq2:501-1000 seq2:1001-1500 seq2:1501-1584
             0              0              0


I've removed the misinformation from the man pages I altered. Also added a unit test for the mates code with 'yieldSize' and 'which' in Rsamtools.

Val

On 03/27/2014 11:36 AM, Hervé Pagès wrote:
Hi Val,

On 03/27/2014 09:13 AM, Valerie Obenchain wrote:
I should also mention that when both 'yieldSize' in the BamFile and
'which' in ScanBamParam are set the 'which' gets priority. The point of
'yieldSize' is to provide a reasonable chunk for processing the file.
When 'which' is provided it's assumed that range is of reasonable chunk
size so the yield is ignored.

Note that more than 1 range can be specified thru 'which'.

What about emitting a warning when 'yieldSize' is ignored?


I've added this info to the 'summarizeOverlaps' and 'readGAlignments'
man pages in GenomicAlignments.

Is this a property of scanBam()? If so then maybe it should be
documented in the man page for scanBam(). I just had a quick look
and was not able to find it, sorry if I missed it.

summarizeOverlaps(), readGAlignments(), and probably other tools,
just rely on scanBam().

Thanks,
H.


Valerie

On 03/27/14 08:30, Valerie Obenchain wrote:
Hi Mike,

This is fixed in Rsamtools 1.15.35.

The bug was related to when the mate-pairing was performed wrt meeting
the 'yieldSize' requirement. Thanks for sending the file and
reproducible example.

The file has ~115 million records:

fl <- "wgEncodeCaltechRnaSeqGm12878R2x75Il200AlignsRep1V2.bam"
countBam(fl)$records
 [1] 114943975

To process the complete file with a yield size of 1e6 took ~ 18 GIG and
25 minutes. (ubuntu server, 16 processors, 387 GIG of ram)

bf <- BamFile(fl, yieldSize=1000000, asMates=TRUE)
grl <- exonsBy(TxDb.Hsapiens.UCSC.hg19.knownGene, by="gene")
SO <- function(x)
     summarizeOverlaps(grl, x, ignore.strand=TRUE, singleEnd=FALSE)

system.time(SO(bf))
     user   system  elapsed
 1545.684   12.412 1558.498

Thanks for reporting the bug.

Valerie



On 03/21/14 13:55, Michael Love wrote:
hi Valerie,

Thanks. I'm trying now to make use of the new mate pairing algorithm
but
keeping running out of memory (i requested 50 Gb) and getting my job
killed. I wonder if you could try this code/example below?

If the new C code is faster for paired-end than the
pre-sorting/obeyQname method that would be great (eliminates the
need to
have the extra qname-sorted Bam), but it seems to me that with the old
method, it was easier to specify hard limits on memory. Maybe I am
missing something though. :)

Here's an RNA-Seq sample from Encode, and then I run samtools index
locally.

from:
http://hgdownload.cse.ucsc.edu/goldenPath/hg19/encodeDCC/wgEncodeCaltechRnaSeq/




I download the hg19 paired end reads:
http://hgdownload.cse.ucsc.edu/goldenPath/hg19/encodeDCC/wgEncodeCaltechRnaSeq/wgEncodeCaltechRnaSeqGm12878R2x75Il200AlignsRep1V2.bam




library("GenomicFeatures")
library("TxDb.Hsapiens.UCSC.hg19.knownGene")
txdb <- TxDb.Hsapiens.UCSC.hg19.knownGene
grl <- exonsBy(txdb, by="gene")
library("Rsamtools")
bamFile <- "wgEncodeCaltechRnaSeqGm12878R2x75Il200AlignsRep1V2.bam"
bf <- BamFile(bamFile, yieldSize=1000000, asMates=TRUE)
library("GenomicAlignments")
system.time({so <- summarizeOverlaps(grl, bf,
                                      ignore.strand=TRUE,
                                      singleEnd=FALSE)})



 > sessionInfo()
R Under development (unstable) (2014-03-18 r65213)
Platform: x86_64-unknown-linux-gnu (64-bit)

locale:
  [1] LC_CTYPE=en_US.UTF-8       LC_NUMERIC=C
  [3] LC_TIME=en_US.UTF-8        LC_COLLATE=en_US.UTF-8
  [5] LC_MONETARY=en_US.UTF-8    LC_MESSAGES=en_US.UTF-8
  [7] LC_PAPER=en_US.UTF-8       LC_NAME=C
  [9] LC_ADDRESS=C               LC_TELEPHONE=C
[11] LC_MEASUREMENT=en_US.UTF-8 LC_IDENTIFICATION=C

attached base packages:
[1] parallel  stats     graphics  grDevices utils     datasets  methods
[8] base

other attached packages:
  [1] TxDb.Hsapiens.UCSC.hg19.knownGene_2.14.0
  [2] GenomicFeatures_1.15.11
  [3] AnnotationDbi_1.25.15
  [4] Biobase_2.23.6
  [5] GenomicAlignments_0.99.32
  [6] BSgenome_1.31.12
  [7] Rsamtools_1.15.33
  [8] Biostrings_2.31.14
  [9] XVector_0.3.7
[10] GenomicRanges_1.15.39
[11] GenomeInfoDb_0.99.19
[12] IRanges_1.21.34
[13] BiocGenerics_0.9.3
[14] Defaults_1.1-1
[15] devtools_1.4.1
[16] knitr_1.5
[17] BiocInstaller_1.13.3

loaded via a namespace (and not attached):
  [1] BatchJobs_1.2       BBmisc_1.5          BiocParallel_0.5.17
  [4] biomaRt_2.19.3      bitops_1.0-6        brew_1.0-6
  [7] codetools_0.2-8     DBI_0.2-7           digest_0.6.4
[10] evaluate_0.5.1      fail_1.2            foreach_1.4.1
[13] formatR_0.10        httr_0.2            iterators_1.0.6
[16] memoise_0.1         plyr_1.8.1          Rcpp_0.11.1
[19] RCurl_1.95-4.1      RSQLite_0.11.4      rtracklayer_1.23.18
[22] sendmailR_1.1-2     stats4_3.2.0        stringr_0.6.2
[25] tools_3.2.0         whisker_0.3-2       XML_3.98-1.1
[28] zlibbioc_1.9.0
 >





On Wed, Mar 19, 2014 at 2:00 PM, Valerie Obenchain <voben...@fhcrc.org
<mailto:voben...@fhcrc.org>> wrote:

    On 03/19/14 10:24, Michael Love wrote:

        hi Valerie,

        If the Bam is not sorted by name, isn't it possible that
        readGAlignment*
        will load > yieldSize number of reads in order to find the
mate?


    Sorry, our emails keep criss-crossing.

    Because the mate-pairing is now done in C yieldSize is no longer a
    constraint.

    When yieldSize is given, say 100, then 100 mates are returned. The
    algo moves through the file until 100 records are successfully
    paired. These 100 are yielded to the user and the code picks up
    where it left off. A related situation is the 'which' in a
param. In
    this case you want the mates in a particular range. The algo moves
    through the range and pairs what it can. If it's left with unmated
    records it goes outside the range looking for them.
    readGAlignmentsList() will return all records in this range, mated
    or not. The metadata column of 'mate_status' indicates the
different
    groupings.

    Valerie



        Mike


        On Wed, Mar 19, 2014 at 1:04 PM, Valerie Obenchain
        <voben...@fhcrc.org <mailto:voben...@fhcrc.org>
        <mailto:voben...@fhcrc.org <mailto:voben...@fhcrc.org>>> wrote:

             Hi Mike,

             You no longer need to sort Bam files to use the pairing
algo or
             yieldSize. The readGAlignment* functions now work with
both
             constraints out of the box.

             Create a BamFile with yieldSize and indicate you want
mates.
             bf <- BamFile(fl, yieldSize=10000, asMates=TRUE)

             Maybe set some specifications in a param:
             param <- ScanBamParam(what = c("qname", "flag"))

             Then call either readGAlignment* method that handles
pairs:
             readGAlignmentsList(bf, param=param)
             readGAlignmentPairs(bf, param=param)

             For summarizeOverlaps():
             summarizeOverlaps(annotation, bf, param=param,
singleEnd=FALSE)

             We've considered removing the 'obeyQname' arg and
        documentation but
             thought the concept may be useful in another application.
I'll
             revisit the summarizeOverlaps() documentation to make sure
             'obeyQname' is downplayed and 'asMates' is encouraged.

             Valerie




             On 03/19/14 07:39, Michael Love wrote:

                 hi,

                      From last year, in order to use yieldSize with
        paired-end
                     BAMs, I

                 should sort the BAMs by qname and then use the
        following call to
                 BamFile:

                 library(pasillaBamSubset)
                 fl <- sortBam(untreated3_chr4(), tempfile(),
byQname=TRUE)
                 bf <- BamFile(fl, index=character(0), yieldSize=3,
        obeyQname=TRUE)


https://stat.ethz.ch/____pipermail/bioconductor/2013-____March/051490.html




<https://stat.ethz.ch/__pipermail/bioconductor/2013-__March/051490.html>



<https://stat.ethz.ch/__pipermail/bioconductor/2013-__March/051490.html

<https://stat.ethz.ch/pipermail/bioconductor/2013-March/051490.html>>

                 If I want to use
        GenomicAlignments::____readGAlignmentsList with

                 asMates=TRUE and respecting the yieldSize, what is the
        proper
                 construction? (in the end, I want to use
        summarizeOverlaps on
                 paired-end BAMs while respecting the yieldSize)

                 library(pasillaBamSubset)
                 fl <- sortBam(untreated3_chr4(), tempfile(),
byQname=TRUE)
                 bf <- BamFile(fl, index=character(0), yieldSize=3,
                 obeyQname=TRUE, asMates=TRUE)
                 x <- readGAlignmentsList(bf)
                 Warning message:
                    In scanBam(bamfile, ..., param = param) :
                      'obeyQname=TRUE' ignored when 'asMates=TRUE'
                    Calls: readGAlignmentsList ... .matesFromBam ->
                 .load_bamcols_from_bamfile -> scanBam -> scanBam

                 I see in the man pages for summarizeOverlaps it has:

                 "In Bioconductor > 2.12 it is not
                 necessary to sort paired-end BAM files by ‘qname’.
When
                 counting with ‘summarizeOverlaps’, setting
        ‘singleEnd=FALSE’
                 will trigger paired-end reading and counting."

                 but I don't see how this can respect the specified
        yieldSize,
                 because
                 readGAlignmentsList has to read in as many reads as
        necessary to
                 find
                 the mate.

                 Sorry in advance if I am missing something in the
        documentation!

                 Mike

                 ___________________________________________________
        Bioc-devel@r-project.org <mailto:Bioc-devel@r-project.org>
        <mailto:Bioc-devel@r-project.__org
        <mailto:Bioc-devel@r-project.org>>
                 mailing list
        https://stat.ethz.ch/mailman/____listinfo/bioc-devel
        <https://stat.ethz.ch/mailman/__listinfo/bioc-devel>
                 <https://stat.ethz.ch/mailman/__listinfo/bioc-devel
        <https://stat.ethz.ch/mailman/listinfo/bioc-devel>>






_______________________________________________
Bioc-devel@r-project.org mailing list
https://stat.ethz.ch/mailman/listinfo/bioc-devel

_______________________________________________
Bioc-devel@r-project.org mailing list
https://stat.ethz.ch/mailman/listinfo/bioc-devel


_______________________________________________
Bioc-devel@r-project.org mailing list
https://stat.ethz.ch/mailman/listinfo/bioc-devel

Reply via email to