Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Image viewer for package demos #1070

Merged
merged 8 commits into from
Apr 23, 2024

Conversation

PaulHax
Copy link
Collaborator

@PaulHax PaulHax commented Feb 21, 2024

  • Inject into outputs
  • Add dep to template.package.json
  • Update existing package.json's with new deps
  • Update packages/vite.config.js's
  • Put viewer inside the image details element. Render viewer when details is expanded
  • Extract image details/viewer into a itk-image-details component
  • Include utilities.js in demo-app package?
  • update itk-viewer/element to fix 2d orienation
  • Check all demo app for all pipelines
  • Fix itk-viewer/element 2d viwere for sample datasets.

Dev steps to rebuild demo web app

Make edits in packages/core/typescript/itk-wasm/src/bindgen/typescript/demo

Then in a package, ie packages/downsample

pnpm build

Then in typescript dir, ie packages/downsample/typescript

pnpm start

@PaulHax PaulHax force-pushed the demo-image-view branch 15 times, most recently from d9f0331 to 692d6d8 Compare March 11, 2024 19:54
@PaulHax PaulHax marked this pull request as ready for review March 11, 2024 22:14
@PaulHax PaulHax force-pushed the demo-image-view branch 2 times, most recently from ec0eccb to 0743f0a Compare April 2, 2024 21:18
"@itk-viewer/element": "^0.2.12",
"@itk-viewer/io": "^0.1.8",
"@shoelace-style/shoelace": "^2.12.0",
"itk-wasm": "1.0.0-b.170",
Copy link
Member

Choose a reason for hiding this comment

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

workspace:

@PaulHax PaulHax requested a review from thewtex April 4, 2024 18:47
@PaulHax
Copy link
Collaborator Author

PaulHax commented Apr 4, 2024

This is worth a review.

The readImageDicomFileSeries pipeline sample input does not look good. Need to update slice axis in viewer.

Copy link
Member

@thewtex thewtex left a comment

Choose a reason for hiding this comment

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

Excellent!

Just a few minor issues noted inline.

@@ -89,7 +61,8 @@ <h3>👨‍💻 Live API Demo ✨</h3>
<sl-details disabled id="compareImages-metrics-details" summary="metrics: Metrics for the baseline with the fewest number of pixels outside the tolerances."></sl-details>
<sl-button variant="neutral" outline name="metrics-download" disabled>Download</sl-button>
<br /><br />
<sl-details disabled id="compareImages-difference-image-details" summary="differenceImage: Absolute difference image"></sl-details>
<!-- <sl-details disabled id="compareImages-difference-image-details" summary="differenceImage: Absolute difference image"></sl-details> -->
Copy link
Member

Choose a reason for hiding this comment

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

remove commented

@@ -115,7 +88,8 @@ <h3>👨‍💻 Live API Demo ✨</h3>
</sl-select>
<sl-button variant="neutral" outline name="difference-image-download" disabled>Download</sl-button>
<br /><br />
<sl-details disabled id="compareImages-difference-uchar-2d-image-details" summary="differenceUchar2dImage: Unsigned char, 2D difference image for rendering"></sl-details>
<!-- <sl-details disabled id="compareImages-difference-uchar-2d-image-details" summary="differenceUchar2dImage: Unsigned char, 2D difference image for rendering"></sl-details> -->
Copy link
Member

Choose a reason for hiding this comment

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

remove commented

@@ -0,0 +1,29 @@
{
"name": "@itk-wasm/demo-app",
"version": "0.0.0",
Copy link
Member

Choose a reason for hiding this comment

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

0.1.0

"name": "@itk-wasm/demo-app",
"version": "0.0.0",
"packageManager": "pnpm@8.11.0",
"description": "Try pipelines in a browser",
Copy link
Member

Choose a reason for hiding this comment

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

Try itk-wasm pipelines in a browser

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Addressed all the comments!

@thewtex
Copy link
Member

thewtex commented Apr 15, 2024

Please rebase for CI testing.

Reflect disable attribute in itk-image-details so cypress waits until
enabled to click.

Ask cypress to query the shadow dom.
@PaulHax
Copy link
Collaborator Author

PaulHax commented Apr 17, 2024

All checks green 💚

@thewtex thewtex merged commit 1c7df01 into InsightSoftwareConsortium:main Apr 23, 2024
73 checks passed
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