OHIF / Viewers

OHIF zero-footprint DICOM viewer and oncology specific Lesion Tracker, plus shared extension packages
https://docs.ohif.org/
MIT License
3.37k stars 3.39k forks source link

fix(multiframe): metadata handling of NM studies and loading order #4540

Open sedghi opened 11 hours ago

sedghi commented 11 hours ago

Fixes https://github.com/OHIF/Viewers/issues/4505 Fixes https://github.com/OHIF/Viewers/issues/4519 Fixes https://github.com/OHIF/Viewers/issues/4485 Fixes https://github.com/OHIF/Viewers/issues/4531 Fixes https://github.com/OHIF/Viewers/issues/4352 fixes potentially https://github.com/OHIF/Viewers/issues/4529

This pull request includes various changes across multiple files to improve functionality and fix issues in the project. The most important changes include fixes to image handling, improvements to metadata management, and updates to UI components.

Image Handling Improvements:

Metadata Management Enhancements:

UI Component Updates:

Miscellaneous Fixes:

These changes collectively enhance the functionality, reliability, and maintainability of the codebase.

netlify[bot] commented 11 hours ago

Deploy Preview for ohif-prod ready!

Name Link
Latest commit
Latest deploy log https://app.netlify.com/sites/ohif-prod/deploys/673fa719e900978c30b1a185
Deploy Preview https://deploy-preview-4540--ohif-prod.netlify.app
Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

cypress[bot] commented 10 hours ago

Viewers    Run #4444

Run Properties:  status check passed Passed #4444  •  git commit f310ef3921: fix
Project Viewers
Branch Review fix/3p9-recovery-1
Run status status check passed Passed #4444
Run duration 02m 59s
Commit git commit f310ef3921: fix
Committer sedghi
View all properties for this run β†—οΈŽ

Test results
Tests that failed  Failures 0
Tests that were flaky  Flaky 0
Tests that did not run due to a developer annotating a test with .skip  Pending 2
Tests that did not run due to a failure in a mocha hook  Skipped 0
Tests that passed  Passing 44
View all changes introduced in this branch β†—οΈŽ