Merge pull request #2016 from sideeffects/sendupstream_nanovdbtranspose

jmlait · web-flow · commit c97acb439dc9 · 2025-03-26T14:19:24.000-04:00
Transpose on NanoToOpenVDB allowing round tripping
diff --git a/nanovdb/nanovdb/tools/NanoToOpenVDB.h b/nanovdb/nanovdb/tools/NanoToOpenVDB.h
@@ -160,7 +160,7 @@ NanoToOpenVDB<NanoBuildT>::operator()(const NanoGrid<NanoBuildT>& grid, int /*ve
     const nanovdb::Map& nanoMap = reinterpret_cast<const GridData*>(srcGrid)->mMap;
     auto                mat = openvdb::math::Mat4<double>::identity();
     mat.setMat3(openvdb::math::Mat3<double>(nanoMap.mMatD));
-    mat.transpose(); // the 3x3 in nanovdb is transposed relative to openvdb's 3x3
+    mat = mat.transpose(); // the 3x3 in nanovdb is transposed relative to openvdb's 3x3
     mat.setTranslation(openvdb::math::Vec3<double>(nanoMap.mVecD));
     dstGrid->setTransform(openvdb::math::Transform::createLinearTransform(mat)); // calls simplify!
 
diff --git a/pendingchanges/nanovdb_transpose.txt b/pendingchanges/nanovdb_transpose.txt
@@ -0,0 +1,3 @@
+NanoVDB:
+  Bug fix:
+    Map is now properly transposed when converting NanoVDB to OpenVDB.

Original file line number	Diff line number	Diff line change
`@@ -0,0 +1,3 @@`
	`1`	`+NanoVDB:`
	`2`	`+ Bug fix:`
	`3`	`+ Map is now properly transposed when converting NanoVDB to OpenVDB.`