Skip to content

Commit 36b119b

Browse files
committed
Use custom StringGrid type (as StringGrids are no longer registered)
Signed-off-by: Dan Bailey <danbailey@ilm.com>
1 parent ff41eb5 commit 36b119b

File tree

1 file changed

+12
-11
lines changed

1 file changed

+12
-11
lines changed

openvdb_ax/openvdb_ax/test/compiler/TestVolumeExecutable.cc

Lines changed: 12 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -578,11 +578,13 @@ TestVolumeExecutable::testActiveTileStreaming()
578578
}
579579

580580
// test spatially varying voxelization for string grids which use specialized implementations
581+
// Note: StringGrids are no longer registered by default
581582
{
582-
OPENVDB_NO_DEPRECATION_WARNING_BEGIN
583-
openvdb::StringGrid grid;
583+
using StringTree = openvdb::tree::Tree4<std::string, 5, 4, 3>::Type;
584+
using StringGrid = openvdb::Grid<openvdb::tree::Tree4<std::string, 5, 4, 3>::Type>;
585+
StringGrid grid;
584586
grid.setName("test");
585-
openvdb::StringTree& tree = grid.tree();
587+
StringTree& tree = grid.tree();
586588
tree.addTile(2, openvdb::Coord(NodeT1::DIM*0, 0, 0), "foo", /*active*/true); // NodeT1 tile
587589
tree.addTile(2, openvdb::Coord(NodeT1::DIM*1, 0, 0), "foo", /*active*/true); // NodeT1 tile
588590
tree.addTile(2, openvdb::Coord(NodeT1::DIM*2, 0, 0), "foo", /*active*/true); // NodeT1 tile
@@ -599,14 +601,14 @@ OPENVDB_NO_DEPRECATION_WARNING_BEGIN
599601
executable->getActiveTileStreaming("empty", openvdb::ax::ast::tokens::CoreType::FLOAT));
600602
executable->getTreeExecutionLevel(min,max);
601603
CPPUNIT_ASSERT_EQUAL(openvdb::Index(0), min);
602-
CPPUNIT_ASSERT_EQUAL(openvdb::Index(openvdb::StringTree::DEPTH-1), max);
604+
CPPUNIT_ASSERT_EQUAL(openvdb::Index(StringTree::DEPTH-1), max);
603605

604606
executable->execute(grid);
605607

606608
const openvdb::Index64 face = NodeT1::DIM * NodeT1::DIM; // face voxel count of NodeT2 x==0
607609
const openvdb::Index64 leafs = // expected leaf nodes that need to be created
608-
(face * openvdb::StringTree::LeafNodeType::DIM) /
609-
openvdb::StringTree::LeafNodeType::NUM_VOXELS;
610+
(face * StringTree::LeafNodeType::DIM) /
611+
StringTree::LeafNodeType::NUM_VOXELS;
610612

611613
// number of child nodes in NodeT1;
612614
const openvdb::Index64 n1ChildAxisCount = NodeT1::DIM / NodeT1::getChildDim();
@@ -618,10 +620,10 @@ OPENVDB_NO_DEPRECATION_WARNING_BEGIN
618620

619621
CPPUNIT_ASSERT_EQUAL(openvdb::Index32(leafs), tree.leafCount());
620622
CPPUNIT_ASSERT_EQUAL(openvdb::Index64(tiles), tree.activeTileCount());
621-
CPPUNIT_ASSERT_EQUAL(int(openvdb::StringTree::DEPTH-3), tree.getValueDepth(openvdb::Coord(NodeT1::DIM*1, 0, 0)));
622-
CPPUNIT_ASSERT_EQUAL(int(openvdb::StringTree::DEPTH-3), tree.getValueDepth(openvdb::Coord(NodeT1::DIM*2, 0, 0)));
623-
CPPUNIT_ASSERT_EQUAL(int(openvdb::StringTree::DEPTH-3), tree.getValueDepth(openvdb::Coord(NodeT1::DIM*3, 0, 0)));
624-
CPPUNIT_ASSERT_EQUAL(int(openvdb::StringTree::DEPTH-2), tree.getValueDepth(openvdb::Coord(NodeT2::DIM)));
623+
CPPUNIT_ASSERT_EQUAL(int(StringTree::DEPTH-3), tree.getValueDepth(openvdb::Coord(NodeT1::DIM*1, 0, 0)));
624+
CPPUNIT_ASSERT_EQUAL(int(StringTree::DEPTH-3), tree.getValueDepth(openvdb::Coord(NodeT1::DIM*2, 0, 0)));
625+
CPPUNIT_ASSERT_EQUAL(int(StringTree::DEPTH-3), tree.getValueDepth(openvdb::Coord(NodeT1::DIM*3, 0, 0)));
626+
CPPUNIT_ASSERT_EQUAL(int(StringTree::DEPTH-2), tree.getValueDepth(openvdb::Coord(NodeT2::DIM)));
625627
CPPUNIT_ASSERT_EQUAL((openvdb::Index64(NodeT1::NUM_VOXELS)*4) +
626628
openvdb::Index64(NodeT0::NUM_VOXELS), tree.activeVoxelCount());
627629

@@ -630,7 +632,6 @@ OPENVDB_NO_DEPRECATION_WARNING_BEGIN
630632
if (coord.x() == 0) CPPUNIT_ASSERT_EQUAL(*it, std::string("bar"));
631633
else CPPUNIT_ASSERT_EQUAL(*it, std::string("foo"));
632634
});
633-
OPENVDB_NO_DEPRECATION_WARNING_END
634635
}
635636

636637
// test streaming with an OFF iterator (no streaming behaviour) and an ALL iterator (streaming behaviour for ON values only)

0 commit comments

Comments
 (0)