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

Add DeidentificationMethod and DeidentificationMethodCodeSequence to json #812

Closed
wants to merge 1 commit into from
Closed

Add DeidentificationMethod and DeidentificationMethodCodeSequence to json #812

wants to merge 1 commit into from

Conversation

CGSchwarzMayo
Copy link
Contributor

This parallels my PR to bids-specification:
bids-standard/bids-specification#1772
bids-standard/bids-specification#1709

The goal is to add DeidentificationMethod and DeidentificationMethodCodeSequence to json, so that users downstream have a chance to be informed of whether some tags were scrubbed and/or if any de-facing or other image scrubbing was applied.

From our email discussion, I understand that you will probably not merge this until bids-specification merges theirs. I thought we could iterate on this, if needed, in the meantime.

Thanks for your consideration,
Chris Schwarz
Mayo Clinic

@neurolabusc
Copy link
Collaborator

The master branch is always the current stable release, and only accepts PRs from the development branch. You must make a PR to the development branch.

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.

2 participants