Alpha 3.1 update

The plug-in is now linked with Houdini 18.0.532 / HAPI3.3.11

New Features:

- Added the ability to split attribute instancers and packed primitives instancers by attribute.
  The attribute used to split the instancers can be set via the "unreal_split_attr" attribute (string).
  The specified attributes values will then be used to create multiple instancers instead of one.
- Instancers now store the split attributes they used so it can be reused to resolve output attributes.
  This allows, for example, to use the tile attribute when baking instancers/meshes to world composition
  (with a tiled landscape) to control the sublevel where the baked actors should be placed.
- Added automatic conversion of attributes when they are of an inappropriate type.
  FHoudiniEngineUtils::HapiGetAttributeDataAsFloat/Integer/String now try to convert attributes to the expected type.
  This will remove frustration and attributes being ignored due to them not being of the expected type.

Bug Fixes:

- Fixed non-editor build errors, game build errors and incorrect initialization warnings when packaging.
- Fixed non-unity build issues caused by #define LOCTEXT_NAMESPACE
- Added 'Scale Changed Uniformly' Lock and Reset to float VEC3 parameter, to Input Geometry objects and instanced objects.
- Parameter UI performance improved:  Construct radio button circle segments exactly once at first use, avoid repeat computation.
- Fixed the 'Editing...' Text in Curve Input UI not updated properly when selecting other types of components in the same HDA.
- Fixed: Made the curve type menu consistent to Houdini. Added 'Points' curve type which displays CVs only.
- Fixed: curves 'reversed' parameter is now synced with Houdini.
- Fixed: bug preventing the editing of curves when cooking is paused.
- Fixed: Unreal Spline Component input cooking infinitely after changing a spline control point or rebuilding.
- Fixed: File/String parameters text field extends when the string is long.
- Fixed: Crash when connecting to a HARS session that was manually started.
  This was caused by calling SetSessionSyncInfo before initializing HAPI.
- Fixed displayed templated geos not updating after having being changed.
- Fixed: PDG Asset Link listing bypassed TOP Networks/TOP Nodes.
- Fixed: Ramp parameters' curves UI disappearing randomly due to GC.
- Fixed: PDG Asset Link listing TOP Networks nested inside other TOP networks.
  This fixes the PDG listing all subnets, ROP Geometry top nodes etc.. found in a HDA.
- Fixed: bug with the PDG Asset Link that prevented certain nodes from being cooked via the UI.
  This was caused by an invalid selection of TOP Nodes due to a mismatch between UI indices and TOP indices caused by hidden nodes.
- Fixed the PDG Asset Link sometimes still an invalid/hidden node after changing the node filter, or after instantiating a new PDG HDA with invalid/hidden nodes.
- Fixed crashes that could happen when recooking a HDA with tiled landscapes that previously baked to WC, after adding new tiles.
- Fixed crash that could happen when loosing the session due to houdini crashing (or being killed manually).
  FHoudiniEngineManager::StopHoudiniTicking() crashed when trying to access/clear the timer out of the game thread.
- Fixed the plugin leaving previous foliage types in the foliage system after changing the foliage mesh.
  When instancing foliage, the previously used foliage type is now destroyed if no instances are left on it after cleaning the HDA's instances.
- Fixed deleting an HDA also deleting the foliage component even if that component still had manually placed instances left.
  We now only delete the foliage component if it doesnt have instances anymore.
- Fixed a bug that prevented clearing string parameters used as Asset pickers.
- Removed outdated warning spam when attempting to fetch unreal_material attributes on instancer:
  "Instancer: the unreal_material attribute must be a primitive or detail attribute, ignoring the attribute."
- Fixed foliage instancers removing/destroying the foliage component for the previously used foliage type
  after changing the instanced mesh, even if said foliage component still had hand-placed instances left.
- Fixed foliage instancers leaving empty foliage types behind after deleting their Houdini asset actor.
- Fixed regression causing "Bake To Actor" to attempt to create a default level when the "unreal_level_path" attribute is not set.
  This caused the baking process to open a "save as" dialog to attempt to save that untitled level...
  Bake to actors now properly uses the current level when the level path attribute is not set. 
- Fixed the "tile" attribute/ "tile_i" token not being properly processed with "unreal_level_path" attributes when baking meshes and instancers to actors.
- Fixed string parameters set as asset picker not updating their thumbnail/selected object after drag & dropping an asset on the widget.
  The string value was properly set to the asset reference, but the widget didn't display the proper asset thumbnail/infos.
- Fixed baking instancers sometimes creating invisible actors.
  This was caused by the default reference mesh setting the HiddenInGame property to true.
- Fixed issues when creating multiple landscape tiles mixed with instancers or other objects that also have the tile attribute.
  This caused the output translator to only create one output (and only one landscape), with all the volumes (height, masks ..) instead of creating an output per tile.
  This was caused by UHoudiniOutput::HeightfieldMatch() not finding previous/new landscape properly.
- Fixed: possible crash in the output details.
- Fixed: Parameter Details endlessly looping due to directories not being parsed correctly if the params were invalid (-1).
- Fixed instancer not properly baking to sublevel via the unreal_level_path attribute.
- Renamed the "tile_i" output token to "tile" to make it compatible with the split instance attribute.
- Fixed: Parameter UI: Wrong divider line on nested tabs
- Fixed: Parameter UI: bugs with the divider of nested multiparm and tabs, and empty multiparm and folder
- Fixed: Parameter UI: tab's innermost divider is too tall for tabs row.

Author: dpernuit

HoudiniEngine.uplugin

@@ -1,8 +1,8 @@
 {
 	"FileVersion" : 3,
 	"FriendlyName" : "Houdini Engine v2",
-	"Version" : 18000499,
-	"VersionName" : "v2 - Alpha3 - H18.0.499",
+	"Version" : 18000532,
+	"VersionName" : "v2 - Alpha3.1 - H18.0.532",
 	"CreatedBy" : "Side Effects Software Inc.",
 	"CreatedByURL" : "http://www.sidefx.com",
 	"DocsURL" : "http://www.sidefx.com/docs/unreal/",

Source/HoudiniEngine/HoudiniEngine.Build.cs

@@ -32,7 +32,7 @@
 
 /*
 
-    Houdini Version: 18.0.499
+    Houdini Version: 18.0.532
     Houdini Engine Version: 3.3.11
     Unreal Version: 4.25.0
 
@@ -47,7 +47,7 @@ public class HoudiniEngine : ModuleRules
 {
     private string GetHFSPath()
     {
-        string HoudiniVersion = "18.0.499";
+        string HoudiniVersion = "18.0.532";
         bool bIsRelease = true;
         string HFSPath = "C:/dev/hfs";
         string RegistryPath = "HKEY_LOCAL_MACHINE\\SOFTWARE\\Side Effects Software";

Source/HoudiniEngine/Private/HoudiniEngine.cpp

@@ -372,6 +372,29 @@ FHoudiniEngine::GetSession() const
 	return Session.type == HAPI_SESSION_MAX ? nullptr : &Session;
 }
 
+HAPI_CookOptions
+FHoudiniEngine::GetDefaultCookOptions()
+{
+	// Default CookOptions
+	HAPI_CookOptions CookOptions;
+	FHoudiniApi::CookOptions_Init(&CookOptions);
+
+	CookOptions.curveRefineLOD = 8.0f;
+	CookOptions.clearErrorsAndWarnings = false;
+	CookOptions.maxVerticesPerPrimitive = 3;
+	CookOptions.splitGeosByGroup = false;
+	CookOptions.splitGeosByAttribute = false;
+	CookOptions.splitAttrSH = 0;
+	CookOptions.refineCurveToLinear = true;
+	CookOptions.handleBoxPartTypes = false;
+	CookOptions.handleSpherePartTypes = false;
+	CookOptions.splitPointsByVertexAttributes = false;
+	CookOptions.packedPrimInstancingMode = HAPI_PACKEDPRIM_INSTANCING_MODE_FLAT;
+	CookOptions.cookTemplatedGeos = true;
+
+	return CookOptions;
+}
+
 bool
 FHoudiniEngine::StartSession(HAPI_Session*& SessionPtr,
 	const bool& StartAutomaticServer,
@@ -491,16 +514,6 @@ FHoudiniEngine::StartSession(HAPI_Session*& SessionPtr,
 	HOUDINI_CHECK_ERROR(FHoudiniApi::GetSessionEnvInt(
 		SessionPtr, HAPI_SESSIONENVINT_LICENSE, (int32 *)&LicenseType));
 
-	// Indicate that Session Sync is enabled
-	if (bEnableSessionSync)
-	{
-		FString Notification = TEXT("Houdini Engine Session Sync enabled.");
-		FHoudiniEngineUtils::CreateSlateNotification(Notification);
-		HOUDINI_LOG_MESSAGE(TEXT("Houdini Engine Session Sync enabled."));
-
-		UploadSessionSyncInfoToHoudini();
-	}
-
 	return true;
 }
 
@@ -568,9 +581,6 @@ FHoudiniEngine::SessionSyncConnect(
 	//bSyncHoudiniViewport = HoudiniRuntimeSettings->bSyncHoudiniViewport;
 	//bSyncUnrealViewport = HoudiniRuntimeSettings->bSyncUnrealViewport;
 
-	// Upload our session sync params to H
-	UploadSessionSyncInfoToHoudini();
-
 	return true;
 }
 
@@ -637,24 +647,13 @@ FHoudiniEngine::InitializeHAPISession()
 	const UHoudiniRuntimeSettings * HoudiniRuntimeSettings = GetDefault< UHoudiniRuntimeSettings >();
 
 	// Default CookOptions
-	HAPI_CookOptions CookOptions;
-	FHoudiniApi::CookOptions_Init(&CookOptions);
-	CookOptions.curveRefineLOD = 8.0f;
-	CookOptions.clearErrorsAndWarnings = false;
-	CookOptions.maxVerticesPerPrimitive = 3;
-	CookOptions.splitGeosByGroup = false;
-	CookOptions.splitGeosByAttribute = false;
-	CookOptions.splitAttrSH = 0;
-	CookOptions.refineCurveToLinear = true;
-	CookOptions.handleBoxPartTypes = false;
-	CookOptions.handleSpherePartTypes = false;
-	CookOptions.splitPointsByVertexAttributes = false;
-	CookOptions.packedPrimInstancingMode = HAPI_PACKEDPRIM_INSTANCING_MODE_FLAT;
-	CookOptions.cookTemplatedGeos = true;
+	HAPI_CookOptions CookOptions = FHoudiniEngine::GetDefaultCookOptions();
 
 	bool bUseCookingThread = true;
 	HAPI_Result Result = FHoudiniApi::Initialize(
-		&Session, &CookOptions, bUseCookingThread,
+		&Session,
+		&CookOptions,
+		bUseCookingThread,
 		HoudiniRuntimeSettings->CookingThreadStackSize,
 		TCHAR_TO_UTF8(*HoudiniRuntimeSettings->HoudiniEnvironmentFiles),
 		TCHAR_TO_UTF8(*HoudiniRuntimeSettings->OtlSearchPath),
@@ -683,6 +682,17 @@ FHoudiniEngine::InitializeHAPISession()
 	// Let HAPI know we are running inside UE4
 	FHoudiniApi::SetServerEnvString(&Session, HAPI_ENV_CLIENT_NAME, HAPI_UNREAL_CLIENT_NAME);
 
+	if (bEnableSessionSync)
+	{
+		// Set the session sync infos if needed
+		UploadSessionSyncInfoToHoudini();
+
+		// Indicate that Session Sync is enabled
+		FString Notification = TEXT("Houdini Engine Session Sync enabled.");
+		FHoudiniEngineUtils::CreateSlateNotification(Notification);
+		HOUDINI_LOG_MESSAGE(TEXT("Houdini Engine Session Sync enabled."));		
+	}
+
 	return true;
 }
 
@@ -1028,12 +1038,17 @@ FHoudiniEngine::UpdateSessionSyncInfoFromHoudini()
 void
 FHoudiniEngine::UploadSessionSyncInfoToHoudini()
 {
+	// No need to set sessionsync info if we're not using session sync
+	if (!bEnableSessionSync)
+		return;
+
 	// Set the Session Sync settings to Houdini
 	HAPI_SessionSyncInfo SessionSyncInfo;
 	SessionSyncInfo.cookUsingHoudiniTime = bCookUsingHoudiniTime;
 	SessionSyncInfo.syncViewport = bSyncViewport;
 
-	FHoudiniApi::SetSessionSyncInfo(&Session, &SessionSyncInfo);
+	if (HAPI_RESULT_SUCCESS != FHoudiniApi::SetSessionSyncInfo(&Session, &SessionSyncInfo))
+		HOUDINI_LOG_WARNING(TEXT("Failed to set the SessionSync Infos."));
 }
 
 #undef LOCTEXT_NAMESPACE

Source/HoudiniEngine/Private/HoudiniEngine.h

@@ -62,6 +62,10 @@ class HOUDINIENGINE_API FHoudiniEngine : public IModuleInterface
 
 		// Session accessor
 		virtual const HAPI_Session* GetSession() const;
+
+		// Default cook options
+		static HAPI_CookOptions GetDefaultCookOptions();
+
 		// Creates a new session
 		bool StartSession(
 			HAPI_Session*& SessionPtr,