@gmod/bbi 8.1.2 → 9.0.0

package/src/block-view.ts

@@ -1,7 +1,7 @@
 import AbortablePromiseCache from '@gmod/abortable-promise-cache'
 import QuickLRU from '@jbrowse/quick-lru'
 
-import Range from './range.ts'
+import { mergeRanges } from './range.ts'
 import {
   decompressAndParseBigWigBlocks,
   decompressAndParseSummaryBlocks,
@@ -12,7 +12,6 @@ import { groupBlocks } from './util.ts'
 import type { Feature } from './types.ts'
 import type { BigWigFeatureArrays, SummaryFeatureArrays } from './unzip.ts'
 import type { GenericFilehandle } from 'generic-filehandle2'
-import type { Observer } from 'rxjs'
 
 const decoder = new TextDecoder('utf8')
 const CIR_TREE_MAGIC = 0x2468ace0
@@ -37,6 +36,268 @@ function coordFilter(s1: number, e1: number, s2: number, e2: number): boolean {
   return s1 < e2 && e1 >= s2
 }
 
+function parseSummaryBlock(
+  b: Uint8Array,
+  startOffset: number,
+  request?: CoordRequest,
+) {
+  const features: Feature[] = []
+  let offset = startOffset
+  const dataView = new DataView(b.buffer, b.byteOffset, b.length)
+  while (offset < b.byteLength) {
+    const chromId = dataView.getUint32(offset, true)
+    offset += 4
+    const start = dataView.getUint32(offset, true)
+    offset += 4
+    const end = dataView.getUint32(offset, true)
+    offset += 4
+    const validCnt = dataView.getUint32(offset, true)
+    offset += 4
+    const minScore = dataView.getFloat32(offset, true)
+    offset += 4
+    const maxScore = dataView.getFloat32(offset, true)
+    offset += 4
+    const sumData = dataView.getFloat32(offset, true)
+    offset += 8
+
+    if (
+      !request ||
+      (chromId === request.chrId &&
+        coordFilter(start, end, request.start, request.end))
+    ) {
+      features.push({
+        start,
+        end,
+        maxScore,
+        minScore,
+        summary: true,
+        score: sumData / (validCnt || 1),
+      })
+    }
+  }
+  return features
+}
+
+function parseBigBedBlock(
+  data: Uint8Array,
+  startOffset: number,
+  offset: number,
+  request?: CoordRequest,
+) {
+  const items: Feature[] = []
+  let currOffset = startOffset
+  const dataView = new DataView(data.buffer, data.byteOffset, data.length)
+  while (currOffset < data.byteLength) {
+    const c2 = currOffset
+    const chromId = dataView.getUint32(currOffset, true)
+    currOffset += 4
+    const start = dataView.getInt32(currOffset, true)
+    currOffset += 4
+    const end = dataView.getInt32(currOffset, true)
+    currOffset += 4
+    let i = currOffset
+    for (; i < data.length; i++) {
+      if (data[i] === 0) {
+        break
+      }
+    }
+    const b = data.subarray(currOffset, i)
+    const rest = decoder.decode(b)
+    currOffset = i + 1
+    if (
+      !request ||
+      (chromId === request.chrId &&
+        coordFilter(start, end, request.start, request.end))
+    ) {
+      items.push({
+        start,
+        end,
+        rest,
+        uniqueId: `bb-${offset + c2}`,
+      })
+    }
+  }
+  return items
+}
+
+function parseBigWigBlock(
+  buffer: Uint8Array,
+  startOffset: number,
+  req?: CoordRequest,
+) {
+  const b = buffer.subarray(startOffset)
+  const dataView = new DataView(b.buffer, b.byteOffset, b.length)
+  const blockStart = dataView.getInt32(4, true)
+  const itemStep = dataView.getUint32(12, true)
+  const itemSpan = dataView.getUint32(16, true)
+  const blockType = dataView.getUint8(20)
+  const itemCount = dataView.getUint16(22, true)
+  let offset = 24
+  const items: Feature[] = []
+  switch (blockType) {
+    case 1: {
+      for (let i = 0; i < itemCount; i++) {
+        const start = dataView.getInt32(offset, true)
+        offset += 4
+        const end = dataView.getInt32(offset, true)
+        offset += 4
+        const score = dataView.getFloat32(offset, true)
+        offset += 4
+        if (!req || coordFilter(start, end, req.start, req.end)) {
+          items.push({ start, end, score })
+        }
+      }
+      break
+    }
+    case 2: {
+      for (let i = 0; i < itemCount; i++) {
+        const start = dataView.getInt32(offset, true)
+        offset += 4
+        const score = dataView.getFloat32(offset, true)
+        offset += 4
+        const end = start + itemSpan
+        if (!req || coordFilter(start, end, req.start, req.end)) {
+          items.push({ score, start, end })
+        }
+      }
+      break
+    }
+    case 3: {
+      for (let i = 0; i < itemCount; i++) {
+        const score = dataView.getFloat32(offset, true)
+        offset += 4
+        const start = blockStart + i * itemStep
+        const end = start + itemSpan
+        if (!req || coordFilter(start, end, req.start, req.end)) {
+          items.push({ score, start, end })
+        }
+      }
+      break
+    }
+  }
+  return items
+}
+
+function parseBigWigBlockAsArrays(
+  buffer: Uint8Array,
+  startOffset: number,
+  req?: CoordRequest,
+): { starts: Int32Array; ends: Int32Array; scores: Float32Array } {
+  const dataView = new DataView(
+    buffer.buffer,
+    buffer.byteOffset + startOffset,
+    buffer.length - startOffset,
+  )
+  const blockStart = dataView.getInt32(4, true)
+  const itemStep = dataView.getUint32(12, true)
+  const itemSpan = dataView.getUint32(16, true)
+  const blockType = dataView.getUint8(20)
+  const itemCount = dataView.getUint16(22, true)
+
+  const starts = new Int32Array(itemCount)
+  const ends = new Int32Array(itemCount)
+  const scores = new Float32Array(itemCount)
+
+  if (!req) {
+    switch (blockType) {
+      case 1: {
+        let offset = 24
+        for (let i = 0; i < itemCount; i++) {
+          starts[i] = dataView.getInt32(offset, true)
+          ends[i] = dataView.getInt32(offset + 4, true)
+          scores[i] = dataView.getFloat32(offset + 8, true)
+          offset += 12
+        }
+        return { starts, ends, scores }
+      }
+      case 2: {
+        let offset = 24
+        for (let i = 0; i < itemCount; i++) {
+          const start = dataView.getInt32(offset, true)
+          starts[i] = start
+          ends[i] = start + itemSpan
+          scores[i] = dataView.getFloat32(offset + 4, true)
+          offset += 8
+        }
+        return { starts, ends, scores }
+      }
+      case 3: {
+        let offset = 24
+        for (let i = 0; i < itemCount; i++) {
+          const start = blockStart + i * itemStep
+          starts[i] = start
+          ends[i] = start + itemSpan
+          scores[i] = dataView.getFloat32(offset, true)
+          offset += 4
+        }
+        return { starts, ends, scores }
+      }
+    }
+    return { starts, ends, scores }
+  }
+
+  const reqStart = req.start
+  const reqEnd = req.end
+  let idx = 0
+
+  switch (blockType) {
+    case 1: {
+      let offset = 24
+      for (let i = 0; i < itemCount; i++) {
+        const start = dataView.getInt32(offset, true)
+        const end = dataView.getInt32(offset + 4, true)
+        if (start < reqEnd && end >= reqStart) {
+          starts[idx] = start
+          ends[idx] = end
+          scores[idx] = dataView.getFloat32(offset + 8, true)
+          idx++
+        }
+        offset += 12
+      }
+      break
+    }
+    case 2: {
+      let offset = 24
+      for (let i = 0; i < itemCount; i++) {
+        const start = dataView.getInt32(offset, true)
+        const end = start + itemSpan
+        if (start < reqEnd && end >= reqStart) {
+          starts[idx] = start
+          ends[idx] = end
+          scores[idx] = dataView.getFloat32(offset + 4, true)
+          idx++
+        }
+        offset += 8
+      }
+      break
+    }
+    case 3: {
+      let offset = 24
+      for (let i = 0; i < itemCount; i++) {
+        const start = blockStart + i * itemStep
+        const end = start + itemSpan
+        if (start < reqEnd && end >= reqStart) {
+          starts[idx] = start
+          ends[idx] = end
+          scores[idx] = dataView.getFloat32(offset, true)
+          idx++
+        }
+        offset += 4
+      }
+      break
+    }
+  }
+
+  if (idx < itemCount) {
+    return {
+      starts: starts.subarray(0, idx),
+      ends: ends.subarray(0, idx),
+      scores: scores.subarray(0, idx),
+    }
+  }
+  return { starts, ends, scores }
+}
+
 /**
  * View into a subset of the data in a BigWig file.
  *
@@ -71,579 +332,216 @@ export class BlockView {
     }
   }
 
-  public async readWigData(
+  private async _collectBlocks(
     chrName: string,
     start: number,
     end: number,
-    observer: Observer<Feature[]>,
     opts?: Options,
-  ) {
-    try {
-      const chrId = this.refsByName[chrName]
-      if (chrId === undefined) {
-        observer.complete()
-        return
-      }
-      const request = { chrId, start, end }
-      if (!this.rTreePromise) {
-        this.rTreePromise = this.bbi.read(48, this.rTreeOffset, opts)
-      }
-      const buffer = await this.rTreePromise
-      const dataView = new DataView(
-        buffer.buffer,
-        buffer.byteOffset,
-        buffer.length,
+  ): Promise<{ blocks: ReadData[]; chrId: number } | undefined> {
+    const chrId = this.refsByName[chrName]
+    if (chrId === undefined) {
+      return undefined
+    }
+    if (!this.rTreePromise) {
+      this.rTreePromise = this.bbi.read(48, this.rTreeOffset, opts)
+    }
+    const buffer = await this.rTreePromise
+    const dataView = new DataView(
+      buffer.buffer,
+      buffer.byteOffset,
+      buffer.length,
+    )
+    const magic = dataView.getUint32(0, true)
+    if (magic !== CIR_TREE_MAGIC) {
+      throw new Error(
+        `invalid cirTree magic: 0x${magic.toString(16)} (expected 0x${CIR_TREE_MAGIC.toString(16)}) at offset ${this.rTreeOffset}, file may be corrupt or unsupported`,
       )
-      const magic = dataView.getUint32(0, true)
-      if (magic !== CIR_TREE_MAGIC) {
-        throw new Error(
-          `invalid cirTree magic: 0x${magic.toString(16)} (expected 0x${CIR_TREE_MAGIC.toString(16)}) at offset ${this.rTreeOffset}, file may be corrupt or unsupported`,
-        )
-      }
-      const rTreeBlockSize = dataView.getUint32(4, true)
-      const blocksToFetch: ReadData[] = []
-      let outstanding = 0
-
-      // R-tree leaf nodes contain the actual data blocks to fetch
-      const processLeafNode = (
-        dataView: DataView,
-        startOffset: number,
-        count: number,
-      ) => {
-        let offset = startOffset
-        for (let i = 0; i < count; i++) {
-          const startChrom = dataView.getUint32(offset, true)
-          offset += 4
-          const startBase = dataView.getUint32(offset, true)
-          offset += 4
-          const endChrom = dataView.getUint32(offset, true)
-          offset += 4
-          const endBase = dataView.getUint32(offset, true)
-          offset += 4
-          const blockOffset = Number(dataView.getBigUint64(offset, true))
-          offset += 8
-          const blockSize = Number(dataView.getBigUint64(offset, true))
-          offset += 8
-          if (
-            blockIntersectsQuery({ startChrom, startBase, endBase, endChrom })
-          ) {
-            blocksToFetch.push({
-              offset: blockOffset,
-              length: blockSize,
-            })
-          }
-        }
-      }
-
-      // R-tree non-leaf nodes contain pointers to child nodes
-      const processNonLeafNode = (
-        dataView: DataView,
-        startOffset: number,
-        count: number,
-        level: number,
-      ) => {
-        const recurOffsets = []
-        let offset = startOffset
-        for (let i = 0; i < count; i++) {
-          const startChrom = dataView.getUint32(offset, true)
-          offset += 4
-          const startBase = dataView.getUint32(offset, true)
-          offset += 4
-          const endChrom = dataView.getUint32(offset, true)
-          offset += 4
-          const endBase = dataView.getUint32(offset, true)
-          offset += 4
-          const blockOffset = Number(dataView.getBigUint64(offset, true))
-          offset += 8
-          if (
-            blockIntersectsQuery({ startChrom, startBase, endChrom, endBase })
-          ) {
-            recurOffsets.push(blockOffset)
-          }
-        }
-        if (recurOffsets.length > 0) {
-          traverseRTree(recurOffsets, level + 1)
-        }
-      }
-
-      const processRTreeNode = (
-        rTreeBlockData: Uint8Array,
-        offset2: number,
-        level: number,
-      ) => {
-        try {
-          const data = rTreeBlockData.subarray(offset2)
-          const dataView = new DataView(
-            data.buffer,
-            data.byteOffset,
-            data.length,
-          )
-          let offset = 0
-
-          const isLeaf = dataView.getUint8(offset)
-          offset += 2 // 1 skip for reserved byte
-          const count = dataView.getUint16(offset, true)
-          offset += 2
-
-          if (isLeaf === 1) {
-            processLeafNode(dataView, offset, count)
-          } else if (isLeaf === 0) {
-            processNonLeafNode(dataView, offset, count, level)
-          }
-        } catch (e) {
-          observer.error(e)
-        }
-      }
-
-      const blockIntersectsQuery = (b: {
-        startChrom: number
-        startBase: number
-        endChrom: number
-        endBase: number
-      }) => {
-        const { startChrom, startBase, endChrom, endBase } = b
-        return (
-          (startChrom < chrId || (startChrom === chrId && startBase <= end)) &&
-          (endChrom > chrId || (endChrom === chrId && endBase >= start))
+    }
+    const rTreeBlockSize = dataView.getUint32(4, true)
+    // Upper bound on size, based on a completely full leaf node.
+    const maxRTreeBlockSpan = 4 + rTreeBlockSize * 32
+
+    const blockIntersectsQuery = (
+      startChrom: number,
+      startBase: number,
+      endChrom: number,
+      endBase: number,
+    ) =>
+      (startChrom < chrId || (startChrom === chrId && startBase <= end)) &&
+      (endChrom > chrId || (endChrom === chrId && endBase >= start))
+
+    const blocks: ReadData[] = []
+    let currentOffsets = [this.rTreeOffset + 48]
+
+    while (currentOffsets.length > 0) {
+      const spans = mergeRanges(
+        currentOffsets.map(o => ({ min: o, max: o + maxRTreeBlockSpan })),
+      )
+      const nextOffsets: number[] = []
+      for (const { min, max } of spans) {
+        const length = max - min
+        const offset = min
+        const resultBuffer = await this.featureCache.get(
+          `${length}_${offset}`,
+          { length, offset },
+          opts?.signal,
         )
-      }
-
-      const fetchAndProcessRTreeBlocks = async (
-        offsets: number[],
-        range: Range,
-        level: number,
-      ) => {
-        try {
-          const length = range.max - range.min
-          const offset = range.min
-          const resultBuffer = await this.featureCache.get(
-            `${length}_${offset}`,
-            { length, offset },
-            opts?.signal,
-          )
-          for (const element of offsets) {
-            if (range.contains(element)) {
-              processRTreeNode(resultBuffer, element - offset, level)
-              outstanding -= 1
-              if (outstanding === 0) {
-                this.readFeatures(observer, blocksToFetch, {
-                  ...opts,
-                  request,
-                }).catch((e: unknown) => {
-                  observer.error(e)
-                })
+        for (const element of currentOffsets) {
+          if (min <= element && element <= max) {
+            const data = resultBuffer.subarray(element - offset)
+            const dv = new DataView(data.buffer, data.byteOffset, data.length)
+            const isLeaf = dv.getUint8(0)
+            const count = dv.getUint16(2, true)
+            let nodeOffset = 4
+            if (isLeaf === 1) {
+              for (let i = 0; i < count; i++) {
+                const startChrom = dv.getUint32(nodeOffset, true)
+                const startBase = dv.getUint32(nodeOffset + 4, true)
+                const endChrom = dv.getUint32(nodeOffset + 8, true)
+                const endBase = dv.getUint32(nodeOffset + 12, true)
+                const blockOffset = Number(
+                  dv.getBigUint64(nodeOffset + 16, true),
+                )
+                const blockSize = Number(dv.getBigUint64(nodeOffset + 24, true))
+                nodeOffset += 32
+                if (
+                  blockIntersectsQuery(startChrom, startBase, endChrom, endBase)
+                ) {
+                  blocks.push({ offset: blockOffset, length: blockSize })
+                }
+              }
+            } else if (isLeaf === 0) {
+              for (let i = 0; i < count; i++) {
+                const startChrom = dv.getUint32(nodeOffset, true)
+                const startBase = dv.getUint32(nodeOffset + 4, true)
+                const endChrom = dv.getUint32(nodeOffset + 8, true)
+                const endBase = dv.getUint32(nodeOffset + 12, true)
+                const childOffset = Number(
+                  dv.getBigUint64(nodeOffset + 16, true),
+                )
+                nodeOffset += 24
+                if (
+                  blockIntersectsQuery(startChrom, startBase, endChrom, endBase)
+                ) {
+                  nextOffsets.push(childOffset)
+                }
               }
             }
           }
-        } catch (e) {
-          observer.error(e)
-        }
-      }
-      const traverseRTree = (offsets: number[], level: number) => {
-        try {
-          outstanding += offsets.length
-
-          // Upper bound on size, based on a completely full leaf node.
-          const maxRTreeBlockSpan = 4 + rTreeBlockSize * 32
-          let spans = new Range([
-            {
-              min: offsets[0]!,
-              max: offsets[0]! + maxRTreeBlockSpan,
-            },
-          ])
-          for (let i = 1; i < offsets.length; i += 1) {
-            const blockSpan = new Range([
-              {
-                min: offsets[i]!,
-                max: offsets[i]! + maxRTreeBlockSpan,
-              },
-            ])
-            spans = spans.union(blockSpan)
-          }
-          spans.getRanges().forEach(range => {
-            fetchAndProcessRTreeBlocks(offsets, range, level).catch(
-              (e: unknown) => {
-                observer.error(e)
-              },
-            )
-          })
-        } catch (e) {
-          observer.error(e)
         }
       }
-
-      traverseRTree([this.rTreeOffset + 48], 1)
-      return
-    } catch (e) {
-      observer.error(e)
+      currentOffsets = nextOffsets
     }
-  }
 
-  private parseSummaryBlock(
-    b: Uint8Array,
-    startOffset: number,
-    request?: CoordRequest,
-  ) {
-    const features: Feature[] = []
-    let offset = startOffset
-
-    const dataView = new DataView(b.buffer, b.byteOffset, b.length)
-    while (offset < b.byteLength) {
-      const chromId = dataView.getUint32(offset, true)
-      offset += 4
-      const start = dataView.getUint32(offset, true)
-      offset += 4
-      const end = dataView.getUint32(offset, true)
-      offset += 4
-      const validCnt = dataView.getUint32(offset, true)
-      offset += 4
-      const minScore = dataView.getFloat32(offset, true)
-      offset += 4
-      const maxScore = dataView.getFloat32(offset, true)
-      offset += 4
-      const sumData = dataView.getFloat32(offset, true)
-      offset += 8
-
-      if (
-        !request ||
-        (chromId === request.chrId &&
-          coordFilter(start, end, request.start, request.end))
-      ) {
-        features.push({
-          start,
-          end,
-          maxScore,
-          minScore,
-          summary: true,
-          score: sumData / (validCnt || 1),
-        })
-      }
-    }
-
-    return features
+    return { blocks, chrId }
   }
 
-  private parseBigBedBlock(
-    data: Uint8Array,
-    startOffset: number,
-    offset: number,
-    request?: CoordRequest,
-  ) {
-    const items: Feature[] = []
-    let currOffset = startOffset
-    const dataView = new DataView(data.buffer, data.byteOffset, data.length)
-    while (currOffset < data.byteLength) {
-      const c2 = currOffset
-      const chromId = dataView.getUint32(currOffset, true)
-      currOffset += 4
-      const start = dataView.getInt32(currOffset, true)
-      currOffset += 4
-      const end = dataView.getInt32(currOffset, true)
-      currOffset += 4
-      let i = currOffset
-      for (; i < data.length; i++) {
-        if (data[i] === 0) {
-          break
-        }
-      }
-      const b = data.subarray(currOffset, i)
-      const rest = decoder.decode(b)
-      currOffset = i + 1
-      if (
-        !request ||
-        (chromId === request.chrId &&
-          coordFilter(start, end, request.start, request.end))
-      ) {
-        items.push({
-          start,
-          end,
-          rest,
-          uniqueId: `bb-${offset + c2}`,
-        })
-      }
+  public async readWigData(
+    chrName: string,
+    start: number,
+    end: number,
+    opts?: Options,
+  ): Promise<Feature[]> {
+    const collected = await this._collectBlocks(chrName, start, end, opts)
+    if (!collected) {
+      return []
     }
-
-    return items
+    const { blocks, chrId } = collected
+    return this.readFeatures(blocks, {
+      ...opts,
+      request: { chrId, start, end },
+    })
   }
 
-  private parseBigWigBlock(
-    buffer: Uint8Array,
-    startOffset: number,
-    req?: CoordRequest,
-  ) {
-    const b = buffer.subarray(startOffset)
-
-    const dataView = new DataView(b.buffer, b.byteOffset, b.length)
-    let offset = 0
-    offset += 4
-    const blockStart = dataView.getInt32(offset, true)
-    offset += 8
-    const itemStep = dataView.getUint32(offset, true)
-    offset += 4
-    const itemSpan = dataView.getUint32(offset, true)
-    offset += 4
-    const blockType = dataView.getUint8(offset)
-    offset += 2
-    const itemCount = dataView.getUint16(offset, true)
-    offset += 2
-    const items = []
-    switch (blockType) {
-      case 1: {
-        for (let i = 0; i < itemCount; i++) {
-          const start = dataView.getInt32(offset, true)
-          offset += 4
-          const end = dataView.getInt32(offset, true)
-          offset += 4
-          const score = dataView.getFloat32(offset, true)
-          offset += 4
-          if (!req || coordFilter(start, end, req.start, req.end)) {
-            items.push({
-              start,
-              end,
-              score,
-            })
-          }
-        }
-        break
-      }
-      case 2: {
-        for (let i = 0; i < itemCount; i++) {
-          const start = dataView.getInt32(offset, true)
-          offset += 4
-          const score = dataView.getFloat32(offset, true)
-          offset += 4
-          const end = start + itemSpan
-          if (!req || coordFilter(start, end, req.start, req.end)) {
-            items.push({
-              score,
-              start,
-              end,
-            })
-          }
-        }
-        break
-      }
-      case 3: {
-        for (let i = 0; i < itemCount; i++) {
-          const score = dataView.getFloat32(offset, true)
-          offset += 4
-          const start = blockStart + i * itemStep
-          const end = start + itemSpan
-          if (!req || coordFilter(start, end, req.start, req.end)) {
-            items.push({
-              score,
-              start,
-              end,
-            })
-          }
-        }
-        break
-      }
+  public async readWigDataAsArrays(
+    chrName: string,
+    start: number,
+    end: number,
+    opts?: Options,
+  ): Promise<BigWigFeatureArrays | SummaryFeatureArrays> {
+    const collected = await this._collectBlocks(chrName, start, end, opts)
+    const blocks = collected?.blocks ?? []
+    const request = collected
+      ? { chrId: collected.chrId, start, end }
+      : undefined
+    const optsWithReq = { ...opts, request }
+    if (this.blockType === 'summary') {
+      return this._readSummaryFeaturesAsArrays(blocks, optsWithReq)
     }
-
-    return items
+    return this._readBigWigFeaturesAsArrays(blocks, optsWithReq)
   }
 
-  private parseBigWigBlockAsArrays(
-    buffer: Uint8Array,
-    startOffset: number,
-    req?: CoordRequest,
-  ): { starts: Int32Array; ends: Int32Array; scores: Float32Array } {
-    const dataView = new DataView(
-      buffer.buffer,
-      buffer.byteOffset + startOffset,
-      buffer.length - startOffset,
-    )
-    const blockStart = dataView.getInt32(4, true)
-    const itemStep = dataView.getUint32(12, true)
-    const itemSpan = dataView.getUint32(16, true)
-    const blockType = dataView.getUint8(20)
-    const itemCount = dataView.getUint16(22, true)
-
-    const starts = new Int32Array(itemCount)
-    const ends = new Int32Array(itemCount)
-    const scores = new Float32Array(itemCount)
-
-    if (!req) {
-      switch (blockType) {
-        case 1: {
-          let offset = 24
-          for (let i = 0; i < itemCount; i++) {
-            starts[i] = dataView.getInt32(offset, true)
-            ends[i] = dataView.getInt32(offset + 4, true)
-            scores[i] = dataView.getFloat32(offset + 8, true)
-            offset += 12
-          }
-          return { starts, ends, scores }
-        }
-        case 2: {
-          let offset = 24
-          for (let i = 0; i < itemCount; i++) {
-            const start = dataView.getInt32(offset, true)
-            starts[i] = start
-            ends[i] = start + itemSpan
-            scores[i] = dataView.getFloat32(offset + 4, true)
-            offset += 8
-          }
-          return { starts, ends, scores }
+  public async readFeatures(
+    blocks: { offset: number; length: number }[],
+    opts: Options = {},
+  ): Promise<Feature[]> {
+    const { blockType, uncompressBufSize } = this
+    const { signal, request } = opts
+    const blockGroupsToFetch = groupBlocks(blocks)
+    const allFeatures: Feature[] = []
+    for (const blockGroup of blockGroupsToFetch) {
+      const data = await this.bbi.read(blockGroup.length, blockGroup.offset, {
+        signal,
+      })
+      const groupOffset = blockGroup.offset
+      const subBlocks = blockGroup.blocks
+
+      let decompressedData: Uint8Array
+      let decompressedOffsets: number[]
+
+      if (uncompressBufSize > 0) {
+        const localBlocks: { offset: number; length: number }[] = []
+        for (const block of subBlocks) {
+          localBlocks.push({
+            offset: block.offset - groupOffset,
+            length: block.length,
+          })
         }
-        case 3: {
-          let offset = 24
-          for (let i = 0; i < itemCount; i++) {
-            const start = blockStart + i * itemStep
-            starts[i] = start
-            ends[i] = start + itemSpan
-            scores[i] = dataView.getFloat32(offset, true)
-            offset += 4
-          }
-          return { starts, ends, scores }
+        const result = await unzipBatch(data, localBlocks, uncompressBufSize)
+        decompressedData = result.data
+        decompressedOffsets = result.offsets
+      } else {
+        decompressedData = data
+        decompressedOffsets = []
+        for (const block of subBlocks) {
+          decompressedOffsets.push(block.offset - groupOffset)
         }
+        decompressedOffsets.push(data.length)
       }
-      return { starts, ends, scores }
-    }
-
-    const reqStart = req.start
-    const reqEnd = req.end
-    let idx = 0
 
-    switch (blockType) {
-      case 1: {
-        let offset = 24
-        for (let i = 0; i < itemCount; i++) {
-          const start = dataView.getInt32(offset, true)
-          const end = dataView.getInt32(offset + 4, true)
-          if (start < reqEnd && end >= reqStart) {
-            starts[idx] = start
-            ends[idx] = end
-            scores[idx] = dataView.getFloat32(offset + 8, true)
-            idx++
-          }
-          offset += 12
-        }
-        break
-      }
-      case 2: {
-        let offset = 24
-        for (let i = 0; i < itemCount; i++) {
-          const start = dataView.getInt32(offset, true)
-          const end = start + itemSpan
-          if (start < reqEnd && end >= reqStart) {
-            starts[idx] = start
-            ends[idx] = end
-            scores[idx] = dataView.getFloat32(offset + 4, true)
-            idx++
-          }
-          offset += 8
+      for (let i = 0; i < subBlocks.length; i++) {
+        const start = decompressedOffsets[i]!
+        const end = decompressedOffsets[i + 1]!
+        const resultData = decompressedData.subarray(start, end)
+        let features: Feature[]
+        switch (blockType) {
+          case 'summary':
+            features = parseSummaryBlock(resultData, 0, request)
+            break
+          case 'bigwig':
+            features = parseBigWigBlock(resultData, 0, request)
+            break
+          case 'bigbed':
+            features = parseBigBedBlock(
+              resultData,
+              0,
+              subBlocks[i]!.offset * (1 << 8),
+              request,
+            )
+            break
+          default:
+            features = []
+            console.warn(`Don't know what to do with ${blockType}`)
         }
-        break
-      }
-      case 3: {
-        let offset = 24
-        for (let i = 0; i < itemCount; i++) {
-          const start = blockStart + i * itemStep
-          const end = start + itemSpan
-          if (start < reqEnd && end >= reqStart) {
-            starts[idx] = start
-            ends[idx] = end
-            scores[idx] = dataView.getFloat32(offset, true)
-            idx++
-          }
-          offset += 4
+        for (const f of features) {
+          allFeatures.push(f)
         }
-        break
-      }
-    }
-
-    if (idx < itemCount) {
-      return {
-        starts: starts.subarray(0, idx),
-        ends: ends.subarray(0, idx),
-        scores: scores.subarray(0, idx),
       }
     }
-    return { starts, ends, scores }
-  }
-
-  public async readFeatures(
-    observer: Observer<Feature[]>,
-    blocks: { offset: number; length: number }[],
-    opts: Options = {},
-  ) {
-    try {
-      const { blockType, uncompressBufSize } = this
-      const { signal, request } = opts
-      const blockGroupsToFetch = groupBlocks(blocks)
-      await Promise.all(
-        blockGroupsToFetch.map(async blockGroup => {
-          const { length, offset } = blockGroup
-          const data = await this.featureCache.get(
-            `${length}_${offset}`,
-            blockGroup,
-            signal,
-          )
-
-          const localBlocks = blockGroup.blocks.map(block => ({
-            offset: block.offset - blockGroup.offset,
-            length: block.length,
-          }))
-
-          let decompressedData: Uint8Array
-          let decompressedOffsets: number[]
-
-          if (uncompressBufSize > 0) {
-            const result = await unzipBatch(
-              data,
-              localBlocks,
-              uncompressBufSize,
-            )
-            decompressedData = result.data
-            decompressedOffsets = result.offsets
-          } else {
-            decompressedData = data
-            decompressedOffsets = localBlocks.map(b => b.offset)
-            decompressedOffsets.push(data.length)
-          }
-
-          for (let i = 0; i < blockGroup.blocks.length; i++) {
-            const block = blockGroup.blocks[i]!
-            const start = decompressedOffsets[i]!
-            const end = decompressedOffsets[i + 1]!
-            const resultData = decompressedData.subarray(start, end)
-
-            switch (blockType) {
-              case 'summary': {
-                observer.next(this.parseSummaryBlock(resultData, 0, request))
-                break
-              }
-              case 'bigwig': {
-                observer.next(this.parseBigWigBlock(resultData, 0, request))
-                break
-              }
-              case 'bigbed': {
-                observer.next(
-                  this.parseBigBedBlock(
-                    resultData,
-                    0,
-                    block.offset * (1 << 8),
-                    request,
-                  ),
-                )
-                break
-              }
-              default: {
-                console.warn(`Don't know what to do with ${blockType}`)
-              }
-            }
-          }
-        }),
-      )
-      observer.complete()
-    } catch (e) {
-      observer.error(e)
-    }
+    return allFeatures
   }
 
-  public async readBigWigFeaturesAsArrays(
+  private async _readBigWigFeaturesAsArrays(
     blocks: { offset: number; length: number }[],
     opts: Options = {},
   ): Promise<BigWigFeatureArrays> {
@@ -656,51 +554,45 @@ export class BlockView {
     const allScores: Float32Array[] = []
     let totalCount = 0
 
-    await Promise.all(
-      blockGroupsToFetch.map(async blockGroup => {
-        const { length, offset } = blockGroup
-        const data = await this.featureCache.get(
-          `${length}_${offset}`,
-          blockGroup,
-          signal,
+    for (const blockGroup of blockGroupsToFetch) {
+      const { length, offset } = blockGroup
+      const data = await this.bbi.read(length, offset, { signal })
+
+      const localBlocks = blockGroup.blocks.map(block => ({
+        offset: block.offset - blockGroup.offset,
+        length: block.length,
+      }))
+
+      if (uncompressBufSize > 0) {
+        const result = await decompressAndParseBigWigBlocks(
+          data,
+          localBlocks,
+          uncompressBufSize,
+          request?.start ?? 0,
+          request?.end ?? 0,
         )
-
-        const localBlocks = blockGroup.blocks.map(block => ({
-          offset: block.offset - blockGroup.offset,
-          length: block.length,
-        }))
-
-        if (uncompressBufSize > 0) {
-          const result = await decompressAndParseBigWigBlocks(
-            data,
-            localBlocks,
-            uncompressBufSize,
-            request?.start ?? 0,
-            request?.end ?? 0,
+        if (result.starts.length > 0) {
+          allStarts.push(result.starts)
+          allEnds.push(result.ends)
+          allScores.push(result.scores)
+          totalCount += result.starts.length
+        }
+      } else {
+        for (const block of localBlocks) {
+          const blockData = data.subarray(
+            block.offset,
+            block.offset + block.length,
           )
+          const result = parseBigWigBlockAsArrays(blockData, 0, request)
           if (result.starts.length > 0) {
             allStarts.push(result.starts)
             allEnds.push(result.ends)
             allScores.push(result.scores)
             totalCount += result.starts.length
           }
-        } else {
-          for (const block of localBlocks) {
-            const blockData = data.subarray(
-              block.offset,
-              block.offset + block.length,
-            )
-            const result = this.parseBigWigBlockAsArrays(blockData, 0, request)
-            if (result.starts.length > 0) {
-              allStarts.push(result.starts)
-              allEnds.push(result.ends)
-              allScores.push(result.scores)
-              totalCount += result.starts.length
-            }
-          }
         }
-      }),
-    )
+      }
+    }
 
     if (allStarts.length === 0) {
       return {
@@ -734,7 +626,7 @@ export class BlockView {
     return { starts, ends, scores, isSummary: false as const }
   }
 
-  public async readSummaryFeaturesAsArrays(
+  private async _readSummaryFeaturesAsArrays(
     blocks: { offset: number; length: number }[],
     opts: Options = {},
   ): Promise<SummaryFeatureArrays> {
@@ -749,69 +641,63 @@ export class BlockView {
     const allMaxScores: Float32Array[] = []
     let totalCount = 0
 
-    await Promise.all(
-      blockGroupsToFetch.map(async blockGroup => {
-        const { length, offset } = blockGroup
-        const data = await this.featureCache.get(
-          `${length}_${offset}`,
-          blockGroup,
-          signal,
+    for (const blockGroup of blockGroupsToFetch) {
+      const { length, offset } = blockGroup
+      const data = await this.bbi.read(length, offset, { signal })
+
+      const localBlocks = blockGroup.blocks.map(block => ({
+        offset: block.offset - blockGroup.offset,
+        length: block.length,
+      }))
+
+      if (uncompressBufSize > 0) {
+        const result = await decompressAndParseSummaryBlocks(
+          data,
+          localBlocks,
+          uncompressBufSize,
+          request?.chrId ?? 0,
+          request?.start ?? 0,
+          request?.end ?? 0,
         )
-
-        const localBlocks = blockGroup.blocks.map(block => ({
-          offset: block.offset - blockGroup.offset,
-          length: block.length,
-        }))
-
-        if (uncompressBufSize > 0) {
-          const result = await decompressAndParseSummaryBlocks(
-            data,
-            localBlocks,
-            uncompressBufSize,
-            request?.chrId ?? 0,
-            request?.start ?? 0,
-            request?.end ?? 0,
+        if (result.starts.length > 0) {
+          allStarts.push(result.starts)
+          allEnds.push(result.ends)
+          allScores.push(result.scores)
+          allMinScores.push(result.minScores)
+          allMaxScores.push(result.maxScores)
+          totalCount += result.starts.length
+        }
+      } else {
+        for (const block of localBlocks) {
+          const blockData = data.subarray(
+            block.offset,
+            block.offset + block.length,
           )
-          if (result.starts.length > 0) {
-            allStarts.push(result.starts)
-            allEnds.push(result.ends)
-            allScores.push(result.scores)
-            allMinScores.push(result.minScores)
-            allMaxScores.push(result.maxScores)
-            totalCount += result.starts.length
-          }
-        } else {
-          for (const block of localBlocks) {
-            const blockData = data.subarray(
-              block.offset,
-              block.offset + block.length,
-            )
-            const features = this.parseSummaryBlock(blockData, 0, request)
-            if (features.length > 0) {
-              const starts = new Int32Array(features.length)
-              const ends = new Int32Array(features.length)
-              const scores = new Float32Array(features.length)
-              const minScores = new Float32Array(features.length)
-              const maxScores = new Float32Array(features.length)
-              for (let i = 0; i < features.length; i++) {
-                const f = features[i]!
-                starts[i] = f.start
-                ends[i] = f.end
-                scores[i] = f.score ?? 0
-                minScores[i] = f.minScore ?? 0
-                maxScores[i] = f.maxScore ?? 0
-              }
-              allStarts.push(starts)
-              allEnds.push(ends)
-              allScores.push(scores)
-              allMinScores.push(minScores)
-              allMaxScores.push(maxScores)
-              totalCount += features.length
+          const features = parseSummaryBlock(blockData, 0, request)
+          if (features.length > 0) {
+            const starts = new Int32Array(features.length)
+            const ends = new Int32Array(features.length)
+            const scores = new Float32Array(features.length)
+            const minScores = new Float32Array(features.length)
+            const maxScores = new Float32Array(features.length)
+            for (let i = 0; i < features.length; i++) {
+              const f = features[i]!
+              starts[i] = f.start
+              ends[i] = f.end
+              scores[i] = f.score ?? 0
+              minScores[i] = f.minScore ?? 0
+              maxScores[i] = f.maxScore ?? 0
             }
+            allStarts.push(starts)
+            allEnds.push(ends)
+            allScores.push(scores)
+            allMinScores.push(minScores)
+            allMaxScores.push(maxScores)
+            totalCount += features.length
           }
         }
-      }),
-    )
+      }
+    }
 
     if (allStarts.length === 0) {
       return {