Pablo Velasco

Results 21 comments of Pablo Velasco

@yarikoptic , I can take a look. Where is the dataset? So the issue is that the acquisition time for run 009 is closer to run 029 than to run...

I would need to find out, for runs 005, 009 and 029, the full `sform` (including the 4th column) and the acquisition times. The `'ImagingVolume'` criterium checks not only the...

Hi @Michael-Sun, Inspecting the json files you provided: ``` cd ses-03 for key in '"ImageOrientationPatient"' '"ImagePositionPatient"'; do grep "$key" fmap/*_dir-pa*.json func/*.json echo "-------------" done ``` results in: ``` fmap/sub-SID001651_ses-03_acq-mb8_dir-pa_epi.json: "ImageOrientationPatient":...

It looks like the [`AcquisitionDate` is optional](https://dicom.innolitics.com/ciods/cr-image/general-image/00080022), while [`StudyDate` is mandatory](https://dicom.innolitics.com/ciods/cr-image/general-study/00080020). `AcquisitionDate` is read here: https://github.com/nipy/heudiconv/blob/0e0f9e33a3ad4dc9bdce33ff7da12ed660e9abd9/heudiconv/dicoms.py#L91 @yarikoptic , Should we modify that line to read: ``` date=dcminfo.get('AcquisitionDate') or dcminfo.get('StudyDate'), ```...

Using @tjhendrickson's data, the only place where the code failed was not where I mentioned above, but in https://github.com/nipy/heudiconv/blob/0e0f9e33a3ad4dc9bdce33ff7da12ed660e9abd9/heudiconv/dicoms.py#L319 (in `get_dicom_series_time`). As far as I can tell, this function is...

Hi @fareshte The documentation on how to have the `"IntendedFor"` field populated is [here](https://github.com/nipy/heudiconv/blob/e747f0595dd393384d65219f5137d42b2a5c8e73/docs/heuristics.rst#populate_intended_for_opts) (I'm afraid that this didn't get propagated to the documentation in [readthedocs.io](https://heudiconv.readthedocs.io/)) Please try that and...

I think you are missing an asterisk (`*`) in front of `.dcm`. Try running: ``` docker run --rm -it -v ~/Volumes/toshiba/DICOMS/2:/data:ro -v ~/Volumes/toshiba/BIDS:/output nipy/heudiconv:latest -d /data/{subject}/{session}/*.dcm -s 2002 --ses 1...

Can you please try to run the following? ``` docker run --rm -it -v ~/Volumes/toshiba/DICOMS/2:/data:ro -v ~/Volumes/toshiba/BIDS:/output --entrypoint /bin/bash nipy/heudiconv:latest ``` That will give you a prompt inside the docker...

Did you enter the `docker run` command all in one line? The `--entrypoint /bin/bash` argument is for the `docker run` to interpret, not for `heudiconv`. I don't understand how the...

So it looks like you cannot access those files from within the docker container. I don't know about your specific setup, but in my case it happened when I tried...