Fetch attribute values on demand in resolvePath

packages/app/src/__tests__/NexusPack.test.tsx

@@ -89,6 +89,24 @@ test('visualize NXdata group with "rgb-image" interpretation', async () => {
   expect(tabs[0]).toHaveTextContent(NexusVis.NxRGB);
 });
 
+test('visualize NXdata group with unknown interpretation', async () => {
+  await renderApp();
+  await selectExplorerNode('nexus_entry/unknown');
+
+  const tabs = await findVisSelectorTabs();
+  expect(tabs).toHaveLength(1);
+  expect(tabs[0]).toHaveTextContent(NexusVis.NxImage); // support check falls back on signal dataset dimensions
+});
+
+test('visualize NXdata group with "rgb-image" interpretation but incompatible signal', async () => {
+  await renderApp();
+  await selectExplorerNode('nexus_entry/rgb_incompatible');
+
+  const tabs = await findVisSelectorTabs();
+  expect(tabs).toHaveLength(1);
+  expect(tabs[0]).toHaveTextContent(NexusVis.NxSpectrum); // support check falls back on signal dataset dimensions
+});
+
 test('visualize NXentry group with implicit default child NXdata group', async () => {
   await renderApp();
   await selectExplorerNode('nexus_no_default');

packages/app/src/vis-packs/nexus/models.ts

@@ -23,12 +23,6 @@ export type NxAttribute =
   | 'SILX_style'
   | 'auxiliary_signals';
 
-export enum NxInterpretation {
-  Spectrum = 'spectrum',
-  Image = 'image',
-  RGB = 'rgb-image',
-}
-
 export interface NxData<
   T extends NumericType | ComplexType = NumericType | ComplexType
 > {

packages/app/src/visualizer/Visualizer.tsx

@@ -15,7 +15,7 @@ interface Props {
 function Visualizer(props: Props) {
   const { path } = props;
 
-  const { entitiesStore } = useContext(ProviderContext);
+  const { entitiesStore, attrValuesStore } = useContext(ProviderContext);
 
   function getEntity(entityPath: string): Entity {
     return handleError(
@@ -25,7 +25,7 @@ function Visualizer(props: Props) {
     );
   }
 
-  const resolution = resolvePath(path, getEntity);
+  const resolution = resolvePath(path, getEntity, attrValuesStore);
 
   if (!resolution) {
     return (

packages/app/src/visualizer/utils.ts

@@ -1,4 +1,4 @@
-import type { Entity } from '@h5web/shared';
+import type { AttributeValues, Entity } from '@h5web/shared';
 import {
   assertGroupWithChildren,
   hasComplexType,
@@ -8,10 +8,10 @@ import {
   isDataset,
   assertStr,
   buildEntityPath,
+  NxInterpretation,
 } from '@h5web/shared';
-import { NxInterpretation } from '@h5web/shared/src/models-nexus';
+import type { FetchStore } from 'react-suspense-fetch';
 
-import { getAttributeValue } from '../utils';
 import type { CoreVisDef } from '../vis-packs/core/visualizations';
 import { Vis, CORE_VIS } from '../vis-packs/core/visualizations';
 import type { VisDef } from '../vis-packs/models';
@@ -24,38 +24,45 @@ import { NexusVis, NEXUS_VIS } from '../vis-packs/nexus/visualizations';
 
 export function resolvePath(
   path: string,
-  getEntity: (path: string) => Entity
+  getEntity: (path: string) => Entity,
+  attrValueStore: FetchStore<AttributeValues, Entity>
 ): { entity: Entity; supportedVis: VisDef[] } | undefined {
   const entity = getEntity(path);
 
-  const supportedVis = findSupportedVis(entity);
+  const supportedVis = findSupportedVis(entity, attrValueStore);
   if (supportedVis.length > 0) {
     return { entity, supportedVis };
   }
 
-  const nxDefaultPath = getNxDefaultPath(entity);
+  const nxDefaultPath = getNxDefaultPath(entity, attrValueStore);
   if (nxDefaultPath) {
-    return resolvePath(nxDefaultPath, getEntity);
+    return resolvePath(nxDefaultPath, getEntity, attrValueStore);
   }
 
   return undefined;
 }
 
-function findSupportedVis(entity: Entity): VisDef[] {
-  const nxVis = getSupportedNxVis(entity);
+function findSupportedVis(
+  entity: Entity,
+  attrValueStore: FetchStore<AttributeValues, Entity>
+): VisDef[] {
+  const nxVis = getSupportedNxVis(entity, attrValueStore);
   if (nxVis) {
     return [nxVis];
   }
 
   return getSupportedCoreVis(entity);
 }
 
-function getNxDefaultPath(entity: Entity): string | undefined {
+function getNxDefaultPath(
+  entity: Entity,
+  attrValueStore: FetchStore<AttributeValues, Entity>
+): string | undefined {
   if (!isGroup(entity)) {
     return undefined;
   }
 
-  const defaultPath = getAttributeValue(entity, 'default');
+  const { default: defaultPath } = attrValueStore.get(entity);
 
   if (defaultPath) {
     assertStr(defaultPath, `Expected 'default' attribute to be a string`);
@@ -69,7 +76,7 @@ function getNxDefaultPath(entity: Entity): string | undefined {
   return getImplicitDefaultChild(entity.children)?.path;
 }
 
-export function getSupportedCoreVis(entity: Entity): CoreVisDef[] {
+function getSupportedCoreVis(entity: Entity): CoreVisDef[] {
   const supportedVis = Object.values(CORE_VIS).filter(
     (vis) => isDataset(entity) && vis.supportsDataset(entity)
   );
@@ -79,15 +86,18 @@ export function getSupportedCoreVis(entity: Entity): CoreVisDef[] {
     : supportedVis;
 }
 
-export function getSupportedNxVis(entity: Entity): VisDef | undefined {
+function getSupportedNxVis(
+  entity: Entity,
+  attrValueStore: FetchStore<AttributeValues, Entity>
+): VisDef | undefined {
   if (!isGroup(entity) || !isNxDataGroup(entity)) {
     return undefined;
   }
 
   assertGroupWithChildren(entity);
   const dataset = findSignalDataset(entity);
   const isCplx = hasComplexType(dataset);
-  const interpretation = getAttributeValue(dataset, 'interpretation');
+  const { interpretation } = attrValueStore.get(dataset);
 
   if (
     interpretation === NxInterpretation.RGB &&

packages/shared/src/index.ts

@@ -1,5 +1,6 @@
 export * from './guards';
 export * from './models-hdf5';
+export * from './models-nexus';
 export * from './models-vis';
 export * from './utils';
 export * from './mock/metadata';

packages/shared/src/mock/metadata.ts

@@ -149,6 +149,16 @@ export const mockMetadata = makeNxGroup(mockFilepath, 'NXroot', {
             })
           ),
         }),
+        makeNxDataGroup('unknown', {
+          signal: makeNxDataset('fourD', intType, [3, 9, 20, 41], {
+            interpretation: 'unknown',
+          }),
+        }),
+        makeNxDataGroup('rgb_incompatible', {
+          signal: makeNxDataset('oneD', intType, [41], {
+            interpretation: 'rgb-image',
+          }),
+        }),
       ],
     }),
     makeNxGroup('nexus_no_default', 'NXprocess', {