jhdf icon indicating copy to clipboard operation
jhdf copied to clipboard

Published 20 hours ago • verified publisher icon jamesmudd

Add support for Time datatype

Open calum-maclean-ul opened this issue 1 year ago • 3 comments

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.

calum-maclean-ul avatar Jan 04 '24 10:01 calum-maclean-ul

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?

jamesmudd avatar Jan 04 '24 11:01 jamesmudd

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.

calum-maclean-ul avatar Jan 04 '24 11:01 calum-maclean-ul

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

ZhaoMJ avatar Jan 15 '24 04:01 ZhaoMJ