The issue was that the code that captures screenshots on a device and sends them to the editor was providing a path that represented what a screenshot *would* be saved at on the local system. (The screenshot was not actually saved, it's returned as raw data in the message).
This meant that the editor would try to convert this to a relative path for its needs which would fail as the the path roots were differenti (e.g. g:\settings\ on xbox).
The fixes here are -
1) Add some warnings to the code that tries to save screenshots if that fails
2) Change the names of the screenshot functions in AutomationCommon to clarify what they should be used for.
3) Change the 'path' variable in the message to screenshot name.
4) Calculate the path to use for screenshots on the editor.
#tests ran EngineTest on WIn64, PS4, and XboxOne (which now passes).
#jira UE-69399
#rb swarm
[CL 11733277 by Andrew Grant in 4.25 branch]
- Bug fix: Browser: Renamed "..." button to "Explore" (Explore the Trace Store Directory). Also aligned the button to the right. (UE-89187)
- Bug fix: Renamed command line params to analyze a session from "-TraceFile" / "-TraceId" to "-OpenTraceFile" / "-OpenTraceId" (in order to avoid conflict with "-TraceFile" cmd line param already in use to direct tracing to a file).
#jira UE-89187
#rb none
[CL 11576085 by ionut matasaru in 4.25 branch]
Added UWorld information so data can be differentiated more easily.
Collapsed anim sequence player tracks into a single track to make better use of space and to reduce track-explosion.
Fixed world tick so that contiguous curves are shown correctly (this uses a world subsystem to keep track of the workds' actual tick count).
Fix duplicate values being reported by TPointTimeline iterations if bEnumerateOutsideRange was set.
Fix FPS drops when viewing pose and notify tracks (removed Snprintf per-event etc.).
Fixed frame rounding and removed duplicate calls to the frame provider.
Fixed notify event names.
Gameplay graph tracks now expand without animating, as graph tracks recompute their points snapped Y values each time they resize, this becomes very expensive.
#jira UE-88153 - Collapse animation sequence player into a single insights track
#jira UE-88154 - Add NetMode (and WorldType) information to UWorld labels within GameplayInsights
#rb Jurre.deBaare
[CL 11564062 by Thomas Sarkanen in 4.25 branch]
- Added AppId, AppVersion and UserId as FEditorAnalyticSession members and updated FEditorSessionSummaryWriter to write them.
- Changed FEngineAnalytics::GetProvider() to return IAnalyticsProviderET rather than IAnalyticsProvider.
- Updated FEditorSessionSummarySender to embed the AppId/AppVersion/UserId as attribute when sending the summary event and also configured the AnalyticsProvider to use the SessionID/AppID/AppVersion/UserID of the event rather than the current one.
- Added a function to return the current IAnalyticProviderET configuration.
#rb Wes.Hunt
Edigrated 11445334 and 11459406.
[CL 11512055 by Patrick Laflamme in 4.25 branch]
Fixes off-by-one error in frames provider
This caused frames to be mis-reported when using FFrameProvider::GetFrameFromTime.
#jira UE-88149 - Previous frame button on Gameplay insights transport control has no impact
#rb Jurre.deBaare
[CL 11498367 by Thomas Sarkanen in 4.25 branch]
