Fixed add_v9layout saving version if it was processed as double

Arthur Glowacki · Arthur Glowacki · commit 8244b81fdc8f · 2025-08-25T08:57:36.000-05:00
diff --git a/src/io/file/hdf5_io.cpp b/src/io/file/hdf5_io.cpp
@@ -2385,13 +2385,16 @@ void HDF5_IO::add_v9_layout(std::string dataset_file)
     }
     if (H5Tget_class(ver_type) == H5T_FLOAT) 
     {
-        float version = 9.0;
-        H5Dwrite(version_id, ver_type, ver_space, ver_space, H5P_DEFAULT, (void*)&version);
-    }
-    else
-    {
-        double version = 9.0;
-        H5Dwrite(version_id, ver_type, ver_space, ver_space, H5P_DEFAULT, (void*)&version);
+        if (H5Tget_size(ver_type) == sizeof(float)) 
+        {
+            float version = 9.0;
+            H5Dwrite(version_id, ver_type, ver_space, ver_space, H5P_DEFAULT, (void*)&version);
+        }
+        else
+        {
+            double version = 9.0;
+            H5Dwrite(version_id, ver_type, ver_space, ver_space, H5P_DEFAULT, (void*)&version);
+        }
     }
     if (H5Gget_objinfo(file_id, "/version", 0, NULL) < 0)
     {

Original file line number	Diff line number	Diff line change
`@@ -2385,13 +2385,16 @@ void HDF5_IO::add_v9_layout(std::string dataset_file)`
`2385`	`2385`	`}`
`2386`	`2386`	`if (H5Tget_class(ver_type) == H5T_FLOAT)`
`2387`	`2387`	`{`
`2388`		`- float version = 9.0;`
`2389`		`- H5Dwrite(version_id, ver_type, ver_space, ver_space, H5P_DEFAULT, (void*)&version);`
`2390`		`- }`
`2391`		`- else`
`2392`		`- {`
`2393`		`- double version = 9.0;`
`2394`		`- H5Dwrite(version_id, ver_type, ver_space, ver_space, H5P_DEFAULT, (void*)&version);`
	`2388`	`+ if (H5Tget_size(ver_type) == sizeof(float))`
	`2389`	`+ {`
	`2390`	`+ float version = 9.0;`
	`2391`	`+ H5Dwrite(version_id, ver_type, ver_space, ver_space, H5P_DEFAULT, (void*)&version);`
	`2392`	`+ }`
	`2393`	`+ else`
	`2394`	`+ {`
	`2395`	`+ double version = 9.0;`
	`2396`	`+ H5Dwrite(version_id, ver_type, ver_space, ver_space, H5P_DEFAULT, (void*)&version);`
	`2397`	`+ }`
`2395`	`2398`	`}`
`2396`	`2399`	`if (H5Gget_objinfo(file_id, "/version", 0, NULL) < 0)`
`2397`	`2400`	`{`