Wow

Author: cmdcolin

plugins/alignments/src/PileupRenderer/PileupRenderer.ts

@@ -1,7 +1,7 @@
 import { readConfObject } from '@jbrowse/core/configuration'
 import { getAdapter } from '@jbrowse/core/data_adapters/dataAdapterCache'
 import BoxRendererType from '@jbrowse/core/pluggableElementTypes/renderers/BoxRendererType'
-import { renderToAbstractCanvas } from '@jbrowse/core/util'
+import { renderToAbstractCanvas, updateStatus } from '@jbrowse/core/util'
 
 import { PileupLayoutSession } from './PileupLayoutSession'
 import { fetchSequence } from '../util'
@@ -53,39 +53,39 @@ export default class PileupRenderer extends BoxRendererType {
   async render(renderProps: RenderArgsDeserialized) {
     const features = await this.getFeatures(renderProps)
     const layout = this.createLayoutInWorker(renderProps)
-    const { colorBy, regions, bpPerPx } = renderProps
+    const { statusCallback = () => {}, colorBy, regions, bpPerPx } = renderProps
     const region = regions[0]!
-
-    const regionSequence =
-      colorBy?.type === 'methylation' && features.size
-        ? await this.fetchSequence(renderProps, region)
-        : undefined
-
+    const width = (region.end - region.start) / bpPerPx
     const { layoutRecords, height } = layoutFeats({
       ...renderProps,
       features,
       layout,
     })
-    const width = (region.end - region.start) / bpPerPx
-    const { makeImageData } = await import('./makeImageData')
 
-    const res = await renderToAbstractCanvas(
-      width,
-      height,
-      renderProps,
-      ctx => {
-        makeImageData({
-          ctx,
-          layoutRecords,
-          canvasWidth: width,
-          renderArgs: {
-            ...renderProps,
-            layout,
-            features,
-            regionSequence,
-          },
+    const res = await updateStatus(
+      'Rendering alignments',
+      statusCallback,
+      async () => {
+        const regionSequence =
+          colorBy?.type === 'methylation' && features.size
+            ? await this.fetchSequence(renderProps, region)
+            : undefined
+        const { makeImageData } = await import('./makeImageData')
+
+        return renderToAbstractCanvas(width, height, renderProps, ctx => {
+          makeImageData({
+            ctx,
+            layoutRecords,
+            canvasWidth: width,
+            renderArgs: {
+              ...renderProps,
+              layout,
+              features,
+              regionSequence,
+            },
+          })
+          return undefined
         })
-        return undefined
       },
     )
 

plugins/alignments/src/PileupRenderer/types.ts

@@ -20,6 +20,7 @@ export interface RenderArgsDeserialized extends BoxRenderArgsDeserialized {
   sortedBy?: SortedBy
   showSoftClip: boolean
   highResolutionScaling: number
+  statusCallback?: (arg: string) => void
 }
 
 export interface ProcessedRenderArgs extends RenderArgsDeserialized {

plugins/alignments/src/SNPCoverageRenderer/SNPCoverageRenderer.ts

@@ -1,6 +1,7 @@
 import { WiggleBaseRenderer } from '@jbrowse/plugin-wiggle'
 
 import type { RenderArgsDeserializedWithFeatures } from './types'
+import { updateStatus } from '@jbrowse/core/util'
 
 export default class SNPCoverageRenderer extends WiggleBaseRenderer {
   // note: the snps are drawn on linear scale even if the data is drawn in log
@@ -10,8 +11,11 @@ export default class SNPCoverageRenderer extends WiggleBaseRenderer {
     ctx: CanvasRenderingContext2D,
     props: RenderArgsDeserializedWithFeatures,
   ) {
+    const { statusCallback = () => {} } = props
     const { makeImage } = await import('./makeImage')
-    await makeImage(ctx, props)
+    await updateStatus('Rendering coverage', statusCallback, () =>
+      makeImage(ctx, props),
+    )
     return undefined
   }
 }

plugins/alignments/src/SNPCoverageRenderer/types.ts

@@ -16,5 +16,6 @@ export interface RenderArgsDeserializedWithFeatures
   ticks: { values: number[] }
   displayCrossHatches: boolean
   visibleModifications?: Record<string, ModificationTypeWithColor>
+  statusCallback?: (arg: string) => void
   colorBy: ColorBy
 }