Error reading .h5 file about scalismo

[marcelluethi]

Hi Ghazi,

Thanks for reporting this. What exactly do you mean by legacy .h5 file? Is a very old one that was still produced by statismo or another software? Does it work for other h5 files that you are workign with? I am asking because I am trying to figure out how severe this bug is.

The reason that we have Array[String] here is that this was required when using the other implementation of h5. As this case does not seem to be covered by our tests, I might have missed it.

[Ghazi-Bouabene]

Hi Marcel,

Thanks for the quick reply. The .h5 file was built by a customer of Shapemeans, most likely with the Statismo CLI. The issue does not seem to happen with .h5 files we generated with Scalismo.

Given that changing an already computed model is a least favorable option, should we consider dropping the cast to Array if it is no longer required by jhdf?

[marcelluethi]

Good to know that it only happens in legacy files.

Yes, we should fix this and also the subsequent method, where we have the same bug for attributes of type int.
Once this is fixed, I can release a version 0.91.2. Do you want to do a PR?

[Ghazi-Bouabene]

sounds good. Will do a PR soon then.