Skip to content

Add Niivue viewer#19995

Merged
guerler merged 5 commits intogalaxyproject:devfrom
guerler:niivue.000
Apr 19, 2025
Merged

Add Niivue viewer#19995
guerler merged 5 commits intogalaxyproject:devfrom
guerler:niivue.000

Conversation

@guerler
Copy link
Copy Markdown
Contributor

@guerler guerler commented Apr 11, 2025

This PR integrates Niivue, a WebGL-based library for interactive visualization of NIfTI images directly in the browser. Niivue supports 2D slice views, 3D rendering, overlays, and user interaction.

Test dataset available at http://cdn.jsdelivr.net/gh/galaxyproject/galaxy-test-data/nifti.nii1.gz

niivue

How to test the changes?

(Select all options that apply)

  • I've included appropriate automated tests.
  • This is a refactoring of components with existing test coverage.
  • Instructions for manual testing are as follows:
    1. [add testing steps and prerequisites here if you didn't write automated tests covering all your changes]

License

  • I agree to license these and all my past contributions to the core galaxy codebase under the MIT license.

@guerler guerler marked this pull request as ready for review April 11, 2025 20:04
Copy link
Copy Markdown
Member

@bgruening bgruening left a comment

Choose a reason for hiding this comment

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

Works great!

I can not open Visualizations as Anonymous user, but that might be a different issue.

<?xml version="1.0" encoding="UTF-8"?>
<!DOCTYPE visualization SYSTEM "../../visualization.dtd">
<visualization name="Niivue" embeddable="true">
<description>WebGL2 based medical image viewer</description>
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Not sure if WebGL2 should be in the main description, isn't it too technical? It could later go into the help.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Changed this to: Interactive viewer for medical and neuroimaging data

<data><label>Winter</label><value>winter</value></data>
</data>
</input>
<input>
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

the indentation is wrong here

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Fixed. Thank you!

</requirements>
<entry_point entry_point_type="script" src="dist/index.js" css="dist/index.css" />
<settings>
<input>
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

The indentation is wrong here, and becomes later a bit confusing.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Fixed. Thank you!

@guerler guerler merged commit 140e974 into galaxyproject:dev Apr 19, 2025
53 of 54 checks passed
@guerler guerler deleted the niivue.000 branch April 19, 2025 17:46
@github-actions
Copy link
Copy Markdown

This PR was merged without a "kind/" label, please correct.

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants