UE5.3-2023.1-rc2

Changelog

General:

• Fixes empty engine version warning
• Fixes nDisplayExtensions leftover reference warning
• Condenses IMCs in one Folder instead of spread out through all components
• Fixes interaction test map
• Fixes crash in TeleportationComponent due to unique naming of NiagaraSystem
• Adds first iteration of ContentExamples style map showcasing and explaining the available functionality

Pawn/Interactions:

• Adds support for interacting with objects through their child geometries (@daniel.rupp @marcel.krueger pls add docu from MR to wiki)
• Adds flag to only grab closest actor
• Renames PawnMovement to CollisionHandlingMovement
• Introduce BP_BasePawn and BP_DefaultPawn, which extends the base pawn with additional components
• Fixes corrupted Pawn BPs
• Clarifies naming on BP_DefaultPawn components
• Adds support to add IMCs via a list
• More fitting naming scheme:
  • Grab -> DirectInteraction
  • RaycastSelection -> RaycastInteraction
  • Selectable -> Interactable
• Adds missing input mappings for e.g. HP Reverb and WidgetInteraction
• Fixes issue with CollisionHandlingMovement: Pawn was getting stuck in objects
• Adds new improved IntenSelect interaction method

CI:

• Switches to generalized docker images instead of version specific runners
• Uses UNREAL_VERSION instead of tags to determine Unreal Engine build and launch version
• Changes only keyword to preferred rules keyword
• Now runs for all MR events

Cave:

• IsValid check to CaveSetup to prevent nullref crash
• Fixes Cave Setup attachment when previously spawned pawn was rotated

Source/RWTHVRToolkit/Private/Pawn/Navigation/CollisionHandlingMovement.cpp

@@ -67,13 +67,21 @@ void UCollisionHandlingMovement::TickComponent(float DeltaTime, enum ELevelTick
 			}
 		}
 
+		// in case we are in a collision and collision checks are temporarily deactivated, we only allow physical
+		// movement without any checks, otherwise check collision during physical movement
+		if (bCollisionChecksTemporarilyDeactivated)
+		{
+			ConsumeInputVector();
+		}
+		else
+		{
+			// so we add stepping-up (for both walk and fly)
+			// and gravity for walking only
+			MoveByGravityOrStepUp(DeltaTime);
 
-		// so we add stepping-up (for both walk and fly)
-		// and gravity for walking only
-		MoveByGravityOrStepUp(DeltaTime);
-
-		// if we physically (in the tracking space) walked into something, move the world away (by moving the pawn)
-		CheckForPhysWalkingCollision();
+			// if we physically (in the tracking space) walked into something, move the world away (by moving the pawn)
+			CheckForPhysWalkingCollision();
+		}
 	}
 
 	if (NavigationMode == EVRNavigationModes::NAV_NONE)
@@ -150,6 +158,7 @@ void UCollisionHandlingMovement::CheckAndRevertCollisionSinceLastTick()
 		if (!CreateCapsuleTrace(CapsuleLocation, CapsuleLocation).bBlockingHit)
 		{
 			LastCollisionFreeCapsulePosition = CapsuleLocation;
+			bCollisionChecksTemporarilyDeactivated = false;
 		}
 		return;
 	}
@@ -157,8 +166,17 @@ void UCollisionHandlingMovement::CheckAndRevertCollisionSinceLastTick()
 	// check whether we are in a collision at the current position
 	if (CreateCapsuleTrace(CapsuleLocation, CapsuleLocation).bBlockingHit)
 	{
-		// if so move back to last position
-		UpdatedComponent->AddWorldOffset(LastCollisionFreeCapsulePosition.GetValue() - CapsuleLocation);
+		// if so move back to last position, but only if that position is still collision free
+		// since the user might have moveed physically in between
+		FVector LastCapsulePos = LastCollisionFreeCapsulePosition.GetValue();
+		if (!CreateCapsuleTrace(LastCapsulePos, LastCapsulePos).bBlockingHit)
+		{
+			UpdatedComponent->AddWorldOffset(LastCapsulePos - CapsuleLocation);
+		}
+		else
+		{
+			bCollisionChecksTemporarilyDeactivated = true;
+		}
 	}
 	else
 	{
@@ -175,6 +193,13 @@ void UCollisionHandlingMovement::MoveOutOfNewDynamicCollisions()
 		FVector ResolveDirection = 1.5f * ResolveDirectionOptional.GetValue(); // scale it up for security distance
 		UpdatedComponent->AddWorldOffset(ResolveDirection);
 
+		// check whether this helped in resolving the collision, and if not deactivate collision checks temporarily
+		if (CreateCapsuleTrace(UpdatedComponent->GetComponentLocation(), UpdatedComponent->GetComponentLocation())
+				.bBlockingHit)
+		{
+			bCollisionChecksTemporarilyDeactivated = true;
+		}
+
 		// invalidate the last collision-free position, since apparently something changed so we got into this collision
 		LastCollisionFreeCapsulePosition.Reset();
 	}
@@ -195,8 +220,8 @@ void UCollisionHandlingMovement::CheckForPhysWalkingCollision()
 	if (HitResult.bBlockingHit)
 	{
 		const FVector MoveOutVector = HitResult.Location - CapsuleLocation;
-		// move it out twice as far, to avoid getting stuck situations
-		UpdatedComponent->AddWorldOffset(2 * MoveOutVector);
+		// move it out a bit farther, to avoid getting stuck situations
+		UpdatedComponent->AddWorldOffset(1.2f * MoveOutVector);
 	}
 }