-
Notifications
You must be signed in to change notification settings - Fork 125
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: add support for PAR/REC files #121
Comments
Still just trying to understand how this code goes together. Does the following sound about right: @mgxd, you're suggesting to skip the loading by pydicom (and conversion into a |
@psadil yes, |
sounds good, I'll go with the --parrec argument. Just as a heads up, I won't be fast at this (end of semester season + slow python coder), but so long as this doesn't need to be finished immediately (read: next couple of weeks) I'd be happy to work on it. |
Finally have a bit of time to sit and work on this. Looks like the simplest workaround would be to create a temporary .nii file with nibabel's Also, although
Is there something else I should have installed first? |
Forget about make install - it is outdated, we or someone needs to fix it up |
Hi! Is someone still working on this? We have a large project and we are very eager for a solution for this as a partner has data in par/rec format (rest in nifti). |
Handling this ended up being fairly far above my head. Sorry to have snagged it and then silently threw up my hands! I'm not aware of anyone else working on it. |
I get that! This is far above my skill level also, was just hoping! |
Would also love to have par/rec supported in heudiconv, we have a lot of datasets in this format. Am I right that no one is actively working on this anymore? PS: For anyone looking for a par/rec to BIDS converter, we are using bidsify (https://github.com/NILAB-UvA/bidsify) now. But still would be nice to have this featured in heudiconv. |
@eduardklap I don't believe anyone is currently working on this issue. The way I see this implementation going is by adding a function that mimics Line 13 in 6b30c75
(groups scans into series, extracts series information, concatenates all series information) - this will require a bit of work. If this is something you'd like to work towards implementing, feel free to open a draft pull request and ping us whenever you get stuck! In the meantime, thank you for sharing the link. |
Good to know, thanks! I will have a look in the coming weeks and see what I can do |
Given that there were no pragmatic interest, and anyways best were possible is to acquire dicoms and heudiconv has DICOM in its name -- I will just close this issue |
original thread on neurostars.
Since Dcm2niix already supports this, we would need to add either a PAR/REC flag or try/except blocks to avoid DICOM operations
EDIT: perhaps we can use nibabel's parrec
The text was updated successfully, but these errors were encountered: