A licensee has reported that the ZoomFactor variable inside of the DrawPoint function is being calculated incorrectly. It currently is calculated using the lesser of the projection matrix's horizontal or vertical scale. Later in the function when the point is being drawn, horizontal values are used in either case. This causes points to grow in size when the viewport is taller than it is wide.
This is where the ZoomFactor is calculated
Engine\Source\Runtime\Renderer\Private\DynamicPrimitiveDrawing.inl:
bool bIsPerspective = (ViewInfo->ViewMatrices.GetProjectionMatrix().M[3][3] < 1.0f) ? true : false;
if( !bIsPerspective )
{
const float ZoomFactor = FMath::Min<float>(View->ViewMatrices.GetProjectionMatrix().M[0][0], View->ViewMatrices.GetProjectionMatrix().M[1][1]);
ScaledPointSize = ScaledPointSize / ZoomFactor;
}
This is where the point is drawn:
Engine\Source\Runtime\Engine\Private\BatchedElements.cpp
// Generate vertices for the point such that the post-transform point size is constant. const uint32 ViewportMajorAxis = ViewportSizeX;//FMath::Max(ViewportSizeX, ViewportSizeY); const FVector WorldPointX = CameraX * Point.Size / ViewportMajorAxis * TransformedPosition.W; const FVector WorldPointY = CameraY * -Point.Size / ViewportMajorAxis * TransformedPosition.W;
The licensee proposes that this can be fixed by changing the following line:
const float ZoomFactor = FMath::Min<float>(View->ViewMatrices.ProjMatrix.M[0][0], View->ViewMatrices.ProjMatrix.M[1][1]);
to
const float ZoomFactor = View->ViewMatrices.ProjMatrix.M[0][0];
Regression?: No
This was present in 4.13 as well
Code issue - N/A
Why does the REMOVE method of map container remove elements have memory leaks?
How does TextureRenderTarget2D get TArray<uint8> type data?
How do I set a material as a post-processing material?
What is the difference between Camera and CineCamera?
How to delete some elements correctly when deleting an array loop?
What is the cause of the packaging error falling back to 'GameUserSettings' in ue5?
How to assign a value to a member of UMG's UMaterialInterface pointer type in C++?
There's no existing public thread on this issue, so head over to Questions & Answers just mention UE-42157 in the post.