Merge pull request #5923 from danieldresser-ie/levelSetFix

MeshToLevelSet Rendering Fixes

Changes.md

@@ -25,6 +25,7 @@ Fixes
 - OpenColorIO : Fixed the display transform used to show colours in popups.
 - SceneInspector : Fixed "Show History" menu items.
 - ImageGadget : Fixed loading of non-8-bit images. Among other things, this fixes the display of 16 bit node icons in the GraphEditor.
+- Arnold : Fixed rendering of VDB volumes without `file_mem_bytes` metadata.
 
 API
 ---

src/IECoreArnold/VDBAlgo.cpp

@@ -87,32 +87,18 @@ struct UCharVectorDataSink
 
 UCharVectorDataPtr createMemoryBuffer(const IECoreVDB::VDBObject* vdbObject)
 {
-	// estimate the size of the memory required to hand the VDB to arnold.
-	// This is required so we can reserve the right amount of space in the output
-	// buffer.
-	int64_t totalSizeBytes = 0;
-	openvdb::GridCPtrVec gridsToWrite;
-	std::vector<std::string> gridNames = vdbObject->gridNames();
-	try
-	{
-		for( const std::string& gridName : gridNames )
-		{
-			openvdb::GridBase::ConstPtr grid = vdbObject->findGrid( gridName );
-			totalSizeBytes += grid->metaValue<int64_t>( "file_mem_bytes" );
-			gridsToWrite.push_back( grid );
-		}
-	}
-	catch( const std::exception & )
-	{
-		IECore::msg( IECore::MessageHandler::Warning, "VDBObject::memoryBuffer", "Unable to estimate vdb size." );
-	}
-
 	IECore::UCharVectorDataPtr buffer = new IECore::UCharVectorData();
-	buffer->writable().reserve( totalSizeBytes );
 	UCharVectorDataSink sink( buffer.get() );
 	boost::iostreams::stream<UCharVectorDataSink> memoryStream( sink );
 
 	openvdb::io::Stream vdbStream( memoryStream );
+
+	openvdb::GridCPtrVec gridsToWrite;
+	std::vector<std::string> gridNames = vdbObject->gridNames();
+	for( const std::string& gridName : gridNames )
+	{
+		gridsToWrite.push_back( vdbObject->findGrid( gridName ) );
+	}
 	vdbStream.write( gridsToWrite );
 
 	return buffer;