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

No DICOMs found #54

Open
araikes opened this issue Oct 19, 2023 · 4 comments
Open

No DICOMs found #54

araikes opened this issue Oct 19, 2023 · 4 comments

Comments

@araikes
Copy link

araikes commented Oct 19, 2023

  • mrQA version: pip current version
  • Python version: 3.10
  • Operating System: Centos 7

Description

Siemens enhanced DICOMs without dcm extension doesn't find DICOMS

What I Did

(qctools) [adamraikes@gpu68 allo_phase2]$ mrqa --data-source $PWD/dicoms/000052/Screening/666e4ef2/ --format dicom --name test     
Traceback (most recent call last):
  File "/home/u26/adamraikes/.conda/envs/qctools/bin/mrqa", line 8, in <module>
    sys.exit(main())
  File "/home/u26/adamraikes/.conda/envs/qctools/lib/python3.10/site-packages/mrQA/cli.py", line 85, in main
    check_compliance(dataset=dataset,
  File "/home/u26/adamraikes/.conda/envs/qctools/lib/python3.10/site-packages/mrQA/project.py", line 63, in check_compliance
    raise DatasetEmptyException
MRdataset.config.DatasetEmptyException: Expected Sidecar DICOM/JSON files in --data_source. Got 0 DICOM/JSON files.

@sinhaharsh
Copy link
Collaborator

@araikes This should not happen if the folder contains dicom files. mrqa doesn't check for extension to filter dicom files.

It skips localizer/phantom/head scouts in the folder. The function used for selecting dicom files can be seen here

@raamana
Copy link
Contributor

raamana commented Oct 19, 2023

Harsh, it might also be due to them being Enhanced DICOM format. We need to start testing MRdataset and mrQA on that format. Let’s try to find a dataset in that format.

@sinhaharsh
Copy link
Collaborator

@araikes Would it be possible to share just a single slice of the Enhanced DICOM format? So that we can debug this issue?

@raamana
Copy link
Contributor

raamana commented Oct 19, 2023

Regardless of the format, the output error message was inaccurate - we should improve it to be more direct, and issue an warning for each folder ignored / not included

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

3 participants