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

Some HDF5 variable length data are missing #96

Open
kyang2014 opened this issue Oct 13, 2015 · 2 comments
Open

Some HDF5 variable length data are missing #96

kyang2014 opened this issue Oct 13, 2015 · 2 comments
Milestone

Comments

@kyang2014
Copy link

  1. File: ftp://ftp.hdfgroup.uiuc.edu/pub/outgoing/HDFCompass/kent-files/hdf5-handler-fake/ntypes.h5

  2. Testing version: Windows version of the release candidate from S3

For a 1-D (4 element) variable length array)(vl_type), compass misses values of the index 2 and 3.

For index 2, h5dump shows the values (200),(210,211),(220,221,222)
compass only shows (200),(210,211)

For index 3, h5dump shows the values (300), (310, 311), (320, 321, 322), (330, 331, 332, 333)
compass only shows (300),,(310,311)

It looks like compass correctly picks up all the values in index 1 and then keep using the number of elements in index 1 for the rest indexes. This is wrong for a variable length array.

@kyang2014 kyang2014 added the bug label Oct 13, 2015
@jreadey
Copy link
Member

jreadey commented Oct 13, 2015

I can reproduce.
@kyang2014 - can you create a smaller test file that just contains the vl_type dataset?

@kyang2014
Copy link
Author

At:
ftp://ftp.hdfgroup.uiuc.edu/pub/outgoing/HDFCompass/kent-files/hdf5-testing-files/vl_vl.h5

Please note that the basetype of this variable length type is a variable length type:
The snippet of h5dump:
DATATYPE H5T_VLEN { H5T_VLEN { H5T_STD_U32LE}}

If you try a simple variable length datatype file, ftp://ftp.hdfgroup.uiuc.edu/pub/outgoing/HDFCompass/kent-files/hdf5-testing-files/tvldtypes1.h5
HDF5 compass works fine.

However, I do think this issue needs to be addressed some way since as a user I am OK if the values are not displayed at all. It is bad if I read data that makes feel complete but actually only partial data values present.

Kent

From: John Readey [mailto:[email protected]]
Sent: Tuesday, October 13, 2015 12:41 PM
To: HDFGroup/hdf-compass
Cc: Kent Yang
Subject: Re: [hdf-compass] Some HDF5 variable length data are missing (#96)

I can reproduce.

@kyang2014https://github.com/kyang2014 - can you create a smaller test file that just contains the vl_type dataset?


Reply to this email directly or view it on GitHubhttps://github.com//issues/96#issuecomment-147788578.

@giumas giumas added the HDF5 label Oct 17, 2015
@giumas giumas added this to the 0.6.x milestone Oct 27, 2015
@giumas giumas modified the milestones: 0.7.0, 0.6.x Jun 16, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants