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

HDF4 specification of Image dimensions #359

Closed
JohnLCaron opened this issue May 19, 2023 · 3 comments
Closed

HDF4 specification of Image dimensions #359

JohnLCaron opened this issue May 19, 2023 · 3 comments
Assignees
Labels
Component - Documentation Doxygen, markdown, etc. Priority - 2. Medium ⏹ It would be nice to have this in the next release Type - Bug Please report security issues to [email protected] instead of creating an issue on GitHub

Comments

@JohnLCaron
Copy link

JohnLCaron commented May 19, 2023

https://support.hdfgroup.org/release4/doc/DS.pdf

p 9-125

DFTAG_LUT
Lookup table
xdim * ydim * elements * NTsize bytes (xdim, ydim, elements,
and NTsize are specified in the corresponding DFTAG_ID)
301 (0x012D)

Should probably say "... are specified in the corresponding DFTAG_LD". It would make no sense to have the lookup table be the same dimensions as the image.

I have at least one example file that has an LUT without LD, so all I can do is ignore. There is also an IP8 tag so I can use that.

p 9-124

DFTAG_RI
Raster image
xdim * ydim * elements * NTsize bytes (xdim, ydim, elements,
and NTsize are specified in the corresponding DFTAG_ID)
302 (0x012E)

This one is more speculative on my part, but it doesnt make sense for elements to be anything other than 1, so one could just leave that out.

@hyoklee
Copy link
Member

hyoklee commented May 23, 2023

@bmribler Please don't forget to convert FrameMaker documents to MS Word documents.

@glennsong09 glennsong09 added Component - Documentation Doxygen, markdown, etc. Type - Bug Please report security issues to [email protected] instead of creating an issue on GitHub Priority - 2. Medium ⏹ It would be nice to have this in the next release labels May 23, 2023
@derobins derobins added this to the 4.3.0 milestone Aug 30, 2023
@bmribler
Copy link
Collaborator

bmribler commented Oct 2, 2023

@JohnLCaron

  1. You are correct, DFTAG_ID should be DFTAG_LD for LUT, I'll make this correction.
  2. My understanding is "elements" is 1 for B/W (gray scale) but for color images, it can be a different number, such as 3 for RGB color scheme.

@bmribler bmribler moved this to Todo in HDF4 4.3.0 Oct 27, 2023
@bmribler bmribler moved this from Todo to In Progress in HDF4 4.3.0 Nov 2, 2023
@bmribler bmribler closed this as completed Nov 8, 2023
@github-project-automation github-project-automation bot moved this from In Progress to Done in HDF4 4.3.0 Nov 8, 2023
@bmribler
Copy link
Collaborator

bmribler commented Nov 8, 2023

Please add new comments to HDFGroup/hdf4doc#8

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Component - Documentation Doxygen, markdown, etc. Priority - 2. Medium ⏹ It would be nice to have this in the next release Type - Bug Please report security issues to [email protected] instead of creating an issue on GitHub
Projects
Status: Done
Development

No branches or pull requests

5 participants