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

Compatibility with octave #55

Open
loneblues opened this issue Sep 14, 2022 · 4 comments
Open

Compatibility with octave #55

loneblues opened this issue Sep 14, 2022 · 4 comments

Comments

@loneblues
Copy link

Hello. I am trying to use dicm2nii on octave, and there appears to be some work done for octave compatibility in the scripts.
However, when I try to run octave on ver 4.2.2 which is installed with apt install on Ubuntu 18, there appears to be some erros. Is this a specific issue of mine, or is it due to the fact that dicm2nii does not support beyon 3.8.2?
Thanks.

@xiangruili
Copy link
Owner

The command line is expected to work under Octave, but it is not thoroughly tested. Anything involving GUI or figure likely won't work under Octave.

Could you provide specific error, so we can try to fix the compatibility if it is an easy fix?

@loneblues
Copy link
Author

Xiangrui Li's dicm2nii.m 20220526 (feedback to [email protected])
Validating 151 files ... (150 valid)
warning: division by zero
warning: called from
dicm2nii at line 437 column 13
error: sortrows: only cell arrays of character strings may be sorted
error: called from
sortrows at line 77 column 7
dicm2nii>checkImagePosition at line 2618 column 8
dicm2nii at line 483 column 14

Of course.

Above is the error message that I ran into.
BTW I installed my Ubuntu on WSL2 environment, on my Windows 10 PC.
I called the function dicm2nii(source dir, output dir, 5) in the octave terminal. The DICOM files are within the source dir, not located in any subdirectories, if this matters.
I also wonder if this is the proper way to use this, but I presume it is because it was successful using the MATLAB command line.

Hope this helps.

@xiangruili
Copy link
Owner

It sounds like the compatibility issue of function sortrows. Did you mean it worked for older Octave?
If you don't mind, you can share the series of the dicom files with me, so I can dig in to the potential cause.

@loneblues
Copy link
Author

I have sent you a e-mail with the anonymized DICOM files. Please let me know if you haven't recieved them. Thanks.

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