@genome-spy/core 0.48.2 → 0.50.0

package/dist/src/view/flowBuilder.js

@@ -35,6 +35,8 @@ export function buildDataFlow(root, existingFlow) {
 
     /** @type {FlowNode[]} "Current nodes" on the path from view root to the current view */
     const nodeStack = [];
+    /** @type {{view: View, nodeStackDepth: number}[]} */
+    const viewStack = [];
 
     /** @type {FlowNode} */
     let currentNode;
@@ -59,6 +61,7 @@ export function buildDataFlow(root, existingFlow) {
         }
         currentNode.addChild(node);
         currentNode = node;
+        nodeStack.push(node);
         return node;
     }
 
@@ -108,10 +111,16 @@ export function buildDataFlow(root, existingFlow) {
         }
     }
 
+    function isInBranchWithIdentifier() {
+        return (
+            nodeStack.findLastIndex(
+                (node) => node instanceof IdentifierTransform
+            ) > nodeStack.findLastIndex((node) => node instanceof DataSource)
+        );
+    }
+
     /** @param {View} view */
     const processView = (view) => {
-        nodeStack.push(currentNode);
-
         if (view.spec.data) {
             const dataSource = isNamedData(view.spec.data)
                 ? new NamedSource(
@@ -122,6 +131,7 @@ export function buildDataFlow(root, existingFlow) {
                 : createDataSource(view.spec.data, view);
 
             currentNode = dataSource;
+            nodeStack.push(dataSource);
             dataFlow.addDataSource(dataSource, view);
         }
 
@@ -148,13 +158,24 @@ export function buildDataFlow(root, existingFlow) {
                 }
             }
 
-            if (view.mark.isPickingParticipant()) {
+            if (
+                view.mark.isPickingParticipant() &&
+                !isInBranchWithIdentifier()
+            ) {
+                // TODO: If two branches receive the same data, they should have
+                // a shared identifier transform. However, unique ids cannot be
+                // assigned right after the data source, because some transforms,
+                // (e.g, RegexFold) may generate new tuples.
+                //
                 appendTransform(new CloneTransform());
                 appendTransform(
                     new IdentifierTransform({ type: "identifier" })
                 );
             }
 
+            // TODO: Handle cases where the spec contains terminal collector nodes
+            // (with custom sort configuration, for example). Now a new collector
+            // is always appended to the end of the current branch.
             const collector = new Collector({
                 type: "collect",
                 groupby: view.getFacetFields(),
@@ -178,10 +199,18 @@ export function buildDataFlow(root, existingFlow) {
 
     for (const dataTree of dataTrees) {
         visitTree(dataTree, {
-            preOrder: (node) => processView(node.ref),
+            preOrder: (node) => {
+                viewStack.push({
+                    view: node.ref,
+                    nodeStackDepth: nodeStack.length,
+                });
+                processView(node.ref);
+            },
             // eslint-disable-next-line no-loop-func
             postOrder: () => {
-                currentNode = nodeStack.pop();
+                const { nodeStackDepth } = viewStack.pop();
+                nodeStack.length = nodeStackDepth;
+                currentNode = nodeStack.at(-1);
             },
         });
     }

package/dist/src/view/flowBuilder.test.d.ts

@@ -0,0 +1,2 @@
+export {};
+//# sourceMappingURL=flowBuilder.test.d.ts.map

package/dist/src/view/flowBuilder.test.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"flowBuilder.test.d.ts","sourceRoot":"","sources":["../../../src/view/flowBuilder.test.js"],"names":[],"mappings":""}

package/dist/src/view/flowBuilder.test.js

@@ -23,7 +23,7 @@ function byPath(root, path) {
     return root;
 }
 
-/** @type {import("../spec/mark.js").MarkConfigAndType} */
+/** @type {import("../spec/mark.js").MarkProps} */
 const mark = {
     type: "rect",
     tooltip: null,

package/dist/src/view/gridView.d.ts

@@ -66,12 +66,6 @@ export default class GridView extends ContainerView {
      * @protected
      */
     protected createAxes(): Promise<void[]>;
-    /**
-     * @param {import("./renderingContext/viewRenderingContext.js").default} context
-     * @param {import("./layout/rectangle.js").default} coords
-     * @param {import("../types/rendering.js").RenderingOptions} [options]
-     */
-    render(context: import("./renderingContext/viewRenderingContext.js").default, coords: import("./layout/rectangle.js").default, options?: import("../types/rendering.js").RenderingOptions): void;
     #private;
 }
 export class GridChild {

package/dist/src/view/gridView.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"gridView.d.ts","sourceRoot":"","sources":["../../../src/view/gridView.js"],"names":[],"mappings":"AAswBA;;;GAGG;AACH,iDAHW,OAAO,iBAAiB,EAAE,cAAc,GACtC,OAAO,iBAAiB,EAAE,QAAQ,CAwB9C;AAED;;;GAGG;AACH,uDAHW,OAAO,iBAAiB,EAAE,cAAc,GACtC,OAAO,iBAAiB,EAAE,QAAQ,CA6C9C;AA2BD;;GAEG;AACH,8EAUC;AAED;;;;;GAKG;AACH,4CAJW,OAAO,uBAAuB,EAAE,OAAO,UACvC,OAAO,iBAAiB,EAAE,UAAU,YACpC,QAAQ,aAmBlB;AA53BD;;;;;;;;;;;;;;;GAeG;AACH;IA6BI;;;;;;;;;OASG;IACH,kBARW,OAAO,iBAAiB,EAAE,aAAa,WACvC,OAAO,yBAAyB,EAAE,OAAO,gBACzC,aAAa,iDAEb,MAAM,WACN,MAAM,YACN,OAAO,WAAW,EAAE,WAAW,EAoBzC;IARG,8CAAgB;IAOhB,uBAA0B;IAG9B;;OAEG;IACH,qDAIC;IAeD;;OAEG;IACH,wDAKC;IAqBD;;OAEG;IACH,8CAEC;IAED,yBAEC;IAED;;OAEG;IACH,wCAmCC;IA2OD;;;;OAIG;IAEH,gBALW,OAAO,4CAA4C,EAAE,OAAO,UAC5D,OAAO,uBAAuB,EAAE,OAAO,YACvC,OAAO,uBAAuB,EAAE,gBAAgB,QA6O1D;;CAoGJ;AAgJD;IACI;;;;OAIG;IACH,6DAHW,aAAa,UACb,MAAM,EAoFhB;IAjFG,4BAAgC;IAChC,kCAAgB;IAChB,eAAoB;IAEpB,uBAAuB;IACvB,YADW,QAAQ,CACQ;IAE3B,uBAAuB;IACvB,kBADW,QAAQ,CACc;IAEjC,mFAAmF;IACnF,MADW,QAAQ,OAAO,OAAO,iBAAiB,EAAE,UAAU,EAAE,QAAQ,CAAC,CAAC,CAC5D;IAEd,4FAA4F;IAC5F,WADW,QAAQ,OAAO,OAAO,iBAAiB,EAAE,UAAU,EAAE,YAAY,CAAC,CAAC,CAC3D;IAEnB,0DAA0D;IAC1D,kEAAoB;IAEpB,uBAAuB;IACvB,OADW,QAAQ,CACG;IAEtB,wBAAwB;IACxB,QADW,SAAS,CACQ;IA4DhC,uEAcC;IAED;;OAEG;IACH,4BAkKC;IAED,uBAqBC;IAED,iCAEC;CACJ;qBAlrC0D,eAAe;sBAFpD,uBAAuB;0BAGnB,oBAAoB;qBAGzB,eAAe;yBALX,mBAAmB;AAqrC5C;IAeI;;;OAGG;IACH,uBAHW,SAAS,8CA6FnB;IAhGD,uBAAmB;IAsCf;;;MAAoB;IAIpB;;;;MAQC;IAgDL,2BAKC;IAWD;;;;OAIG;IACH,gCAHW,SAAS,UACT,SAAS,QA8CnB;;CACJ;oBAx2CmB,qBAAqB"}
+{"version":3,"file":"gridView.d.ts","sourceRoot":"","sources":["../../../src/view/gridView.js"],"names":[],"mappings":"AAswBA;;;GAGG;AACH,iDAHW,OAAO,iBAAiB,EAAE,cAAc,GACtC,OAAO,iBAAiB,EAAE,QAAQ,CAwB9C;AAED;;;GAGG;AACH,uDAHW,OAAO,iBAAiB,EAAE,cAAc,GACtC,OAAO,iBAAiB,EAAE,QAAQ,CA6C9C;AA2BD;;GAEG;AACH,8EAUC;AAED;;;;;GAKG;AACH,4CAJW,OAAO,uBAAuB,EAAE,OAAO,UACvC,OAAO,iBAAiB,EAAE,UAAU,YACpC,QAAQ,aAmBlB;AA53BD;;;;;;;;;;;;;;;GAeG;AACH;IA6BI;;;;;;;;;OASG;IACH,kBARW,OAAO,iBAAiB,EAAE,aAAa,WACvC,OAAO,yBAAyB,EAAE,OAAO,gBACzC,aAAa,iDAEb,MAAM,WACN,MAAM,YACN,OAAO,WAAW,EAAE,WAAW,EAoBzC;IARG,8CAAgB;IAOhB,uBAA0B;IAG9B;;OAEG;IACH,qDAIC;IAeD;;OAEG;IACH,wDAKC;IAqBD;;OAEG;IACH,8CAEC;IAED,yBAEC;IAED;;OAEG;IACH,wCAmCC;;CA+jBJ;AAgJD;IACI;;;;OAIG;IACH,6DAHW,aAAa,UACb,MAAM,EAoFhB;IAjFG,4BAAgC;IAChC,kCAAgB;IAChB,eAAoB;IAEpB,uBAAuB;IACvB,YADW,QAAQ,CACQ;IAE3B,uBAAuB;IACvB,kBADW,QAAQ,CACc;IAEjC,mFAAmF;IACnF,MADW,QAAQ,OAAO,OAAO,iBAAiB,EAAE,UAAU,EAAE,QAAQ,CAAC,CAAC,CAC5D;IAEd,4FAA4F;IAC5F,WADW,QAAQ,OAAO,OAAO,iBAAiB,EAAE,UAAU,EAAE,YAAY,CAAC,CAAC,CAC3D;IAEnB,0DAA0D;IAC1D,kEAAoB;IAEpB,uBAAuB;IACvB,OADW,QAAQ,CACG;IAEtB,wBAAwB;IACxB,QADW,SAAS,CACQ;IA4DhC,uEAcC;IAED;;OAEG;IACH,4BAkKC;IAED,uBAqBC;IAED,iCAEC;CACJ;qBAlrC0D,eAAe;sBAFpD,uBAAuB;0BAGnB,oBAAoB;qBAGzB,eAAe;yBALX,mBAAmB;AAqrC5C;IAeI;;;OAGG;IACH,uBAHW,SAAS,8CA6FnB;IAhGD,uBAAmB;IAsCf;;;MAAoB;IAIpB;;;;MAQC;IAgDL,2BAKC;IAWD;;;;OAIG;IACH,gCAHW,SAAS,UACT,SAAS,QA8CnB;;CACJ;oBAx2CmB,qBAAqB"}

package/dist/src/view/gridView.js

@@ -835,7 +835,7 @@ export function createBackgroundStroke(viewBackground) {
             size: viewBackground.strokeWidth ?? 1.0,
             color: viewBackground.stroke ?? "lightgray",
             strokeCap: "square",
-            strokeOpacity: viewBackground.strokeOpacity ?? 1.0,
+            opacity: viewBackground.strokeOpacity ?? 1.0,
             type: "rule",
             clip: false,
             tooltip: null,

package/dist/src/view/layerView.d.ts

@@ -11,12 +11,6 @@ export default class LayerView extends ContainerView {
     constructor(spec: import("../spec/view.js").LayerSpec, context: import("../types/viewContext.js").default, layoutParent: ContainerView, dataParent: import("./view.js").default, name: string, options?: import("./view.js").ViewOptions);
     spec: import("../spec/view.js").LayerSpec;
     get children(): (import("./unitView.js").default | LayerView)[];
-    /**
-     * @param {import("./renderingContext/viewRenderingContext.js").default} context
-     * @param {import("./layout/rectangle.js").default} coords
-     * @param {import("../types/rendering.js").RenderingOptions} [options]
-     */
-    render(context: import("./renderingContext/viewRenderingContext.js").default, coords: import("./layout/rectangle.js").default, options?: import("../types/rendering.js").RenderingOptions): void;
     #private;
 }
 import ContainerView from "./containerView.js";

package/dist/src/view/layerView.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"layerView.d.ts","sourceRoot":"","sources":["../../../src/view/layerView.js"],"names":[],"mappings":"AAIA;IAQI;;;;;;;;OAQG;IACH,kBAPW,OAAO,iBAAiB,EAAE,SAAS,WACnC,OAAO,yBAAyB,EAAE,OAAO,gBACzC,aAAa,cACb,OAAO,WAAW,EAAE,OAAO,QAC3B,MAAM,YACN,OAAO,WAAW,EAAE,WAAW,EAQzC;IAHG,0CAAgB;IAoCpB,gEAEC;IAWD;;;;OAIG;IACH,gBAJW,OAAO,4CAA4C,EAAE,OAAO,UAC5D,OAAO,uBAAuB,EAAE,OAAO,YACvC,OAAO,uBAAuB,EAAE,gBAAgB,QAgB1D;;CAeJ;0BA1GyB,oBAAoB"}
+{"version":3,"file":"layerView.d.ts","sourceRoot":"","sources":["../../../src/view/layerView.js"],"names":[],"mappings":"AAIA;IAQI;;;;;;;;OAQG;IACH,kBAPW,OAAO,iBAAiB,EAAE,SAAS,WACnC,OAAO,yBAAyB,EAAE,OAAO,gBACzC,aAAa,cACb,OAAO,WAAW,EAAE,OAAO,QAC3B,MAAM,YACN,OAAO,WAAW,EAAE,WAAW,EAQzC;IAHG,0CAAgB;IAoCpB,gEAEC;;CA6CJ;0BA1GyB,oBAAoB"}

package/dist/src/view/layout/flexLayout.test.d.ts

@@ -0,0 +1,2 @@
+export {};
+//# sourceMappingURL=flexLayout.test.d.ts.map

package/dist/src/view/layout/flexLayout.test.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"flexLayout.test.d.ts","sourceRoot":"","sources":["../../../../src/view/layout/flexLayout.test.js"],"names":[],"mappings":""}

package/dist/src/view/layout/grid.test.d.ts

@@ -0,0 +1,2 @@
+export {};
+//# sourceMappingURL=grid.test.d.ts.map

package/dist/src/view/layout/grid.test.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"grid.test.d.ts","sourceRoot":"","sources":["../../../../src/view/layout/grid.test.js"],"names":[],"mappings":""}

package/dist/src/view/layout/rectangle.test.d.ts

@@ -0,0 +1,2 @@
+export {};
+//# sourceMappingURL=rectangle.test.d.ts.map

package/dist/src/view/layout/rectangle.test.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"rectangle.test.d.ts","sourceRoot":"","sources":["../../../../src/view/layout/rectangle.test.js"],"names":[],"mappings":""}

package/dist/src/view/paramMediator.d.ts

@@ -11,7 +11,17 @@ export function isExprRef(x: any): x is import("../spec/parameter.js").ExprRef;
  * @template T
  * @returns {T}
  */
-export function withoutExprRef<T>(x: T | import("../spec/parameter.js").ExprRef): T;
+export function withoutExprRef<T>(x: import("../spec/parameter.js").ExprRef | T): T;
+/**
+ * @param {Parameter} param
+ * @returns {param is import("../spec/parameter.js").VariableParameter}
+ */
+export function isVariableParameter(param: import("../spec/parameter.js").Parameter): param is import("../spec/parameter.js").VariableParameter;
+/**
+ * @param {Parameter} param
+ * @returns {param is import("../spec/parameter.js").SelectionParameter}
+ */
+export function isSelectionParameter(param: import("../spec/parameter.js").Parameter): param is import("../spec/parameter.js").SelectionParameter<import("../spec/parameter.js").SelectionType>;
 /**
  * Takes a record of properties that may have ExprRefs as values. Converts the
  * ExprRefs to getters and setups a listener that is called when any of the
@@ -24,6 +34,23 @@ export function withoutExprRef<T>(x: T | import("../spec/parameter.js").ExprRef)
  * @template {Record<string, any | import("../spec/parameter.js").ExprRef>} T
  */
 export function activateExprRefProps<T extends Record<string, any>>(paramMediator: ParamMediator, props: T, listener?: (props: (keyof T)[]) => void): T;
+/**
+ * Validates a parameter name. If the name is invalid, throws an error.
+ * Otherwise, returns the name.
+ *
+ * @param {string} name
+ * @returns {string} the name
+ */
+export function validateParameterName(name: string): string;
+/**
+ * Creates a function that always returns the same value.
+ * Provides functionality for creating a constant expression reference.
+ * They just do nothing.
+ *
+ * @param {any} value
+ * @returns {ExprRefFunction}
+ */
+export function makeConstantExprRef(value: any): ExprRefFunction;
 /**
  * A class that manages parameters and expressions.
  * Supports nesting and scoped parameters.
@@ -44,10 +71,10 @@ export default class ParamMediator {
      */
     protected paramListeners: Map<string, Set<() => void>>;
     /**
-     * @param {VariableParameter} param
+     * @param {Parameter} param
      * @returns {ParameterSetter}
      */
-    registerParam(param: import("../spec/parameter.js").VariableParameter): (value: any) => void;
+    registerParam(param: import("../spec/parameter.js").Parameter): (value: any) => void;
     /**
      *
      * @param {string} paramName
@@ -75,7 +102,7 @@ export default class ParamMediator {
     /**
      * Returns configs for all parameters that have been registered using `registerParam`.
      */
-    get paramConfigs(): ReadonlyMap<string, import("../spec/parameter.js").VariableParameter>;
+    get paramConfigs(): ReadonlyMap<string, import("../spec/parameter.js").Parameter>;
     /**
      *
      * @param {string} paramName
@@ -95,13 +122,20 @@ export default class ParamMediator {
      * @param {string} expr
      */
     evaluateAndGet(expr: string): any;
+    /**
+     * Returns true if this ParamMediator has any parameters that are point selections.
+     * Point selections necessitate the use of uniqueIds in the data.
+     *
+     * @returns {boolean}
+     */
+    hasPointSelections(): boolean;
     #private;
 }
 /**
  * A class that manages parameters and expressions.
  * Supports nesting and scoped parameters.
  */
-export type ExprRefFunction = ((datum?: object) => any) & import("../utils/expression.js").ExpressionProps & {
+export type ExprRefFunction = ((datum?: import("../data/flowNode.js").Datum) => any) & import("../utils/expression.js").ExpressionProps & {
     addListener: (listener: () => void) => void;
     invalidate: () => void;
     identifier: () => string;

package/dist/src/view/paramMediator.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"paramMediator.d.ts","sourceRoot":"","sources":["../../../src/view/paramMediator.js"],"names":[],"mappings":"AAuQA;;;GAGG;AACH,6BAHW,GAAG,+CAKb;AAED;;;;;;;GAOG;AACH,oFASC;AAED;;;;;;;;;;GAUG;AACH,mFANW,aAAa,+CAEW,IAAI,KAwCtC;AA9UD;;;;;GAKG;AACH;IA2BI;;;;;OAKG;IACH,2BALW,MAAM,aAAa,EAU7B;IA7BD;;;OAGG;IACH,0BAHU,IAAI,MAAM,EAAE,IAAI,MAAM,IAAI,CAAC,CAAC,CAGvB;IA2Bf;;;OAGG;IACH,gFAzCqB,GAAG,KAAK,IAAI,CAkEhC;IAED;;;;;;;OAOG;IACH,6BANW,MAAM,6BAEN,OAAO,iBACS,IAAI,CA8B9B;IAED;;;OAGG;IACH,qBAFW,MAAM,WA/Fc,GAAG,KAAK,IAAI,CAuG1C;IAED;;;OAGG;IACH,oBAFW,MAAM,OAIhB;IAED;;;OAGG;IACH,qBAFW,MAAM,OAKhB;IAED;;OAEG;IACH,0FAIC;IAED;;;;;OAKG;IACH,0CAJW,MAAM,GACJ,aAAa,CASzB;IAID;;;;OAIG;IACH,uBAFW,MAAM,mBA4EhB;IAED;;;;OAIG;IACH,qBAFW,MAAM,OAKhB;;CACJ;;;;;;4BA9P4F,MAAM,IAAI,KAAK,IAAI;gBAAc,MAAM,IAAI;gBAAc,MAAM,MAAM"}
+{"version":3,"file":"paramMediator.d.ts","sourceRoot":"","sources":["../../../src/view/paramMediator.js"],"names":[],"mappings":"AAkUA;;;GAGG;AACH,6BAHW,GAAG,+CAKb;AAED;;;;;;;GAOG;AACH,oFASC;AAED;;;GAGG;AACH,gJAEC;AAED;;;GAGG;AACH,gMAEC;AAED;;;;;;;;;;GAUG;AACH,mFANW,aAAa,+CAEW,IAAI,KAwCtC;AAED;;;;;;GAMG;AACH,4CAHW,MAAM,GACJ,MAAM,CAUlB;AAED;;;;;;;GAOG;AACH,2CAHW,GAAG,GACD,eAAe,CAW3B;AA5bD;;;;;GAKG;AACH;IA2BI;;;;;OAKG;IACH,2BALW,MAAM,aAAa,EAU7B;IA7BD;;;OAGG;IACH,0BAHU,IAAI,MAAM,EAAE,IAAI,MAAM,IAAI,CAAC,CAAC,CAGvB;IA2Bf;;;OAGG;IACH,wEAzCqB,GAAG,KAAK,IAAI,CAmGhC;IAED;;;;;;;OAOG;IACH,6BANW,MAAM,6BAEN,OAAO,iBACS,IAAI,CAgC9B;IAED;;;OAGG;IACH,qBAFW,MAAM,WAlIc,GAAG,KAAK,IAAI,CA0I1C;IAED;;;OAGG;IACH,oBAFW,MAAM,OAIhB;IAED;;;OAGG;IACH,qBAFW,MAAM,OAKhB;IAED;;OAEG;IACH,kFAIC;IAED;;;;;OAKG;IACH,0CAJW,MAAM,GACJ,aAAa,CASzB;IAID;;;;OAIG;IACH,uBAFW,MAAM,mBA4EhB;IAED;;;;OAIG;IACH,qBAFW,MAAM,OAKhB;IAED;;;;;OAKG;IACH,sBAFa,OAAO,CAiBnB;;CACJ;;;;;;4BAxT4F,MAAM,IAAI,KAAK,IAAI;gBAAc,MAAM,IAAI;gBAAc,MAAM,MAAM"}

package/dist/src/view/paramMediator.js

@@ -1,5 +1,6 @@
 import { isString } from "vega-util";
 import createFunction from "../utils/expression.js";
+import { createSinglePointSelection } from "../selection/selection.js";
 
 /**
  * A class that manages parameters and expressions.
@@ -9,7 +10,7 @@ import createFunction from "../utils/expression.js";
  */
 export default class ParamMediator {
     /**
-     * @typedef {import("../spec/parameter.js").VariableParameter} VariableParameter
+     * @typedef {import("../spec/parameter.js").Parameter} Parameter
      * @typedef {(value: any) => void} ParameterSetter
      */
 
@@ -28,7 +29,7 @@ export default class ParamMediator {
     /** @type {Map<string, ExprRefFunction>} */
     #expressions = new Map();
 
-    /** @type {Map<string, VariableParameter>} */
+    /** @type {Map<string, Parameter>} */
     #paramConfigs = new Map();
 
     /** @type {() => ParamMediator} */
@@ -48,32 +49,65 @@ export default class ParamMediator {
     }
 
     /**
-     * @param {VariableParameter} param
+     * @param {Parameter} param
      * @returns {ParameterSetter}
      */
     registerParam(param) {
+        const name = param.name;
+
         if ("value" in param && "expr" in param) {
             throw new Error(
-                "Parameter must not have both value and expr: " + param.name
+                `The parameter "${name}" must not have both value and expr properties!`
             );
         }
 
         /** @type {ParameterSetter} */
         let setter;
 
-        if ("value" in param) {
-            setter = this.allocateSetter(param.name, param.value);
+        if (param.push == "outer") {
+            const outerMediator = this.findMediatorForParam(name);
+            if (!outerMediator) {
+                throw new Error(
+                    `Parameter "${name}" not found in outer scope!`
+                );
+            }
+
+            const outerProps = outerMediator.paramConfigs.get(name);
+            if ("expr" in outerProps || "select" in outerProps) {
+                throw new Error(
+                    `The outer parameter "${name}" must not have expr or select properties!`
+                );
+            }
+            setter = outerMediator.getSetter(name);
+            // The following will become a bit fragile if the view hierarchy is going to
+            // support mutation (i.e. adding/removing children) in future.
+            this.#allocatedSetters.set(name, setter);
+        } else if ("value" in param) {
+            setter = this.allocateSetter(name, param.value);
         } else if ("expr" in param) {
             const expr = this.createExpression(param.expr);
             // TODO: getSetter(param) should return a setter that throws if
             // modifying the value is attempted.
-            const realSetter = this.allocateSetter(param.name, expr(null));
+            const realSetter = this.allocateSetter(name, expr(null));
             expr.addListener(() => realSetter(expr(null)));
             // NOP
             setter = (_) => undefined;
+        } else {
+            setter = this.allocateSetter(name, null);
         }
 
-        this.#paramConfigs.set(param.name, param);
+        if ("select" in param) {
+            const type = isString(param.select)
+                ? param.select
+                : param.select.type;
+
+            // Set initial value so that production rules in shaders can be generated, etc.
+            if (type == "point") {
+                setter(createSinglePointSelection(null));
+            }
+        }
+
+        this.#paramConfigs.set(name, param);
 
         return setter;
     }
@@ -87,6 +121,8 @@ export default class ParamMediator {
      * @template T
      */
     allocateSetter(paramName, initialValue, passive = false) {
+        validateParameterName(paramName);
+
         if (this.#allocatedSetters.has(paramName)) {
             throw new Error(
                 "Setter already allocated for parameter: " + paramName
@@ -148,7 +184,7 @@ export default class ParamMediator {
      * Returns configs for all parameters that have been registered using `registerParam`.
      */
     get paramConfigs() {
-        return /** @type {ReadonlyMap<string, VariableParameter>} */ (
+        return /** @type {ReadonlyMap<string, Parameter>} */ (
             this.#paramConfigs
         );
     }
@@ -259,6 +295,29 @@ export default class ParamMediator {
         const fn = this.createExpression(expr);
         return fn();
     }
+
+    /**
+     * Returns true if this ParamMediator has any parameters that are point selections.
+     * Point selections necessitate the use of uniqueIds in the data.
+     *
+     * @returns {boolean}
+     */
+    hasPointSelections() {
+        for (const param of this.#paramConfigs.values()) {
+            if (isSelectionParameter(param)) {
+                const select = param.select;
+                if (isString(select)) {
+                    if (select == "point") {
+                        return true;
+                    }
+                } else if (select.type == "point") {
+                    return true;
+                }
+            }
+        }
+
+        return false;
+    }
 }
 
 /**
@@ -288,6 +347,22 @@ export function withoutExprRef(x) {
     return /** @type {T} */ (x);
 }
 
+/**
+ * @param {Parameter} param
+ * @returns {param is import("../spec/parameter.js").VariableParameter}
+ */
+export function isVariableParameter(param) {
+    return ("expr" in param || "bind" in param) && !("select" in param);
+}
+
+/**
+ * @param {Parameter} param
+ * @returns {param is import("../spec/parameter.js").SelectionParameter}
+ */
+export function isSelectionParameter(param) {
+    return !("expr" in param || "bind" in param) && "select" in param;
+}
+
 /**
  * Takes a record of properties that may have ExprRefs as values. Converts the
  * ExprRefs to getters and setups a listener that is called when any of the
@@ -336,3 +411,39 @@ export function activateExprRefProps(paramMediator, props, listener) {
 
     return /** @type {T} */ (activatedProps);
 }
+
+/**
+ * Validates a parameter name. If the name is invalid, throws an error.
+ * Otherwise, returns the name.
+ *
+ * @param {string} name
+ * @returns {string} the name
+ */
+export function validateParameterName(name) {
+    if (!/^[a-zA-Z_$][0-9a-zA-Z_$]*$/.test(name)) {
+        throw new Error(
+            `Invalid parameter name: ${name}. Must be a valid JavaScript identifier.`
+        );
+    }
+
+    return name;
+}
+
+/**
+ * Creates a function that always returns the same value.
+ * Provides functionality for creating a constant expression reference.
+ * They just do nothing.
+ *
+ * @param {any} value
+ * @returns {ExprRefFunction}
+ */
+export function makeConstantExprRef(value) {
+    return Object.assign(() => value, {
+        addListener: () => undefined,
+        invalidate: () => undefined,
+        identifier: () => "constant",
+        fields: [],
+        globals: [],
+        code: JSON.stringify(value),
+    });
+}

package/dist/src/view/paramMediator.test.d.ts

@@ -0,0 +1,2 @@
+export {};
+//# sourceMappingURL=paramMediator.test.d.ts.map

package/dist/src/view/paramMediator.test.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"paramMediator.test.d.ts","sourceRoot":"","sources":["../../../src/view/paramMediator.test.js"],"names":[],"mappings":""}

package/dist/src/view/paramMediator.test.js

@@ -173,6 +173,22 @@ describe("Nested ParamMediators", () => {
         childSetter(20);
         expect(result).toBe(30);
     });
+
+    test("Pushing to outer parameter", () => {
+        const parent = new ParamMediator();
+        const child = new ParamMediator(() => parent);
+
+        parent.registerParam({ name: "foo", value: 1 });
+        const childSetter = child.registerParam({ name: "foo", push: "outer" });
+
+        expect(parent.findValue("foo")).toBe(1);
+        expect(child.findValue("foo")).toBe(1);
+
+        childSetter(2);
+
+        expect(parent.findValue("foo")).toBe(2);
+        expect(child.findValue("foo")).toBe(2);
+    });
 });
 
 test("activateExprRefProps", async () => {
@@ -181,7 +197,7 @@ test("activateExprRefProps", async () => {
     const fooSetter = pm.registerParam({ name: "foo", value: 7 });
     const barSetter = pm.registerParam({ name: "bar", value: 11 });
 
-    /** @type {Record<string, any | import("../spec/parameter.js").ExprRef} */
+    /** @type {Record<string, any | import("../spec/parameter.js").ExprRef>} */
     const props = {
         a: 42,
         b: { expr: "foo" },
@@ -222,3 +238,23 @@ test("activateExprRefProps", async () => {
         c: 2,
     });
 });
+
+describe("hasPointSelections()", () => {
+    test("false if there are no point selections", () => {
+        const pm = new ParamMediator();
+        pm.registerParam({ name: "foo", value: 42 });
+        expect(pm.hasPointSelections()).toBe(false);
+    });
+
+    test("true if there are point selections (1/2)", () => {
+        const pm = new ParamMediator();
+        pm.registerParam({ name: "foo", select: "point" });
+        expect(pm.hasPointSelections()).toBe(true);
+    });
+
+    test("true if there are point selections (2/2)", () => {
+        const pm = new ParamMediator();
+        pm.registerParam({ name: "foo", select: { type: "point" } });
+        expect(pm.hasPointSelections()).toBe(true);
+    });
+});

package/dist/src/view/scaleResolution.d.ts

@@ -17,8 +17,12 @@ export const LOCUS: "locus";
 export const INDEX: "index";
 /**
  * @template {ChannelWithScale}[T=ChannelWithScale]
- * @typedef {{view: import("./unitView.js").default, channel: T}} ResolutionMember
  *
+ * @typedef {object} ScaleResolutionMember
+ * @prop {import("./unitView.js").default} view TODO: Get rid of the view reference
+ * @prop {T} channel
+ * @prop {import("../spec/channel.js").ChannelDefWithScale} channelDef
+ * @prop {(channel: ChannelWithScale, type: import("../spec/channel.js").Type) => DomainArray} dataDomainSource
  */
 /**
  * Resolution takes care of merging domains and scales from multiple views.
@@ -34,10 +38,10 @@ export default class ScaleResolution implements ScaleResolutionApi {
      */
     constructor(channel: import("../spec/channel.js").Channel);
     channel: import("../spec/channel.js").Channel;
-    /** @type {ResolutionMember[]} The involved views */
-    members: ResolutionMember[];
-    /** @type {string} Data type (quantitative, nominal, etc...) */
-    type: string;
+    /** @type {ScaleResolutionMember[]} The involved views */
+    members: ScaleResolutionMember[];
+    /** @type {import("../spec/channel.js").Type} Data type (quantitative, nominal, etc...) */
+    type: import("../spec/channel.js").Type;
     /** @type {string} An optional unique identifier for the scale */
     name: string;
     /**
@@ -58,16 +62,9 @@ export default class ScaleResolution implements ScaleResolutionApi {
      * Add a view to this resolution.
      * N.B. This is expected to be called in depth-first order
      *
-     * @param {UnitView} view
-     * @param {ChannelWithScale} channel
+     * @param {ScaleResolutionMember} newMember
      */
-    pushUnitView(view: import("./unitView.js").default, channel: import("../spec/channel.js").ChannelWithScale): void;
-    /**
-     * Unions the configured domains of all participating views.
-     *
-     * @return { DomainArray }
-     */
-    getConfiguredDomain(): import("../utils/domainArray.js").DomainArray;
+    addMember(newMember: ScaleResolutionMember): void;
     /**
      * Extracts and unions the data domains of all participating views.
      *
@@ -157,8 +154,13 @@ export default class ScaleResolution implements ScaleResolutionApi {
     fromComplexInterval(interval: import("../spec/scale.js").ScalarDomain | import("../spec/scale.js").ComplexDomain): number[];
     #private;
 }
-export type ResolutionMember<T extends import("../spec/channel.js").ChannelWithScale = import("../spec/channel.js").ChannelWithScale> = {
+export type ScaleResolutionMember<T extends import("../spec/channel.js").ChannelWithScale = import("../spec/channel.js").ChannelWithScale> = {
+    /**
+     * TODO: Get rid of the view reference
+     */
     view: import("./unitView.js").default;
     channel: T;
+    channelDef: import("../spec/channel.js").ChannelDefWithScale;
+    dataDomainSource: (channel: import("../spec/channel.js").ChannelWithScale, type: import("../spec/channel.js").Type) => import("../utils/domainArray.js").DomainArray;
 };
 //# sourceMappingURL=scaleResolution.d.ts.map

package/dist/src/view/scaleResolution.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"scaleResolution.d.ts","sourceRoot":"","sources":["../../../src/view/scaleResolution.js"],"names":[],"mappings":"AA86BA;;;;;;;;;;GAUG;AACH,6CAFW,OAAO,WAAW,EAAE,OAAO,GAAG,OAAO,WAAW,EAAE,OAAO,EAAE,QA4BrE;AAr6BD,0CAA2C;AAC3C,gCAAiC;AACjC,gCAAiC;AACjC,4BAA6B;AAC7B,4BAA6B;AAE7B;;;;GAIG;AACH;;;;;;;GAOG;AACH;IAyCI;;OAEG;IACH,2DASC;IARG,8CAAsB;IACtB,oDAAoD;IACpD,SADW,gBAAgB,EAAE,CACZ;IACjB,+DAA+D;IAC/D,MADW,MAAM,CACD;IAEhB,iEAAiE;IACjE,MADW,MAAM,CACI;IAWzB;;;;;;;OAOG;IACH,4KAEC;IAED;;;OAGG;IACH,+KAEC;IAcD;;;;;;OAMG;IACH,kHA0BC;IAmLD;;;;OAIG;IACH,qEAMC;IAED;;;;OAIG;IACH,+DAQC;IAED;;OAEG;IACH,oBAyCC;IAED;;OAEG;IACH;eAhXkC,OAAO,kBAAkB,EAAE,KAAK;MAuZjE;IAED,mBAEC;IAED;;OAEG;IACH,oBAFa,mFAA6B,CAOzC;IAED;;;;OAIG;IACH,oBAKC;IAED;;OAEG;IACH,sBAGC;IAUD;;;;;;;OAOG;IACH,kBALW,MAAM,eACN,MAAM,OACN,MAAM,GACJ,OAAO,CAwEnB;IAED;;;;;;OAMG;IACH,eAJW,mFAA6B,aAC7B,OAAO,GAAG,MAAM,iBAgD1B;IAED;;;;OAIG;IACH,qBAcC;IAED;;;;;OAKG;IACH,uBAOC;IA8DD;;;OAGG;IACH,aAFa,OAAO,qBAAqB,EAAE,OAAO,CAajD;IAID;;;;;OAKG;IACH,uBAFW,MAAM,yDAUhB;IAED;;OAEG;IACH,iBAFW,MAAM,yDAKhB;IAED;;;OAGG;IACH,6EAFa,MAAM,CAQlB;IAED;;;OAGG;IACH,mHAFa,MAAM,EAAE,CAOpB;;CA6BJ;wIAjyBY;IAAC,IAAI,EAAE,OAAO,eAAe,EAAE,OAAO,CAAC;IAAC,OAAO,EAAE,CAAC,CAAA;CAAC"}
+{"version":3,"file":"scaleResolution.d.ts","sourceRoot":"","sources":["../../../src/view/scaleResolution.js"],"names":[],"mappings":"AAg7BA;;;;;;;;;;GAUG;AACH,6CAFW,OAAO,WAAW,EAAE,OAAO,GAAG,OAAO,WAAW,EAAE,OAAO,EAAE,QA4BrE;AAx6BD,0CAA2C;AAC3C,gCAAiC;AACjC,gCAAiC;AACjC,4BAA6B;AAC7B,4BAA6B;AAE7B;;;;;;;;GAQG;AACH;;;;;;;GAOG;AACH;IAyCI;;OAEG;IACH,2DASC;IARG,8CAAsB;IACtB,yDAAyD;IACzD,SADW,qBAAqB,EAAE,CACjB;IACjB,0FAA0F;IAC1F,MADW,OAAO,oBAAoB,EAAE,IAAI,CAC5B;IAEhB,iEAAiE;IACjE,MADW,MAAM,CACI;IAWzB;;;;;;;OAOG;IACH,4KAEC;IAED;;;OAGG;IACH,+KAEC;IAcD;;;;;OAKG;IACH,qBAFW,qBAAqB,QA4B/B;IA8MD;;;;OAIG;IACH,+DAOC;IAED;;OAEG;IACH,oBAyCC;IAED;;OAEG;IACH;eA5XkC,OAAO,kBAAkB,EAAE,KAAK;MAwajE;IAED,mBAEC;IAED;;OAEG;IACH,oBAFa,mFAA6B,CAOzC;IAED;;;;OAIG;IACH,oBAKC;IAED;;OAEG;IACH,sBAGC;IAUD;;;;;;;OAOG;IACH,kBALW,MAAM,eACN,MAAM,OACN,MAAM,GACJ,OAAO,CAwEnB;IAED;;;;;;OAMG;IACH,eAJW,mFAA6B,aAC7B,OAAO,GAAG,MAAM,iBA0D1B;IAED;;;;OAIG;IACH,qBAcC;IAED;;;;;OAKG;IACH,uBAOC;IA8DD;;;OAGG;IACH,aAFa,OAAO,qBAAqB,EAAE,OAAO,CAajD;IAID;;;;;OAKG;IACH,uBAFW,MAAM,yDAUhB;IAED;;OAEG;IACH,iBAFW,MAAM,yDAKhB;IAED;;;OAGG;IACH,6EAFa,MAAM,CAQlB;IAED;;;OAGG;IACH,mHAFa,MAAM,EAAE,CAOpB;;CACJ;;;;;UAlyBS,OAAO,eAAe,EAAE,OAAO;aAC/B,CAAC;gBACD,OAAO,oBAAoB,EAAE,mBAAmB;qFACd,OAAO,oBAAoB,EAAE,IAAI"}