search

dcmjs-org / dcmjs

Javascript implementation of DICOM manipulation
https://dcmjs.netlify.com/
MIT License
291 stars 110 forks source link

Removed comment around getTID300RepresentationArguments 'tool' parameter #322

Closed SveinungOverland closed 1 year ago

SveinungOverland commented 1 year ago

Fix for issue #306. Missing tests but since dcmjs seems to be missing a testing setup for adapters I think someone more familiar with js testing and dcmjs should set the standard.

pieper commented 1 year ago

Thanks for PR. I agree more testing should happen, but realistically testing this requires cornerstone so for this reason and others it would make sense for these adapters to migrate out of the dcmjs core someday.

github-actions[bot] commented 1 year ago

:tada: This PR is included in version 0.28.2 :tada:

The release is available on:

Your semantic-release bot :package::rocket: