Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Allow attributes to be set on niftitodicom command line #174

Open
dgobbi opened this issue Jun 19, 2019 · 0 comments
Open

Allow attributes to be set on niftitodicom command line #174

dgobbi opened this issue Jun 19, 2019 · 0 comments

Comments

@dgobbi
Copy link
Owner

dgobbi commented Jun 19, 2019

Many of the dicom tools (dicomtocsv, dicomfind, etc) allow attributes to be set via "-k" options:

-k StudyDate=20120706

For those other tools, this is used to specify a query. For niftitodicom, could it be used to specify attributes for the output data set? Questions:

  • Will these override attributes from the "source" dicom files? Obviously, yes.
  • Will these override attributes set by the generator, such as PixelSpacing or SOPInstanceUID? Maybe best not to.
  • What about setting values that will vary, e.g. be different for each slice? Defer.

This idea could be taken further by creating a grammar for writing "rules" for filtering the metadata, e.g. like the RSNA-CTP deidentifier. Then we could filter the source metadata in order to produce output metadata, instead of simply replacing values.

For now, though, simple replacement is enough.

Edit: Instead of using "-k", it might be best to use "-a" to specify attributes that are to be added, and perhaps "-e" to specify attributes that are to be deleted.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant