Hdf5.WriteCompounds and attributes

[flappah]

I see the method WriteCompounds accepts a dictionary of attributes. I was hoping this would give me the option to add an attribute to the array that is being written. However nothing is being done with the attribute I supply. When I check the source I see there's no handling of the attributes supplied. Am I correct in this? And if so, any chance this feature gets added?

[LiorBanai]

Hi @flappah
do you want to add attribute to existing item or during creation of the item?
Can you add sample code?
It can be added relative simple (if this is not a bug which may be).

[flappah]

I'd like to have the attribute added to the dataset that is created at that moment. In this case you can see in the sample I've added that I'm creating a group of waterlevel feature descriptions. The IHO S104 standard dictates it needs an attribute at that specific level with the name 'chunking'.

For me this code sample is getting to know (a) the standards itself and (b) your HDF5 library. To get a feel for it. This is the line of code where the attributes are needed:

Dictionary<string, List<string>> attributes = new Dictionary<string, List<string>>();
attributes.Add("chunking", new List<string>() { "0,0" });
Hdf5.WriteCompounds(groupFId, "WaterLevel", waterLevelFeatures, attributes);

HDF5TestCreator.cs.zip

[LiorBanai]

no problem. I'll check and will add the missing code.

[flappah]

Thanks Lior. Really appreciate it!

[LiorBanai]

Ok I see the attribute code is missing. working on it.

[LiorBanai]

@flappah I have created and issue for this discussion #248

[LiorBanai]

Hi @flappah
is this output what you need (using the above code)? If so I'll publish new version..
TestAttributesCreation.zip

[flappah]

Hi LiorBanai!! Yep, that's exactly what I'm looking for. Looks great! Thanks!!!

[LiorBanai]

glad to know. I'll release new nuget tomorrow.

[LiorBanai]

Great. I'll release new version later today (need to go for some errands)

[flappah]

Great. I'll release new version later today (need to go for some errands)

Thanks man! No worries. I'm in no particular hurry since we're currently only testing. I really appreciate your swift efforts!

[LiorBanai]

I have release new version (1.16.2) . Give it a try.

[flappah]

I have release new version (1.16.2) . Give it a try.

Thanks LiorBanai! I will. Thanks for your support!

[flappah]

I can confirm it works like a charm!! Thanks man!