Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

saveAs gives misleading exceptions #6

Open
sbenthall opened this issue Feb 27, 2013 · 2 comments
Open

saveAs gives misleading exceptions #6

sbenthall opened this issue Feb 27, 2013 · 2 comments

Comments

@sbenthall
Copy link

using saveAs, I've gotten an HDF5LibraryException even when load works fine.

I made two changes and it fixed the problem:

  • adding a .mat to the file name
  • creating the directory that I was telling it to save into so that it gets a valid path

There's nothing blocking, but wanted to post an issue because there may be a more helpful way to do error reporting here. Since the HDF5LibraryException seems to be related to a wide variety of possible problems (including not having LD_LIBRARY_PATH set properly), it would be nice for usability to catch it and report a more specific error when possible.

@eternia478
Copy link

Hm, well, it might be even better to just have it create the folder (well, with an option, i.e. -f or something). The fact that it isn't named as *.mat though, should have a check that automatically adds it to the end.

@jcanny
Copy link
Contributor

jcanny commented Aug 5, 2014

I agree with all of this, but the problem is that HDF5 I/O is implemented using an external C library and java wrapper. We inherit the error reporting from there. Its quite difficult to fix externally, although for some things (e.g. missing files or directories) we could add an external check.

I'll leave the issue open, and given enough time, or enough complaints, we'll try to fix it later.

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants