@genome-spy/core 0.14.1 → 0.17.0

package/src/view/testUtils.js

@@ -15,15 +15,15 @@ import { ViewFactory } from "./viewFactory";
 /** @type {<V extends View>(spec: ViewSpec, viewClass: { new(...args: any[]): V }, context?: ViewContext) => V} */
 export function create(spec, viewClass, context = undefined) {
     const viewTypeRegistry = new ViewFactory();
-    /** @type {ViewContext} */
-    const c = {
+
+    const c = /** @type {ViewContext} */ ({
         ...(context || {}),
         accessorFactory: new AccessorFactory(),
 
         createView: function (spec, parent, defaultName) {
             return viewTypeRegistry.createView(spec, c, parent, defaultName);
         },
-    };
+    });
 
     const view = c.createView(spec, null, "root");
 
@@ -40,7 +40,10 @@ export async function createAndInitialize(
     viewClass,
     context = undefined
 ) {
-    context = { ...(context || {}), dataFlow: new DataFlow() };
+    context = /** @type {ViewContext} */ ({
+        ...(context || {}),
+        dataFlow: new DataFlow(),
+    });
     const view = create(spec, viewClass, context);
     resolveScalesAndAxes(view);
     await initializeData(view, context.dataFlow);

package/src/view/unitView.js

@@ -170,7 +170,18 @@ export default class UnitView extends ContainerView {
                         : new AxisResolution(targetChannel);
             }
 
-            view.resolutions[type][targetChannel].pushUnitView(this, channel);
+            // Looks silly, but keeps type checking happy
+            if (isPositionalChannel(channel)) {
+                view.resolutions[type][targetChannel].pushUnitView(
+                    this,
+                    channel
+                );
+            } else if (type == "scale") {
+                view.resolutions[type][targetChannel].pushUnitView(
+                    this,
+                    channel
+                );
+            }
         }
     }
 
@@ -223,16 +234,11 @@ export default class UnitView extends ContainerView {
         }
 
         const channelDef = this.mark.encoding[channel];
+        // TODO: Broken. Fix.
         if (!isChannelDefWithScale(channelDef)) {
             throw new Error("The channel has no scale, cannot get domain!");
         }
 
-        const type = channelDef.type;
-        if (!type) {
-            throw new Error(`No data type for channel "${channel}"!`);
-            // TODO: Support defaults
-        }
-
         return channelDef;
     }
 
@@ -252,7 +258,7 @@ export default class UnitView extends ContainerView {
                 channelDef.resolutionChannel ?? channel
             );
             return createDomain(
-                channelDef.type,
+                channelDef.type ?? "nominal",
                 // Chrom/pos must be linearized first
                 scaleResolution.fromComplexInterval(specDomain)
             );
@@ -274,7 +280,7 @@ export default class UnitView extends ContainerView {
      */
     extractDataDomain(channel) {
         const channelDef = this._validateDomainQuery(channel);
-        const type = channelDef.type;
+        const type = channelDef.type ?? "nominal"; // TODO: Should check that this is a channel without scale
 
         /** @param {Channel} channel */
         const extract = (channel) => {

package/src/view/view.js

@@ -142,13 +142,10 @@ export default class View {
      */
     getSizeFromSpec() {
         /**
-         *
          * @param {"width" | "height"} dimension
          * @return {SizeDef}
          */
         const handleSize = (dimension) => {
-            /** @type {SizeDef} */
-            let sizeDef;
             let value = this.spec[dimension];
 
             if (isStepSize(value)) {
@@ -172,22 +169,27 @@ export default class View {
                         );
                     }
 
+                    // TODO: Type guards maybe?
+                    const _scale =
+                        /** @type {import("d3-scale").ScaleBand<any> | import("../genome/scaleLocus").ScaleLocus | import("../genome/scaleIndex").ScaleIndex} */ (
+                            scale
+                        );
+
                     steps = bandSpace(
                         steps,
-                        scale.paddingInner(),
-                        scale.paddingOuter()
+                        _scale.paddingInner(),
+                        _scale.paddingOuter()
                     );
 
-                    sizeDef = { px: steps * stepSize, grow: 0 };
+                    return { px: steps * stepSize, grow: 0 };
                 } else {
                     throw new Error(
                         "Cannot use 'step' size with missing scale!"
                     );
                 }
             } else {
-                sizeDef = (value && parseSizeDef(value)) || { px: 0, grow: 1 };
+                return (value && parseSizeDef(value)) ?? { px: 0, grow: 1 };
             }
-            return sizeDef;
         };
 
         return this._cache(

package/src/view/viewFactory.js

@@ -123,8 +123,10 @@ export function isLayerSpec(spec) {
 export function isFacetSpec(spec) {
     return (
         "facet" in spec &&
+        // @ts-expect-error
         isObject(spec.facet) &&
         "spec" in spec &&
+        // @ts-expect-error
         isObject(spec.spec)
     );
 }

package/src/view/viewUtils.js

@@ -35,10 +35,9 @@ import { rollup } from "d3-array";
  * @typedef {import("../spec/view").ImportSpec} ImportSpec
  * @typedef {import("../spec/view").ImportConfig} ImportConfig
  * @typedef {import("../spec/root").RootSpec} RootSpec
- * @typedef {import("../spec/root").RootConfig} RootConfig
  *
- * @typedef {import("../spec/channel").FacetFieldDef} FacetFieldDef
  * @typedef {import("../spec/view").FacetMapping} FacetMapping
+ * @typedef {import("../spec/channel").FacetFieldDef} FacetFieldDef
  */
 
 /**
@@ -253,14 +252,14 @@ export async function initializeData(root, existingFlow) {
  * @param {View} view
  */
 export function findEncodedFields(view) {
-    /** @type {{view: UnitView, channel: string, field: string, type: string}[]} */
+    /** @type {{view: UnitView, channel: import("../spec/channel").Channel, field: import("../spec/channel").Field, type: import("../spec/channel").Type}[]} */
     const fieldInfos = [];
 
     view.visit((view) => {
         if (view instanceof UnitView) {
             const encoding = view.getEncoding();
             for (const [channel, def] of Object.entries(encoding)) {
-                if (isFieldDef(def)) {
+                if (isFieldDef(def) && "type" in def) {
                     fieldInfos.push({
                         view,
                         channel,
@@ -292,7 +291,7 @@ async function loadExternalViewSpec(spec, baseUrl, viewContext) {
     const url = spec.import.url;
 
     const importedSpec = JSON.parse(
-        await loader.load(url).catch((e) => {
+        await loader.load(url).catch((/** @type {Error} */ e) => {
             throw new Error(
                 `Could not load imported view spec: ${url} \nReason: ${e.message}`
             );

package/src/options.d.ts

@@ -1,9 +0,0 @@
-import { TooltipHandler } from "./tooltip/tooltipHandler";
-
-export interface EmbedOptions {
-    /** If true, don't display the toolbar. */
-    bare?: boolean;
-
-    /** Custom tooltip handlers. Use "default" to override the default handler */
-    tooltipHandlers?: Record<string, TooltipHandler>;
-}

package/src/utils/fisheye.js

@@ -1,60 +0,0 @@
-/**
- * Based on:
- * fisheye d3-plugin (c) Mike Bostock
- * https://github.com/d3/d3-plugins/blob/master/fisheye/
- */
-export default function () {
-    let radius = 200;
-    let distortion = 2;
-    let k0 = 1,
-        k1 = 1;
-    let focus = 0;
-
-    /**
-     *
-     * @param {number} x
-     */
-    function fisheye(x) {
-        const dx = x - focus;
-        const dd = Math.abs(dx);
-        if (!dd || dd >= radius) return x;
-        const k = ((k0 * (1 - Math.exp(-dd * k1))) / dd) * 0.75 + 0.25;
-        return focus + dx * k;
-    }
-
-    function rescale() {
-        k0 = Math.exp(distortion);
-        k0 = (k0 / (k0 - 1)) * radius;
-        k1 = distortion / radius;
-        return fisheye;
-    }
-
-    /**
-     * @param {number} _
-     */
-    fisheye.radius = function (_) {
-        if (!arguments.length) return radius;
-        radius = +_;
-        return rescale();
-    };
-
-    /**
-     * @param {number} _
-     */
-    fisheye.distortion = function (_) {
-        if (!arguments.length) return distortion;
-        distortion = +_;
-        return rescale();
-    };
-
-    /**
-     * @param {number} _
-     */
-    fisheye.focus = function (_) {
-        if (!arguments.length) return focus;
-        focus = _;
-        return fisheye;
-    };
-
-    return rescale();
-}

package/src/utils/html.js

@@ -1,23 +0,0 @@
-/* https://stackoverflow.com/a/41699140/1547896 */
-
-export function escapeHtml(str) {
-    var map = {
-        "&": "&",
-        "<": "&lt;",
-        ">": "&gt;",
-        '"': "&quot;",
-        "'": "&#039;",
-    };
-    return str.replace(/[&<>"']/g, (m) => map[m]);
-}
-
-export function decodeHtml(str) {
-    var map = {
-        "&amp;": "&",
-        "&lt;": "<",
-        "&gt;": ">",
-        "&quot;": '"',
-        "&#039;": "'",
-    };
-    return str.replace(/&amp;|&lt;|&gt;|&quot;|&#039;/g, (m) => map[m]);
-}

package/src/utils/html.test.js

@@ -1,13 +0,0 @@
-import * as html from "./html";
-
-test("Escape HTML", () => {
-    expect(html.escapeHtml('< "x" & "y" >')).toEqual(
-        "&lt; &quot;x&quot; &amp; &quot;y&quot; &gt;"
-    );
-});
-
-test("Decode HTML", () => {
-    expect(
-        html.decodeHtml("&lt; &quot;x&quot; &amp; &quot;y&quot; &gt;")
-    ).toEqual('< "x" & "y" >');
-});

package/src/view/channel.js

@@ -1,5 +0,0 @@
-export default class Channel {
-    constructor() {
-        this.type = null; // x, y, etc
-    }
-}