-
Notifications
You must be signed in to change notification settings - Fork 278
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
fix: Navigation issues and naming issues for video/wsi #1386
Conversation
✅ Deploy Preview for cornerstone-3d-docs ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
@@ -175,5 +175,6 @@ | |||
"not dead", | |||
"not ie < 11", | |||
"not op_mini all" | |||
] | |||
], | |||
"packageManager": "yarn@1.22.22+sha512.a6b2f7906b721bba3d67d4aff083df04dad64c399707841b7acf00f6b133b7ac24255f2652fa22ae3534329dc6180534e98d17432037ff6fd140556e2bb3137e" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yarn now enforces updating this. Sigh.
@@ -224,7 +224,7 @@ class VideoViewport extends Viewport implements IVideoViewport { | |||
public setDataIds(imageIds: string[], options?: ImageSetOptions) { | |||
this.setVideo( | |||
imageIds[0], | |||
(options?.viewReference?.sliceIndex as number) || 1 | |||
((options?.viewReference?.sliceIndex as number) || 0) + 1 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Wrong frame value as it is a frame#, not an index.
* class inheritance values and member functions/proxy instances, but runs | ||
* the risk that the raw object can be modified through side affects. | ||
*/ | ||
addRaw: (imageId: string, payload: { metadata: any; type: string }) => { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
better
Context
There was a request to change the generic metadata provider as to how to stores metadata for object based metadata. Also, changed some navigation setup so that WSI and Video viewports navigate correctly.
Changes & Results
Testing
Checklist
PR
semantic-release format and guidelines.
Code
[] My code has been well-documented (function documentation, inline comments,
etc.)
[] I have run the
yarn build:update-api
to update the API documentation, and havecommitted the changes to this PR. (Read more here https://www.cornerstonejs.org/docs/contribute/update-api)
Public Documentation Updates
additions or removals.
Tested Environment