Add WriteOption to override missing transfer syntax in dataset - Githubissues

suyashkumar / dicom

⚡High Performance DICOM Medical Image Parser in Go.

MIT License

943 stars 137 forks source link

Add WriteOption to override missing transfer syntax in dataset #332

Closed suyashkumar closed 3 months ago

suyashkumar commented 3 months ago

This fixes #328. This change will also help set the stage for roundtrip tests for transfer syntax uid inference in the case of missing transfer syntax UIDs (part of #327).

This also makes an API breaking change to return an error from NewWriter, which imo makes sense.

For the future:

should we give this option as something to always override whatever transfer syntax is present (or if not present, use the override as well)?