@jbrowse/plugin-wiggle 2.4.1 → 2.5.0

package/src/WiggleRPC/WiggleGetMultiRegionQuantitativeStats.ts

@@ -0,0 +1,73 @@
+import RpcMethodType from '@jbrowse/core/pluggableElementTypes/RpcMethodType'
+import SerializableFilterChain from '@jbrowse/core/pluggableElementTypes/renderers/util/serializableFilterChain'
+import { RenderArgs } from '@jbrowse/core/rpc/coreRpcMethods'
+import { getAdapter } from '@jbrowse/core/data_adapters/dataAdapterCache'
+import { RemoteAbortSignal } from '@jbrowse/core/rpc/remoteAbortSignals'
+import { Region, renameRegionsIfNeeded } from '@jbrowse/core/util'
+import { BaseFeatureDataAdapter } from '@jbrowse/core/data_adapters/BaseAdapter'
+
+export class WiggleGetMultiRegionQuantitativeStats extends RpcMethodType {
+  name = 'WiggleGetMultiRegionQuantitativeStats'
+
+  // eslint-disable-next-line @typescript-eslint/no-explicit-any
+  async deserializeArguments(args: any, rpcDriverClassName: string) {
+    const l = await super.deserializeArguments(args, rpcDriverClassName)
+    return {
+      ...l,
+      filters: args.filters
+        ? new SerializableFilterChain({
+            filters: args.filters,
+          })
+        : undefined,
+    }
+  }
+
+  async serializeArguments(
+    args: RenderArgs & {
+      signal?: AbortSignal
+      statusCallback?: (arg: string) => void
+    },
+    rpcDriverClassName: string,
+  ) {
+    const pm = this.pluginManager
+    const assemblyManager = pm.rootModel?.session?.assemblyManager
+    if (!assemblyManager) {
+      return args
+    }
+
+    const renamedArgs = await renameRegionsIfNeeded(assemblyManager, {
+      ...args,
+      filters: args.filters?.toJSON().filters,
+    })
+
+    return super.serializeArguments(renamedArgs, rpcDriverClassName)
+  }
+
+  async execute(
+    args: {
+      adapterConfig: {}
+      signal?: RemoteAbortSignal
+      sessionId: string
+      headers?: Record<string, string>
+      regions: Region[]
+      bpPerPx: number
+    },
+    rpcDriverClassName: string,
+  ) {
+    const pm = this.pluginManager
+    const deserializedArgs = await this.deserializeArguments(
+      args,
+      rpcDriverClassName,
+    )
+    const { regions, adapterConfig, sessionId } = deserializedArgs
+    const { dataAdapter } = await getAdapter(pm, sessionId, adapterConfig)
+
+    if (dataAdapter instanceof BaseFeatureDataAdapter) {
+      return dataAdapter.getMultiRegionQuantitativeStats(
+        regions,
+        deserializedArgs,
+      )
+    }
+    throw new Error('Data adapter not found')
+  }
+}

package/src/WiggleRPC/rpcMethods.ts

@@ -1,170 +1,3 @@
-import RpcMethodType from '@jbrowse/core/pluggableElementTypes/RpcMethodType'
-import SerializableFilterChain from '@jbrowse/core/pluggableElementTypes/renderers/util/serializableFilterChain'
-import { getAdapter } from '@jbrowse/core/data_adapters/dataAdapterCache'
-import { RenderArgs } from '@jbrowse/core/rpc/coreRpcMethods'
-import { renameRegionsIfNeeded, Region } from '@jbrowse/core/util'
-import { RemoteAbortSignal } from '@jbrowse/core/rpc/remoteAbortSignals'
-import { BaseFeatureDataAdapter } from '@jbrowse/core/data_adapters/BaseAdapter'
-import { FeatureStats } from '@jbrowse/core/util/stats'
-
-export class WiggleGetGlobalStats extends RpcMethodType {
-  name = 'WiggleGetGlobalStats'
-
-  // eslint-disable-next-line @typescript-eslint/no-explicit-any
-  async deserializeArguments(args: any, rpcDriverClassName: string) {
-    const l = await super.deserializeArguments(args, rpcDriverClassName)
-    return {
-      ...l,
-      filters: args.filters
-        ? new SerializableFilterChain({
-            filters: args.filters,
-          })
-        : undefined,
-    }
-  }
-
-  async execute(
-    args: {
-      adapterConfig: {}
-      signal?: RemoteAbortSignal
-      headers?: Record<string, string>
-      sessionId: string
-    },
-    rpcDriverClassName: string,
-  ): Promise<FeatureStats> {
-    const pm = this.pluginManager
-    const deserializedArgs = await this.deserializeArguments(
-      args,
-      rpcDriverClassName,
-    )
-    const { adapterConfig, sessionId } = deserializedArgs
-    const { dataAdapter } = await getAdapter(pm, sessionId, adapterConfig)
-
-    // @ts-expect-error
-    return dataAdapter.getGlobalStats(deserializedArgs)
-  }
-}
-
-export class WiggleGetMultiRegionStats extends RpcMethodType {
-  name = 'WiggleGetMultiRegionStats'
-
-  // eslint-disable-next-line @typescript-eslint/no-explicit-any
-  async deserializeArguments(args: any, rpcDriverClassName: string) {
-    const l = await super.deserializeArguments(args, rpcDriverClassName)
-    return {
-      ...l,
-      filters: args.filters
-        ? new SerializableFilterChain({
-            filters: args.filters,
-          })
-        : undefined,
-    }
-  }
-
-  async serializeArguments(
-    args: RenderArgs & {
-      signal?: AbortSignal
-      statusCallback?: (arg: string) => void
-    },
-    rpcDriverClassName: string,
-  ) {
-    const pm = this.pluginManager
-    const assemblyManager = pm.rootModel?.session?.assemblyManager
-    if (!assemblyManager) {
-      return args
-    }
-
-    const renamedArgs = await renameRegionsIfNeeded(assemblyManager, {
-      ...args,
-      filters: args.filters?.toJSON().filters,
-    })
-
-    return super.serializeArguments(renamedArgs, rpcDriverClassName)
-  }
-
-  async execute(
-    args: {
-      adapterConfig: {}
-      signal?: RemoteAbortSignal
-      sessionId: string
-      headers?: Record<string, string>
-      regions: Region[]
-      bpPerPx: number
-    },
-    rpcDriverClassName: string,
-  ) {
-    const pm = this.pluginManager
-    const deserializedArgs = await this.deserializeArguments(
-      args,
-      rpcDriverClassName,
-    )
-    const { regions, adapterConfig, sessionId } = deserializedArgs
-    const { dataAdapter } = await getAdapter(pm, sessionId, adapterConfig)
-
-    if (dataAdapter instanceof BaseFeatureDataAdapter) {
-      return dataAdapter.getMultiRegionStats(regions, deserializedArgs)
-    }
-    throw new Error('Data adapter not found')
-  }
-}
-
-export class MultiWiggleGetSources extends RpcMethodType {
-  name = 'MultiWiggleGetSources'
-
-  // eslint-disable-next-line @typescript-eslint/no-explicit-any
-  async deserializeArguments(args: any, rpcDriverClassName: string) {
-    const l = await super.deserializeArguments(args, rpcDriverClassName)
-    return {
-      ...l,
-      filters: args.filters
-        ? new SerializableFilterChain({
-            filters: args.filters,
-          })
-        : undefined,
-    }
-  }
-
-  async serializeArguments(
-    args: RenderArgs & {
-      signal?: AbortSignal
-      statusCallback?: (arg: string) => void
-    },
-    rpcDriverClassName: string,
-  ) {
-    const pm = this.pluginManager
-    const assemblyManager = pm.rootModel?.session?.assemblyManager
-    if (!assemblyManager) {
-      return args
-    }
-
-    const renamedArgs = await renameRegionsIfNeeded(assemblyManager, {
-      ...args,
-      filters: args.filters?.toJSON().filters,
-    })
-
-    return super.serializeArguments(renamedArgs, rpcDriverClassName)
-  }
-
-  async execute(
-    args: {
-      adapterConfig: {}
-      signal?: RemoteAbortSignal
-      sessionId: string
-      headers?: Record<string, string>
-      regions: Region[]
-      bpPerPx: number
-    },
-    rpcDriverClassName: string,
-  ) {
-    const pm = this.pluginManager
-    const deserializedArgs = await this.deserializeArguments(
-      args,
-      rpcDriverClassName,
-    )
-    const { regions, adapterConfig, sessionId } = deserializedArgs
-    const { dataAdapter } = await getAdapter(pm, sessionId, adapterConfig)
-
-    // @ts-expect-error
-    return dataAdapter.getSources(regions, deserializedArgs)
-  }
-}
+export * from './MultiWiggleGetSources'
+export * from './WiggleGetMultiRegionQuantitativeStats'
+export * from './WiggleGetGlobalQuantitativeStats'

package/src/index.ts

@@ -30,8 +30,8 @@ import MultiWiggleAddTrackWidgetF from './MultiWiggleAddTrackWidget'
 import * as utils from './util'
 
 import {
-  WiggleGetGlobalStats,
-  WiggleGetMultiRegionStats,
+  WiggleGetGlobalQuantitativeStats,
+  WiggleGetMultiRegionQuantitativeStats,
   MultiWiggleGetSources,
 } from './WiggleRPC/rpcMethods'
 
@@ -101,8 +101,8 @@ export default class WigglePlugin extends Plugin {
       },
     )
 
-    pm.addRpcMethod(() => new WiggleGetGlobalStats(pm))
-    pm.addRpcMethod(() => new WiggleGetMultiRegionStats(pm))
+    pm.addRpcMethod(() => new WiggleGetGlobalQuantitativeStats(pm))
+    pm.addRpcMethod(() => new WiggleGetMultiRegionQuantitativeStats(pm))
     pm.addRpcMethod(() => new MultiWiggleGetSources(pm))
   }
 

package/src/util.ts

@@ -5,7 +5,7 @@ import {
   getSession,
   getContainingView,
 } from '@jbrowse/core/util'
-import { FeatureStats } from '@jbrowse/core/util/stats'
+import { QuantitativeStats } from '@jbrowse/core/util/stats'
 import { getRpcSessionId } from '@jbrowse/core/util/tracks'
 import { addDisposer, isAlive } from 'mobx-state-tree'
 
@@ -176,9 +176,10 @@ export function groupBy<T>(array: T[], predicate: (v: T) => string) {
   return result
 }
 
-export async function getStats(
+export async function getQuantitativeStats(
   self: {
     adapterConfig: AnyConfigurationModel
+    configuration: AnyConfigurationModel
     autoscaleType: string
     setMessage: (str: string) => void
   },
@@ -187,7 +188,7 @@ export async function getStats(
     signal?: AbortSignal
     filters?: string[]
   },
-): Promise<FeatureStats> {
+): Promise<QuantitativeStats> {
   const { rpcManager } = getSession(self)
   const nd = getConf(self, 'numStdDev') || 3
   const { adapterConfig, autoscaleType } = self
@@ -204,11 +205,11 @@ export async function getStats(
   }
 
   if (autoscaleType === 'global' || autoscaleType === 'globalsd') {
-    const results: FeatureStats = (await rpcManager.call(
+    const results: QuantitativeStats = (await rpcManager.call(
       sessionId,
-      'WiggleGetGlobalStats',
+      'WiggleGetGlobalQuantitativeStats',
       params,
-    )) as FeatureStats
+    )) as QuantitativeStats
     const { scoreMin, scoreMean, scoreStdDev } = results
     // globalsd uses heuristic to avoid unnecessary scoreMin<0
     // if the scoreMin is never less than 0
@@ -225,7 +226,7 @@ export async function getStats(
     const { dynamicBlocks, bpPerPx } = getContainingView(self) as LGV
     const results = (await rpcManager.call(
       sessionId,
-      'WiggleGetMultiRegionStats',
+      'WiggleGetMultiRegionQuantitativeStats',
       {
         ...params,
         regions: dynamicBlocks.contentBlocks.map(region => {
@@ -238,7 +239,7 @@ export async function getStats(
         }),
         bpPerPx,
       },
-    )) as FeatureStats
+    )) as QuantitativeStats
     const { scoreMin, scoreMean, scoreStdDev } = results
 
     // localsd uses heuristic to avoid unnecessary scoreMin<0 if the
@@ -255,20 +256,25 @@ export async function getStats(
   if (autoscaleType === 'zscale') {
     return rpcManager.call(
       sessionId,
-      'WiggleGetGlobalStats',
+      'WiggleGetGlobalQuantitativeStats',
       params,
-    ) as Promise<FeatureStats>
+    ) as Promise<QuantitativeStats>
   }
   throw new Error(`invalid autoscaleType '${autoscaleType}'`)
 }
 
-export function statsAutorun(self: {
-  estimatedStatsReady: boolean
+export function quantitativeStatsAutorun(self: {
+  featureDensityStatsReady: boolean
   regionTooLarge: boolean
+  error: unknown
   setLoading: (aborter: AbortController) => void
   setError: (error: unknown) => void
-  updateStats: (stats: FeatureStats, statsRegion: string) => void
+  updateQuantitativeStats: (
+    stats: QuantitativeStats,
+    statsRegion: string,
+  ) => void
   renderProps: () => Record<string, unknown>
+  configuration: AnyConfigurationModel
   adapterConfig: AnyConfigurationModel
   autoscaleType: string
   setMessage: (str: string) => void
@@ -284,20 +290,21 @@ export function statsAutorun(self: {
 
           if (
             !view.initialized ||
-            !self.estimatedStatsReady ||
-            self.regionTooLarge
+            !self.featureDensityStatsReady ||
+            self.regionTooLarge ||
+            self.error
           ) {
             return
           }
           const statsRegion = JSON.stringify(view.dynamicBlocks)
 
-          const wiggleStats = await getStats(self, {
+          const wiggleStats = await getQuantitativeStats(self, {
             signal: aborter.signal,
             ...self.renderProps(),
           })
 
           if (isAlive(self)) {
-            self.updateStats(wiggleStats, statsRegion)
+            self.updateQuantitativeStats(wiggleStats, statsRegion)
           }
         } catch (e) {
           if (!isAbortException(e) && isAlive(self)) {