Skip to content
This repository has been archived by the owner. It is now read-only.

DICOM viewer: zoom, pan, and navigate through slices #240

Closed
Serhiy-Shekhovtsov opened this issue Nov 20, 2017 · 3 comments
Closed

DICOM viewer: zoom, pan, and navigate through slices #240

Serhiy-Shekhovtsov opened this issue Nov 20, 2017 · 3 comments

Comments

@Serhiy-Shekhovtsov
Copy link
Contributor

@Serhiy-Shekhovtsov Serhiy-Shekhovtsov commented Nov 20, 2017

According to documentation:

Allow the user to zoom, pan, and navigate through slices in the image viewer.

During these actions the following model properties should be updated: zoomRate,
offsetX, offsetY. These properties has been introduced in this commit. Hopefully PR will be merged soon.

These properties should taken into by "Detect and Select" and "Segmentation" components (#226 and #150)

Current Behavior

User can't navigate through slices, nor zoom+pan.

Checklist before submitting

  • I have searched through the other currently open issues and am confident this is not a duplicate of an existing bug
  • I provided a minimal code snippet or list of steps that reproduces the bug.
  • I provided screenshots where appropriate
  • I filled out all the relevant sections of this template
@Serhiy-Shekhovtsov Serhiy-Shekhovtsov changed the title DICOM viewer: zoom, pan, and navigate through slices in the image viewer DICOM viewer: zoom, pan, and navigate through slices Nov 20, 2017
@Serhiy-Shekhovtsov
Copy link
Contributor Author

@Serhiy-Shekhovtsov Serhiy-Shekhovtsov commented Dec 26, 2017

Looks like the new cornerstone version have support for zoom and pan. Will check how can we use it with our components for nodule selection and segmentation.

@Serhiy-Shekhovtsov
Copy link
Contributor Author

@Serhiy-Shekhovtsov Serhiy-Shekhovtsov commented Dec 27, 2017

Activated window width and window center adjustment, zooming and panning of cornerstone tools component. Updated the NoduleMarker component to rescale and move the marker accordingly.
zoom, pan
Will push as soon as PR #269 is merged.

@Serhiy-Shekhovtsov
Copy link
Contributor Author

@Serhiy-Shekhovtsov Serhiy-Shekhovtsov commented Jan 1, 2018

The feature has been implemented.

@reubano reubano closed this Jan 3, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
2 participants