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

[ENH] Allow datman to handle linked XNAT sessions #358

Merged
merged 10 commits into from
Oct 17, 2023

Conversation

DESm1th
Copy link
Contributor

@DESm1th DESm1th commented Oct 17, 2023

  • Add a function to detect a shared XNAT experiment (4646814)
  • Add a datman.exporters.SharedExporter class that finds and symlinks relevant files (i.e. those with the correctly configured tags) from the XNAT source session (b642855, 9584af8, 85b3d01, b61671f, d45703c)
  • Update datman.exporters.DBExporter to mark shared scans with the correct source scan ID (7f8f8a7)
  • Fix datman.exporters.DBExporter bugs related to some scans missing on the first run of datman/bin/dm_xnat_extract.py when those scans first export only to bids (instead of the legacy nii folder) (09bb73f)
  • Fix a datman.scan.Scan bug causing some scans to be missed because the .niftis property did not update after object creation (b4620a1)
  • Fix a date parsing related exception that prevented DEEPPI redcap records from being added to the database (525dfdb)

DESm1th added 10 commits October 4, 2023 15:55
- The manifest files linked from the source session were causing
  incorrect paths to be generated in the dashboard, causing metrics
  to appear as broken images. This is resolved now, by omitting
  them from the symlink name_map
- Files that could not be traditionally exported directly from
  xnat (i.e. files that could only be exported with dcm2niix)
  were being missed when searching datman-style paths. This is
  fixed now, and all valid datman-style names with matching tags
  will be detected.
@DESm1th DESm1th added the feature Tagged PR addes a new feature label Oct 17, 2023
@DESm1th DESm1th requested a review from a team as a code owner October 17, 2023 23:00
@DESm1th DESm1th merged commit 0558304 into TIGRLab:main Oct 17, 2023
3 of 4 checks passed
@DESm1th DESm1th deleted the linked_sessions branch October 17, 2023 23:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature Tagged PR addes a new feature
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant