@genome-spy/core 0.15.0 → 0.18.1

package/src/spec/scale.d.ts

@@ -10,6 +10,23 @@
 
 import { ChromosomalLocus } from "./genome";
 
+export type ScaleType =
+    | "null"
+    | "linear"
+    | "log"
+    | "pow"
+    | "sqrt"
+    | "symlog"
+    | "identity"
+    | "sequential"
+    | "quantize"
+    | "threshold"
+    | "ordinal"
+    | "point"
+    | "band"
+    | "index"
+    | "locus";
+
 export interface Scale {
     /**
      * The name of the scale. Names are optional but allow the scales to be referenced and found with the API.
@@ -27,7 +44,7 @@ export interface Scale {
      *
      * __Default value:__ please see the [scale type table](https://vega.github.io/vega-lite/docs/scale.html#type).
      */
-    type?: string; // ScaleType;
+    type?: ScaleType;
 
     /**
      * Customized domain values.

package/src/spec/view.d.ts

@@ -1,7 +1,13 @@
 import { Data } from "./data";
 import { Scale } from "./scale";
 import { TransformParams } from "./transform";
-import { Channel, Encoding, FacetFieldDef, PositionalChannel } from "./channel";
+import {
+    Channel,
+    Encoding,
+    FacetFieldDef,
+    PrimaryPositionalChannel,
+    Type,
+} from "./channel";
 import {
     FillAndStrokeProps,
     MarkConfigAndType,
@@ -30,7 +36,7 @@ export interface FacetMapping {
  * The opacity is interpolated between the specified stops.
  */
 export interface DynamicOpacity {
-    channel?: PositionalChannel;
+    channel?: PrimaryPositionalChannel;
     /** Stops expressed as units (base pairs, for example) per pixel. */
     unitsPerPixel: number[];
     /** Opacity values that match the given stops. */
@@ -124,14 +130,14 @@ export interface LayerSpec extends ViewSpecBase, AggregateSamplesSpec {
 }
 
 export interface FacetSpec extends ViewSpecBase {
-    facet: FacetFieldDef | FacetMapping;
+    facet: any; //FacetMapping | FacetFieldDef
     spec: LayerSpec | UnitSpec;
     columns?: number;
     spacing?: number;
 }
 
 export interface SampleAttributeDef {
-    type: string;
+    type: Type; // TODO: Omit index/locus
     /** Color scale (primary) */
     scale?: Scale;
     barScale?: Scale;
@@ -170,7 +176,7 @@ export interface ResolveSpec {
 
 export type ContainerSpec = (
     | LayerSpec
-    | FacetSpec
+    //    | FacetSpec
     | SampleSpec
     | VConcatSpec
     | HConcatSpec
@@ -182,7 +188,7 @@ export type ContainerSpec = (
 export type ViewSpec =
     | UnitSpec
     | LayerSpec
-    | FacetSpec
+    //    | FacetSpec
     | SampleSpec
     | VConcatSpec
     | HConcatSpec

package/src/tooltip/refseqGeneTooltipHandler.js

@@ -43,6 +43,7 @@ async function fetchGeneSummary(symbol) {
 
     console.log("Searching: " + symbol);
 
+    /** @type {RequestInit} */
     const opts = { mode: "cors" };
 
     const searchResult = await fetch(

package/src/types/filetypes.d.ts

@@ -2,3 +2,13 @@ declare module "*.svg" {
     const content: string;
     export default content;
 }
+
+declare module "*.txt" {
+    const content: string;
+    export default content;
+}
+
+declare module "*.png" {
+    const value: any;
+    export = value;
+}

package/src/types/internmap.d.ts

@@ -0,0 +1,22 @@
+// Source: https://github.com/DefinitelyTyped/DefinitelyTyped/blob/master/types/d3-array/index.d.ts
+
+declare module "internmap" {
+    /**
+     * The InternMap class extends the native JavaScript Map class, allowing Dates and other non-primitive keys by bypassing the SameValueZero algorithm when determining key equality.
+     */
+    export class InternMap<K = any, V = any> extends Map<K, V> {
+        constructor(
+            entries?: readonly (readonly [K, V])[] | null,
+            keyFunction?: KeyFunction
+        );
+    }
+
+    /**
+     * The InternSet class extends the native JavaScript Set class, allowing Dates and other non-primitive keys by bypassing the SameValueZero algorithm when determining key equality.
+     */
+    export class InternSet<T = any> extends Set<T> {
+        constructor(values?: readonly T[] | null, keyFunction?: KeyFunction);
+    }
+}
+
+type KeyFunction = (key: any) => string | number | bigint | boolean | symbol;

package/src/types/vega-loader.d.ts

@@ -0,0 +1 @@
+declare module "vega-loader";

package/src/utils/addBaseUrl.js

@@ -0,0 +1,19 @@
+/**
+ * @param {string} url
+ * @param {string} baseUrl
+ */
+export default function addBaseUrl(url, baseUrl) {
+    // Regex copied from vega-loader
+    if (!baseUrl || /^(data:|([A-Za-z]+:)?\/\/)/.test(url)) {
+        return url;
+    }
+
+    if (!url.startsWith("/")) {
+        if (!baseUrl.endsWith("/")) {
+            baseUrl += "/";
+        }
+        return baseUrl + url;
+    }
+
+    return url;
+}

package/src/utils/addBaseUrl.test.js

@@ -0,0 +1,21 @@
+import addBaseUrl from "./addBaseUrl";
+
+test("addBaseUrl adds baseUrl when needed", () => {
+    expect(addBaseUrl("foo.html", "https://site.com/")).toEqual(
+        "https://site.com/foo.html"
+    );
+    expect(addBaseUrl("foo.html", "https://site.com")).toEqual(
+        "https://site.com/foo.html"
+    );
+    expect(addBaseUrl("bar/foo.html", "https://site.com/")).toEqual(
+        "https://site.com/bar/foo.html"
+    );
+    expect(addBaseUrl("../foo.html", "https://site.com/bar/")).toEqual(
+        "https://site.com/bar/../foo.html"
+    );
+});
+
+test("addBaseUrl doesn't add baseUrl when not needed", () => {
+    expect(addBaseUrl("/foo.html", "https://site.com/")).toEqual("/foo.html");
+    expect(addBaseUrl("foo.html", undefined)).toEqual("foo.html");
+});

package/src/utils/arrayUtils.js

@@ -1,14 +1,20 @@
 /**
- *
+ * @param {T[]} a
+ * @param {T[]} b
+ * @template T
+ */
+export function shallowArrayEquals(a, b) {
+    return a.length == b.length && a.every((s, i) => a[i] === b[i]);
+}
+
+/**
  * @param {A[]} a
  * @param {B[]} b
- * @param {function(A):T} [aAccessor]
- * @param {function(B):T} [bAccessor]
+ * @param {function(A):T} aAccessor
+ * @param {function(B):T} bAccessor
  * @template A, B, T
  */
-export function shallowArrayEquals(a, b, aAccessor, bAccessor) {
-    aAccessor = aAccessor || ((x) => x);
-    bAccessor = bAccessor || ((x) => x);
+export function shallowArrayEqualsWithAccessors(a, b, aAccessor, bAccessor) {
     return (
         a.length == b.length &&
         a.every((s, i) => aAccessor(a[i]) === bAccessor(b[i]))

package/src/utils/cloner.js

@@ -7,14 +7,16 @@
  * https://mrale.ph/blog/2014/07/30/constructor-vs-objectcreate.html
  *
  * @param {T} template The template object that
- * @returns {function(T):T}
+ * @returns {(function(T):T) & { properties: string[] }}
  * @template T
  */
 export default function createCloner(template) {
     // TODO: Check that only properties, not methods get cloned
-    const properties = Object.keys(template);
+    const properties = /** @type {string[]} */ (
+        Object.keys(template).filter((k) => typeof k == "string")
+    );
 
-    const cloner = /** @type {function(T):T} */ (
+    const cloner = /** @type {(function(T):T) & { properties: string[] }} */ (
         new Function(
             "source",
             "return { " +

package/src/utils/concatIterables.js

@@ -1,6 +1,6 @@
 /**
- *
- * @param  {...Iterable} iterables
+ * @param  {...Iterable<T>} iterables
+ * @template T
  */
 export default function concatIterables(...iterables) {
     if (iterables.length <= 0) {

package/src/utils/domainArray.js

@@ -194,14 +194,6 @@ export default function createDomain(type, initialDomain) {
     throw new Error("Unknown type: " + type);
 }
 
-/**
- *
- * @param {array} array
- */
-export function isDomainArray(array) {
-    return array instanceof DomainArray;
-}
-
 /**
  * For unit tests
  *

package/src/utils/propertyCoalescer.js

@@ -2,16 +2,19 @@
  * Coalesces properties. Allows for creating chains of defaults without
  * using object destructuring, which may generate piles of garbage for GC.
  *
- * Still WIP. TODO: Efficient computed defaults.
+ * Still WIP.
+ * TODO: Efficient computed defaults.
+ * TODO: Make sense of the types
  *
  * @param  {...function():T} sources
  * @returns {T}
  *
- * @template T
+ * @template {Record<string | symbol, any>} [T=object]
  */
 export default function coalesceProperties(...sources) {
+    /** @type {ProxyHandler<T>} */
     const handler = {
-        get: function (target, prop, receiver) {
+        get(_target, prop, _receiver) {
             for (const source of sources) {
                 const props = source();
                 const value = props[prop];
@@ -22,7 +25,8 @@ export default function coalesceProperties(...sources) {
             return undefined;
         },
 
-        has: function (target, prop, receiver) {
+        // @ts-ignore
+        has(target, prop, _receiver) {
             for (const source of sources) {
                 const props = source();
                 if (prop in props) {
@@ -33,5 +37,6 @@ export default function coalesceProperties(...sources) {
         },
     };
 
+    // @ts-ignore
     return new Proxy({}, handler);
 }

package/src/view/axisResolution.js

@@ -5,6 +5,7 @@ import {
     isExprDef,
     isFieldDef,
     isSecondaryChannel,
+    isValueDef,
 } from "../encoder/encoder";
 import { peek } from "../utils/arrayUtils";
 import coalesce from "../utils/coalesce";
@@ -22,7 +23,7 @@ export default class AxisResolution {
      */
     constructor(channel) {
         this.channel = channel;
-        /** @type {import("./scaleResolution").ResolutionMember[]} The involved views */
+        /** @type {import("./scaleResolution").ResolutionMember<import("../spec/channel").PositionalChannel>[]} The involved views */
         this.members = [];
     }
 
@@ -35,7 +36,7 @@ export default class AxisResolution {
      * scales have been resolved.
      *
      * @param {UnitView} view
-     * @param {import("../spec/channel").Channel} channel TODO: Do something for this
+     * @param {import("../spec/channel").PositionalChannel} channel TODO: Do something for this
      */
     pushUnitView(view, channel) {
         const newScaleResolution = view.getScaleResolution(this.channel);
@@ -56,10 +57,10 @@ export default class AxisResolution {
 
     getAxisProps() {
         return getCachedOrCall(this, "axisProps", () => {
-            const propArray = this.members.map(
-                (member) =>
-                    getChannelDefWithScale(member.view, member.channel).axis
-            );
+            const propArray = this.members.map((member) => {
+                const channelDef = member.view.mark.encoding[member.channel];
+                return "axis" in channelDef && channelDef.axis;
+            });
 
             if (
                 propArray.length > 0 &&
@@ -87,6 +88,10 @@ export default class AxisResolution {
                 member.channel
             );
 
+            if (isValueDef(channelDef)) {
+                return undefined;
+            }
+
             // Retain nulls as they indicate that no title should be shown
 
             return {

package/src/view/axisView.js

@@ -2,14 +2,17 @@ import { validTicks, tickValues, tickFormat, tickCount } from "../scale/ticks";
 import LayerView from "./layerView";
 import { isNumber } from "vega-util";
 import smoothstep from "../utils/smoothstep";
-import { shallowArrayEquals } from "../utils/arrayUtils";
+import {
+    shallowArrayEquals,
+    shallowArrayEqualsWithAccessors,
+} from "../utils/arrayUtils";
 import { FlexDimensions } from "../utils/layout/flexLayout";
 import DynamicCallbackSource from "../data/sources/dynamicCallbackSource";
 
 const CHROM_LAYER_NAME = "chromosome_ticks_and_labels";
 
 /**
- * @typedef {import("../spec/channel").PositionalChannel} PositionalChannel
+ * @typedef {import("../spec/channel").PrimaryPositionalChannel} PositionalChannel
  * @typedef {import("../spec/view").GeometricDimension} GeometricDimension
  */
 
@@ -265,7 +268,7 @@ function generateTicks(axisProps, scale, axisLength, oldTicks = []) {
         : tickValues(scale, count);
 
     if (
-        shallowArrayEquals(
+        shallowArrayEqualsWithAccessors(
             values,
             oldTicks,
             (v) => v,
@@ -428,7 +431,7 @@ function createAxis(axisProps) {
         },
         encoding: {
             [main]: { field: "value", type: "quantitative" },
-            text: { field: "label", type: "quantitative" },
+            text: { field: "label" },
         },
     });
 
@@ -649,7 +652,7 @@ export function createGenomeAxis(axisProps) {
             },
             encoding: {
                 [main + "2"]: { field: "continuousEnd", type: "locus" },
-                text: { field: "name", type: "ordinal" },
+                text: { field: "name" },
             },
         };
         return labels;

package/src/view/decoratorView.js

@@ -6,7 +6,7 @@ import UnitView from "./unitView";
 import { ZERO_FLEXDIMENSIONS } from "../utils/layout/flexLayout";
 
 /**
- * @typedef {import("../spec/channel").PositionalChannel} PositionalChannel
+ * @typedef {import("../spec/channel").PrimaryPositionalChannel} PositionalChannel
  * @typedef {import("../spec/view").GeometricDimension} GeometricDimension
  */
 
@@ -384,7 +384,10 @@ export default class DecoratorView extends ContainerView {
                     zDelta: 0,
                 });
             }
-        } else if (event.type == "mousedown" && event.uiEvent.button === 0) {
+        } else if (
+            event.type == "mousedown" &&
+            /** @type {MouseEvent} */ (event.uiEvent).button === 0
+        ) {
             const mouseEvent = /** @type {MouseEvent} */ (event.uiEvent);
             mouseEvent.preventDefault();
 
@@ -424,7 +427,7 @@ export default class DecoratorView extends ContainerView {
 
     _getZoomableResolutions() {
         return this._cache("zoomableResolutions", () => {
-            /** @type {Record<import("../spec/channel").PositionalChannel, Set<import("./scaleResolution").default>>} */
+            /** @type {Record<import("../spec/channel").PrimaryPositionalChannel, Set<import("./scaleResolution").default>>} */
             const resolutions = {
                 x: new Set(),
                 y: new Set(),

package/src/view/facetView.js

@@ -1,3 +1,6 @@
+// @ts-nocheck
+// TODO: Enable when this is taken to use again
+
 import { isFacetFieldDef, isFacetMapping } from "./viewUtils";
 import ContainerView from "./containerView";
 import UnitView from "./unitView";

package/src/view/flowBuilder.js

@@ -286,6 +286,7 @@ export function linearizeLocusAccess(view) {
                       ...rewrittenEncoding,
                   };
                   // This is so ugly...
+                  // @ts-ignore
                   invalidate(view.mark, "encoding");
               },
           }
@@ -294,7 +295,7 @@ export function linearizeLocusAccess(view) {
 
 /**
  * @param {View} view
- * @param {Record<string, import("../spec/channel").ChannelDef>} [encoding]
+ * @param {Encoding} [encoding]
  * @returns {import("../spec/transform").CompareParams}
  */
 function getCompareParamsForView(view, encoding) {

package/src/view/renderingContext/svgViewRenderingContext.js

@@ -8,8 +8,12 @@ import ViewRenderingContext from "./viewRenderingContext";
  * @typedef {import("../view").default} View
  */
 export default class SvgViewRenderingContext extends ViewRenderingContext {
-    constructor() {
-        super();
+    /**
+     *
+     * @param {import("../rendering").GlobalRenderingOptions} globalOptions
+     */
+    constructor(globalOptions) {
+        super(globalOptions);
 
         /** @type {import("../../utils/layout/rectangle").default} */
         this.coords = undefined;
@@ -39,7 +43,7 @@ export default class SvgViewRenderingContext extends ViewRenderingContext {
             this.svg.setAttributeNS(
                 null,
                 "viewBox",
-                ["x", "y", "width", "height"].map((a) => viewBox[a]).join(" ")
+                [viewBox.x, viewBox.y, viewBox.width, viewBox.height].join(" ")
             );
         }