Skip to content

Conversation

@abhijith-trenser
Copy link
Contributor

Context

Changes & Results

  • Send additional metrics (Radius, Mean, Maximum, Std Dev, Perimeter) from tools to support metric saving in dcmjs

What are the effects of this change?

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

@abhijith-trenser abhijith-trenser marked this pull request as draft October 28, 2025 10:21
@abhijith-trenser abhijith-trenser marked this pull request as ready for review October 28, 2025 10:49
@abhijith-trenser abhijith-trenser marked this pull request as draft October 28, 2025 10:49
@abhijith-trenser abhijith-trenser marked this pull request as ready for review October 28, 2025 12:15
@sedghi
Copy link
Member

sedghi commented Oct 28, 2025

please wait until our other PR fixing scale and unit is merged before merging this

@abhijith-trenser
Copy link
Contributor Author

@sedghi Is there any update on the pending merge requests that need to be merged before this PR? Also, do we have a list of those MRs for reference? Thanks in advance

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.

3 participants