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

Backup missing sessions and sessions with shorter JSON sidecars to S3 #17

Open
rbreslow opened this issue Apr 12, 2022 · 1 comment
Open
Assignees

Comments

@rbreslow
Copy link
Contributor

rbreslow commented Apr 12, 2022

The following cases result in files failing to be processed. These need to be accounted for in order for a user to review manually and re-process as needed.

For these studies, DICOMs are not converted to NIfTIs and the entire study fails. These are currently being tracked in a table that is output to a CSV in program/site/files/:

  • Missing sessions
  • Missing c-ids
  • Missing target Flywheel project

NOTE: Need to avoid overwriting output CSV files on s3

For these acquisitions, the NIfTIs+JSONs get separated into the following directories (the remaining acquisitions in the study are processed completely):

  • NIfTIs_to_check
  • NIfTIs_short_json
@afamiliar
Copy link
Collaborator

afamiliar commented Apr 21, 2022

Note to self: if missing sub/ses label, currently fails at fw ingest (structure_nifti_files outputs to command line NIfTI files created but no subject mapping to use.) - consider exiting earlier

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

No branches or pull requests

2 participants