@jbrowse/core 2.3.1 → 2.3.3

package/PluginManager.js

@@ -258,7 +258,7 @@ class PluginManager {
             .filter(t => (0, mobx_state_tree_1.isType)(t) && (0, mobx_state_tree_1.isModelType)(t));
         // try to smooth over the case when no types are registered, mostly
         // encountered in tests
-        if (pluggableTypes.length === 0) {
+        if (pluggableTypes.length === 0 && typeof jest === 'undefined') {
             console.warn(`No pluggable types found matching ('${groupName}','${fieldName}')`);
             return fallback;
         }

package/data_adapters/BaseAdapter.d.ts

@@ -96,6 +96,7 @@ export declare abstract class BaseFeatureDataAdapter extends BaseAdapter {
      * Currently this just calls getFeatureInRegion for each region. Adapters that
      * are frequently called on multiple regions simultaneously may want to
      * implement a more efficient custom version of this method.
+     *
      * @param regions - Regions
      * @param opts - Feature adapter options
      * @returns Observable of Feature objects in the regions

package/data_adapters/BaseAdapter.js

@@ -7,6 +7,7 @@ exports.isTextSearchAdapter = exports.isRefNameAliasAdapter = exports.isFeatureA
 const rxjs_1 = require("rxjs");
 const operators_1 = require("rxjs/operators");
 const mobx_state_tree_1 = require("mobx-state-tree");
+// locals
 const rxjs_2 = require("../util/rxjs");
 const util_1 = require("../util");
 const configuration_1 = require("../configuration");
@@ -95,6 +96,7 @@ class BaseFeatureDataAdapter extends BaseAdapter {
      * Currently this just calls getFeatureInRegion for each region. Adapters that
      * are frequently called on multiple regions simultaneously may want to
      * implement a more efficient custom version of this method.
+     *
      * @param regions - Regions
      * @param opts - Feature adapter options
      * @returns Observable of Feature objects in the regions
@@ -122,12 +124,12 @@ class BaseFeatureDataAdapter extends BaseAdapter {
             return (0, stats_1.blankStats)();
         }
         const feats = await Promise.all(regions.map(region => this.getRegionStats(region, opts)));
-        const scoreMax = feats.map(a => a.scoreMax).reduce((a, b) => Math.max(a, b));
-        const scoreMin = feats.map(a => a.scoreMin).reduce((a, b) => Math.min(a, b));
-        const scoreSum = feats.reduce((a, b) => a + b.scoreSum, 0);
-        const scoreSumSquares = feats.reduce((a, b) => a + b.scoreSumSquares, 0);
-        const featureCount = feats.reduce((a, b) => a + b.featureCount, 0);
-        const basesCovered = feats.reduce((a, b) => a + b.basesCovered, 0);
+        const scoreMax = (0, util_1.max)(feats.map(a => a.scoreMax));
+        const scoreMin = (0, util_1.min)(feats.map(a => a.scoreMin));
+        const scoreSum = (0, util_1.sum)(feats.map(a => a.scoreSum));
+        const scoreSumSquares = (0, util_1.sum)(feats.map(a => a.scoreSumSquares));
+        const featureCount = (0, util_1.sum)(feats.map(a => a.featureCount));
+        const basesCovered = (0, util_1.sum)(feats.map(a => a.basesCovered));
         return (0, stats_1.rectifyStats)({
             scoreMin,
             scoreMax,
@@ -151,9 +153,7 @@ class BaseFeatureDataAdapter extends BaseAdapter {
                 start: Math.max(0, Math.round(sampleCenter - length / 2)),
                 end: Math.min(Math.round(sampleCenter + length / 2), end),
             };
-            const features = await this.getFeatures(query, opts)
-                .pipe((0, operators_1.toArray)())
-                .toPromise();
+            const features = await (0, rxjs_1.firstValueFrom)(this.getFeatures(query, opts).pipe((0, operators_1.toArray)()));
             return maybeRecordStats(length, { featureDensity: features.length / length }, features.length, expansionTime);
         };
         const maybeRecordStats = async (interval, stats, statsSampleFeatures, expansionTime) => {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@jbrowse/core",
-  "version": "2.3.1",
+  "version": "2.3.3",
   "description": "JBrowse 2 core libraries used by plugins",
   "keywords": [
     "jbrowse",
@@ -30,6 +30,7 @@
   "dependencies": {
     "@babel/runtime": "^7.17.9",
     "@mui/icons-material": "^5.0.1",
+    "@types/clone": "^2.0.0",
     "abortable-promise-cache": "^1.5.0",
     "canvas-sequencer": "^3.1.0",
     "canvas2svg": "^1.0.16",
@@ -66,12 +67,12 @@
     "prop-types": "^15.0.0",
     "react": ">=16.8.0",
     "react-dom": ">=16.8.0",
-    "rxjs": "^6.0.0",
+    "rxjs": "^7.0.0",
     "tss-react": "^4.0.0"
   },
   "publishConfig": {
     "access": "public",
     "directory": "dist"
   },
-  "gitHead": "75c4cba2f50c626c62881abd5851dbf2435c2401"
+  "gitHead": "283e0387ccd5acc9f092cf00804d1fcac212e68d"
 }

package/pluggableElementTypes/renderers/ComparativeServerSideRendererType.js

@@ -8,6 +8,7 @@ const operators_1 = require("rxjs/operators");
 const ServerSideRendererType_1 = __importDefault(require("./ServerSideRendererType"));
 const dataAdapterCache_1 = require("../../data_adapters/dataAdapterCache");
 const util_1 = require("../../util");
+const rxjs_1 = require("rxjs");
 class ComparativeServerSideRenderer extends ServerSideRendererType_1.default {
     /**
      * directly modifies the render arguments to prepare
@@ -78,10 +79,9 @@ class ComparativeServerSideRenderer extends ServerSideRendererType_1.default {
             return requestRegion;
         });
         // note that getFeaturesInMultipleRegions does not do glyph expansion
-        const res = await dataAdapter
+        const res = await (0, rxjs_1.firstValueFrom)(dataAdapter
             .getFeaturesInMultipleRegions(requestRegions, renderArgs)
-            .pipe((0, operators_1.filter)(f => this.featurePassesFilters(renderArgs, f)), (0, operators_1.toArray)())
-            .toPromise();
+            .pipe((0, operators_1.filter)(f => this.featurePassesFilters(renderArgs, f)), (0, operators_1.toArray)()));
         // dedupe needed xref https://github.com/GMOD/jbrowse-components/pull/3404/
         return (0, util_1.dedupe)(res, f => f.id());
     }

package/pluggableElementTypes/renderers/FeatureRendererType.js

@@ -5,6 +5,8 @@ var __importDefault = (this && this.__importDefault) || function (mod) {
 Object.defineProperty(exports, "__esModule", { value: true });
 const operators_1 = require("rxjs/operators");
 const clone_1 = __importDefault(require("clone"));
+const rxjs_1 = require("rxjs");
+// locals
 const util_1 = require("../../util");
 const simpleFeature_1 = __importDefault(require("../../util/simpleFeature"));
 const dataAdapterCache_1 = require("../../data_adapters/dataAdapterCache");
@@ -106,7 +108,7 @@ class FeatureRendererType extends ServerSideRendererType_1.default {
         const featureObservable = requestRegions.length === 1
             ? dataAdapter.getFeatures(this.getExpandedRegion(region, renderArgs), renderArgs)
             : dataAdapter.getFeaturesInMultipleRegions(requestRegions, renderArgs);
-        const feats = await featureObservable.pipe((0, operators_1.toArray)()).toPromise();
+        const feats = await (0, rxjs_1.firstValueFrom)(featureObservable.pipe((0, operators_1.toArray)()));
         (0, util_1.checkAbortSignal)(signal);
         return new Map(feats
             .filter(feat => this.featurePassesFilters(renderArgs, feat))

package/rpc/methods/CoreGetFeatures.js

@@ -5,6 +5,8 @@ var __importDefault = (this && this.__importDefault) || function (mod) {
 Object.defineProperty(exports, "__esModule", { value: true });
 /* eslint-disable @typescript-eslint/no-non-null-assertion */
 const operators_1 = require("rxjs/operators");
+const rxjs_1 = require("rxjs");
+// locals
 const dataAdapterCache_1 = require("../../data_adapters/dataAdapterCache");
 const RpcMethodType_1 = __importDefault(require("../../pluggableElementTypes/RpcMethodType"));
 const BaseAdapter_1 = require("../../data_adapters/BaseAdapter");
@@ -37,7 +39,7 @@ class CoreGetFeatures extends RpcMethodType_1.default {
             ...opts,
             signal,
         });
-        const r = await ret.pipe((0, operators_1.toArray)()).toPromise();
+        const r = await (0, rxjs_1.firstValueFrom)(ret.pipe((0, operators_1.toArray)()));
         return r.map(f => f.toJSON());
     }
 }