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

[STORY] Move JSON edits functionality to converter from Bids store #81

Closed
tclose opened this issue May 27, 2022 · 2 comments
Closed

[STORY] Move JSON edits functionality to converter from Bids store #81

tclose opened this issue May 27, 2022 · 2 comments
Assignees
Labels
deploy-framework Deployment framework epic mid-level Requires moderate framework/domain-specific knowledge pipelines story a unit of work
Milestone

Comments

@tclose
Copy link
Contributor

tclose commented May 27, 2022

Description

Post-hoc editing of JSON side cars of NIFTIX (see #92) should be handled Dicom->Nifti converter instead of part of the Bids store, because it will be needed for non-BIDS apps too.

Acceptance Criteria

When DICOM files are being converted to NIFTI format, editable arguments should be able to be passed to the converter to enable post-hoc editing of the generated JSON side cars

Notes

The edits could be stored in XNAT (potentially in a different resource of the scans to be converted ARCANA__CONVERTER_ARGS) and editable by an arcana CLI

@tclose tclose added this to the 2.0.0b milestone May 27, 2022
@tclose
Copy link
Contributor Author

tclose commented Jun 19, 2022

@tclose tclose moved this to Backlog in AIS Master Project Jun 20, 2022
@tclose tclose changed the title Move JSON edits functionality to converter from Bids store [STORY] Move JSON edits functionality to converter from Bids store Jun 20, 2022
@tclose tclose added story a unit of work mid-level Requires moderate framework/domain-specific knowledge deploy-framework Deployment framework epic ready and removed enhancement labels Jun 21, 2022
@tclose tclose moved this to Done in AIS Master Project Aug 4, 2022
@tclose
Copy link
Contributor Author

tclose commented Aug 4, 2022

This has ben included as part of the fMRIPrep field maps changes

@tclose tclose closed this as completed Aug 4, 2022
@tclose tclose added the 3pt label Aug 4, 2022
@tclose tclose reopened this Aug 4, 2022
@tclose tclose moved this from Done to In Progress in AIS Master Project Aug 4, 2022
@tclose tclose moved this from In Progress to Done in AIS Master Project Aug 4, 2022
@tclose tclose closed this as completed Aug 4, 2022
@tclose tclose self-assigned this Aug 4, 2022
@tclose tclose reopened this Aug 8, 2022
@tclose tclose closed this as completed Aug 9, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
deploy-framework Deployment framework epic mid-level Requires moderate framework/domain-specific knowledge pipelines story a unit of work
Projects
None yet
Development

No branches or pull requests

1 participant