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

Updated some incompatibilities #6

Open
rmustakos opened this issue Aug 14, 2024 · 1 comment
Open

Updated some incompatibilities #6

rmustakos opened this issue Aug 14, 2024 · 1 comment

Comments

@rmustakos
Copy link

Handled the case where the filename is not passed into mha_read_volume, which caused an error in calling mha_read_header with an undeclared variable (filename)
fopen doesn't seem to need the filename to be transposed (and Octave hates it)
if fopen doesn't work with big/little endian cues, open it and hope for the best
Functions require an "end" before the next function definition
Added *.mhd to file types to open
Works the same for both Matlab and Octave

Fixes are attached

mha_read_volume.zip

@lassoan
Copy link
Contributor

lassoan commented Aug 17, 2024

Thank you for these updates. Could you please send a pull request with the suggested changes to allow a quick review and proper attribution of these updates to your account?

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