- Fixed an issue that caused landscape inputs to trigger unnecessary …

…checkouts when reading landscape data.

Source/HoudiniEngine/Private/UnrealLandscapeTranslator.cpp

@@ -74,11 +74,7 @@ FUnrealLandscapeTranslator::CreateMeshOrPointsFromLandscape(
 
 	if(!FHoudiniEngineUtils::HapiCookNode(InputNodeId, nullptr, true))
 		return false;
-	/*
-	HAPI_CookOptions CookOptions = FHoudiniEngine::GetDefaultCookOptions();
-	HOUDINI_CHECK_ERROR_RETURN(FHoudiniApi::CookNode(
-		FHoudiniEngine::Get().GetSession(), InputNodeId, &CookOptions), false);
-	*/
+
 	//--------------------------------------------------------------------------------------------------
     // 2. Set the part info
     //--------------------------------------------------------------------------------------------------
@@ -263,13 +259,6 @@ FUnrealLandscapeTranslator::CreateMeshOrPointsFromLandscape(
 	HOUDINI_CHECK_ERROR_RETURN(FHoudiniApi::CommitGeo(
 		FHoudiniEngine::Get().GetSession(), DisplayGeoInfo.nodeId), false);
 
-	// TODO: Remove me!
-	/*
-	HAPI_CookOptions CookOptions = FHoudiniEngine::GetDefaultCookOptions();
-	HOUDINI_CHECK_ERROR_RETURN(FHoudiniApi::CookNode(
-		FHoudiniEngine::Get().GetSession(), InputNodeId, &CookOptions), false);
-	*/
-
 	return FHoudiniEngineUtils::HapiCookNode(InputNodeId, nullptr, true);
 }
 
@@ -498,11 +487,6 @@ FUnrealLandscapeTranslator::CreateHeightfieldFromLandscape(
 	*/
 
 	// Finally, cook the Heightfield node
-	/*
-	HAPI_CookOptions CookOptions = FHoudiniEngine::GetDefaultCookOptions();
-	HOUDINI_CHECK_ERROR_RETURN(FHoudiniApi::CookNode(
-		FHoudiniEngine::Get().GetSession(), HeightFieldId, &CookOptions), false);
-	*/
 	if(!FHoudiniEngineUtils::HapiCookNode(HeightFieldId, nullptr, true))
 		return false;
 
@@ -722,8 +706,9 @@ FUnrealLandscapeTranslator::CreateHeightfieldFromLandscapeComponent(
 	*/
 
 	// Finally, cook the Heightfield node
-	HOUDINI_CHECK_ERROR_RETURN(FHoudiniApi::CookNode(
-	FHoudiniEngine::Get().GetSession(), HeightFieldId, nullptr), false);
+	if (!FHoudiniEngineUtils::HapiCookNode(HeightFieldId, nullptr, true))
+		return false;
+
 	return true;
 }
 
@@ -968,6 +953,9 @@ FUnrealLandscapeTranslator::GetLandscapeData(
 
 	// Extracting the uint16 values from the landscape 
 	FLandscapeEditDataInterface LandscapeEdit(LandscapeInfo);
+	// Ensure we're not triggering a checkout, as we're just reading data
+	LandscapeEdit.SetShouldDirtyPackage(false);
+
 	HeightData.AddZeroed(XSize * YSize);
 	LandscapeEdit.GetHeightDataFast(MinX, MinY, MaxX, MaxY, HeightData.GetData(), 0);
 
@@ -1198,13 +1186,6 @@ FUnrealLandscapeTranslator::CreateHeightfieldInputNode(
 
 	// Cook it
 	return FHoudiniEngineUtils::HapiCookNode(HeightfieldNodeId, nullptr, true);
-	/*
-	HAPI_CookOptions CookOptions = FHoudiniEngine::GetDefaultCookOptions();
-	HOUDINI_CHECK_ERROR_RETURN(FHoudiniApi::CookNode(
-		FHoudiniEngine::Get().GetSession(), HeightfieldNodeId, &CookOptions), false);
-
-	return true;
-	*/
 }
 
 bool
@@ -1216,11 +1197,6 @@ FUnrealLandscapeTranslator::SetHeightfieldData(
 	const FString& HeightfieldName)
 {
 	// Cook the node to get proper infos on it
-	/*
-	HAPI_CookOptions CookOptions = FHoudiniEngine::GetDefaultCookOptions();
-	HOUDINI_CHECK_ERROR_RETURN(FHoudiniApi::CookNode(
-		FHoudiniEngine::Get().GetSession(), VolumeNodeId, &CookOptions), false);
-	*/
 	if(!FHoudiniEngineUtils::HapiCookNode(VolumeNodeId, nullptr, true))
 		return false;
 
@@ -1455,6 +1431,9 @@ FUnrealLandscapeTranslator::GetLandscapeLayerData(
 
 	// extracting the uint8 values from the layer
 	FLandscapeEditDataInterface LandscapeEdit(LandscapeInfo);
+	// Ensure we're not triggering a checkout, as we're just reading data
+	LandscapeEdit.SetShouldDirtyPackage(false);
+
 	LayerData.AddZeroed(XSize * YSize);
 	LandscapeEdit.GetWeightDataFast(LayerInfo, MinX, MinY, MaxX, MaxY, LayerData.GetData(), 0);