Fixed add_v9layout saving version if it was processed as double

Arthur Glowacki · Arthur Glowacki · commit c240711bb811 · 2025-08-25T08:54:11.000-05:00
diff --git a/src/io/file/hdf5_io.cpp b/src/io/file/hdf5_io.cpp
@@ -2368,7 +2368,6 @@ void HDF5_IO::add_v9_layout(std::string dataset_file)
     }
 
     //change version to 9
-    float version = 9;
     hid_t version_id = H5Dopen(file_id, "/MAPS/version", H5P_DEFAULT);
     if(version_id > -1)
     {
@@ -2384,7 +2383,16 @@ void HDF5_IO::add_v9_layout(std::string dataset_file)
     {
         _global_close_map.push({ver_type, H5O_DATATYPE });
     }
-    H5Dwrite(version_id, ver_type, ver_space, ver_space, H5P_DEFAULT, (void*)&version);
+    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 (H5Gget_objinfo(file_id, "/version", 0, NULL) < 0)
     {
         H5Lcreate_hard(file_id, "/MAPS/version", H5L_SAME_LOC, "/version", H5P_DEFAULT, H5P_DEFAULT);

Original file line number	Diff line number	Diff line change
`@@ -2368,7 +2368,6 @@ void HDF5_IO::add_v9_layout(std::string dataset_file)`
`2368`	`2368`	`}`
`2369`	`2369`
`2370`	`2370`	`//change version to 9`
`2371`		`- float version = 9;`
`2372`	`2371`	`hid_t version_id = H5Dopen(file_id, "/MAPS/version", H5P_DEFAULT);`
`2373`	`2372`	`if(version_id > -1)`
`2374`	`2373`	`{`
`@@ -2384,7 +2383,16 @@ void HDF5_IO::add_v9_layout(std::string dataset_file)`
`2384`	`2383`	`{`
`2385`	`2384`	`_global_close_map.push({ver_type, H5O_DATATYPE });`
`2386`	`2385`	`}`
`2387`		`- H5Dwrite(version_id, ver_type, ver_space, ver_space, H5P_DEFAULT, (void*)&version);`
	`2386`	`+ if (H5Tget_class(ver_type) == H5T_FLOAT)`
	`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);`
	`2395`	`+ }`
`2388`	`2396`	`if (H5Gget_objinfo(file_id, "/version", 0, NULL) < 0)`
`2389`	`2397`	`{`
`2390`	`2398`	`H5Lcreate_hard(file_id, "/MAPS/version", H5L_SAME_LOC, "/version", H5P_DEFAULT, H5P_DEFAULT);`