Skip to content

Conversation

@abhijith-trenser
Copy link

@abhijith-trenser abhijith-trenser commented Oct 28, 2025

Context

Changes & Results

  • Added Radius, Mean, Maximum, and Minimum metrics to TID300 for Circle, Ellipse, and Polyline tools.
  • Introduced a helper function buildContentSequence to eliminate repetitive graphic data usage across tools.

What are the effects of this change?

The following tools in Cornerstone3D, Length, Bidirectional, Ellipse, Circle, Rectangle, and Freehand now include all annotation metrics without requiring rendering.

This feature applies only to newly created SRs. Existing SR files will not reflect these changes unless re-saved.

Testing

  • Launch a Study using /local
  • Draw annotations in different slices using different tools.
  • Open right side panel
  • Click 'Create SR' and save the SR with a file name.
  • Launch the same study along with the downloaded SR
  • Load the SR into viewport and confirm to track the study
  • Observe the annotations and all annotaions display all metrics without rendering it.

Checklist

PR

  • My Pull Request title is descriptive, accurate and follows the
    semantic-release format and guidelines.

Code

  • My code has been well-documented (function documentation, inline comments,
    etc.)

Public Documentation Updates

  • The documentation page has been updated as necessary for any public API
    additions or removals.

Tested Environment

  • OS: Windows 11
  • Node version: 22
  • Browser: Chrome 141.0.7390.123

@netlify
Copy link

netlify bot commented Oct 28, 2025

Deploy Preview for dcmjs2 ready!

Name Link
🔨 Latest commit cc8b84b
🔍 Latest deploy log https://app.netlify.com/projects/dcmjs2/deploys/6900b3cbec40a40008fe2ecb
😎 Deploy Preview https://deploy-preview-460--dcmjs2.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 project configuration.

@abhijith-trenser abhijith-trenser marked this pull request as ready for review October 28, 2025 12:15
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.

1 participant