search

jamesmudd / jhdf

A pure Java HDF5 library
http://jhdf.io
MIT License
137 stars 37 forks source link

Add support for Time datatype #523

Open calum-maclean-ul opened 9 months ago

calum-maclean-ul commented 9 months ago

In DataType.readDataType(), for case 2 (Time), UnsupportedHdfException is currently thrown.

It would be useful if support could be added for Time, so that files which contain them can be loaded.

jamesmudd commented 9 months ago

Thanks for looking at jHDF and finding this issue. Your right that Time is currently not supported. I would be happy to look at how difficult this would be.

Do you have a sample file you would like to be able to read that I can test with?

calum-maclean-ul commented 9 months ago

Sample.zip

Thanks for the quick response. jhdf is working great for me, apart from these issues with my file content.

Note that the sample file contains both the Time datatype and the version = 0 issue. I've added temporary workarounds to my local copy of the repo: to use OpaqueDataType for Time; and to comment out the version check.

ZhaoMJ commented 8 months ago

I've faced the same issue, both with the Time datatype and DataType.version = 0.