Skip to content

Conversation

Jarbuckle
Copy link
Contributor

[Proposal] Revised OME-Zarr data type hierarchy and interface

What

Detailed discussion here:

In brief, this is a proposed new data type hierarchy and loading/access interface for OME-Zarr data. Its primary characteristics:

  • Raw Zarrita constructs accessed and loaded via OmeZarrConnection
  • Clean and direct access to OME-Zarr metadata via OmeZarrFileset
  • Contextual level-of-detail data accessed via OmeZarrLevel
  • Revised Zarr Slice rendering pipeline that uses these new data types and their methods

PR Checklist

  • Is your PR title following our conventional commit naming recommendations?
  • Have you filled in the PR Description Template?
  • Is your branch up to date with the latest in main?
  • Do the CI checks pass successfully?
  • Have you smoke tested the example applications?
  • Did you check that the changes meet accessibility standards?
  • Have you tested the application on these browsers?
    • Chrome (Fully supported)
    • Firefox (Major bug fixes supported)
    • Safari (Major bug fixes supported)

Comment on lines 291 to 314
/**
* HEY!!!
* this is an example React Component for rendering A single slice of an OMEZARR image in a react component
* This example is as bare-bones as possible! It is NOT the recommended way to do anything, its just trying to show
* one way of:
* 1. using our rendering utilities for OmeZarr data, specifically in a react component. Your needs for state-management,
* slicing logic, etc might all be different!
*
*/
function DataPlease() {
// load our canned data for now:
const [omezarr, setfile] = useState<OmeZarrMetadata | undefined>(undefined);
useEffect(() => {
loadMetadata(demoOptions[0].res).then((dataset) => {
setfile(dataset);
logger.info('loaded!');
});
}, []);
return (
<RenderServerProvider>
<SliceView omezarr={omezarr} />
</RenderServerProvider>
);
}
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Moved this to its own file, basic-omezarr-demo.tsx

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants