Commit Graph

62 Commits

Author SHA1 Message Date
aurel cordonnier
43fa62fcd8 Merge from Release-Engine-Test @ 16487383 to UE5/Main
This represents UE4/Main @ 16445039 and Dev-PerfTest @ 16444526

[CL 16488106 by aurel cordonnier in ue5-main branch]
2021-05-27 13:40:37 -04:00
ben woodhouse
fb1017baa8 PerfReportTool 4.49.2 - Fix non-rounded numbers in the FPSChart summary
Update test data
[FYI] robert.millar

#ROBOMERGE-SOURCE: CL 16479765 via CL 16479776 via CL 16479781 via CL 16479784 via CL 16479786
#ROBOMERGE-BOT: STARSHIP (Release-Engine-Test -> Main) (v823-16466674)

[CL 16479790 by ben woodhouse in ue5-main branch]
2021-05-26 21:38:46 -04:00
ben woodhouse
29e891e9a7 PerfReportTool 4.49.1: Fix access none with single CSV mode peak summary
#ROBOMERGE-OWNER: ben.woodhouse
#ROBOMERGE-AUTHOR: ben.woodhouse
#ROBOMERGE-SOURCE: CL 16479526 via CL 16479531 via CL 16479538 via CL 16479542 via CL 16479692
#ROBOMERGE-BOT: STARSHIP (Release-Engine-Test -> Main) (v823-16466674)

[CL 16479693 by ben woodhouse in ue5-main branch]
2021-05-26 21:23:11 -04:00
ben woodhouse
e3dcc8c844 PerfReportTool 4.49: Fix latent bug with PRC color serialization (ToString() not implemented).
- Don't specify colours for PeakSummary report type. Pass null instead and have it set them automatically (this also avoids the need to serialize)

#ROBOMERGE-OWNER: ben.woodhouse
#ROBOMERGE-AUTHOR: ben.woodhouse
#ROBOMERGE-SOURCE: CL 16459036 via CL 16459039 via CL 16459051 via CL 16459069 via CL 16459138
#ROBOMERGE-BOT: STARSHIP (Release-Engine-Test -> Main) (v818-16446889)

[CL 16459144 by ben woodhouse in ue5-main branch]
2021-05-25 20:23:17 -04:00
ben woodhouse
f0d083fc1b PerfReportTool 4.48
- Refactor Peaks Summary to remove a lot of legacy code
  - Sections can now be specified via summarySection elements in the XML
  - Remove inMainSummary from graph elements
- Fix precision issue with SummaryTable data. Always store as doubles - never go via strings for numeric data
   - NOTE: we are intentionally not bumping the SummaryTableCache version to avoid unnecessary churn here. Users can force an invalidate if necessary
- Pre-sort summary table to ensure deterministic output
- Update test results

#ROBOMERGE-OWNER: ben.woodhouse
#ROBOMERGE-AUTHOR: ben.woodhouse
#ROBOMERGE-SOURCE: CL 16451878 via CL 16452760 via CL 16452854 via CL 16452934 via CL 16459127
#ROBOMERGE-BOT: STARSHIP (Release-Engine-Test -> Main) (v818-16446889)

[CL 16459131 by ben woodhouse in ue5-main branch]
2021-05-25 20:22:29 -04:00
ben woodhouse
3a4ab3758b Integrate CL 16308862 from Dev-EngineMerge: PerfReportTool 4.47 - ExtraLinks summary
Change 16308862 by ben.woodhouse[at]ROBOMERGE_FORTNITE_Dev_EngineMerge on 2021/05/12 22:00:30

PerfReportTool 4.47 - ExtraLinks summary
Adds a new ExtraLinks summary type, which reads a <csvfilename>.links file. The file contains lines with additional links in the format:
LongName,ShortName,URL

These links are added to a new Links section of the report. They are also added to summary table row, with the LongName as the key.

The section can be added to all reports by adding with -extraLinksSummary, or add the summary type "ExtraLinks" to the Report type XML.

#ROBOMERGE-SOURCE: CL 16308651 via CL 16309102 via CL 16315492 via CL 16315504
#ROBOMERGE-BOT: STARSHIP (Release-Engine-Test -> Main) (v804-16311228)

[CL 16315530 by ben woodhouse in ue5-main branch]
2021-05-13 13:55:13 -04:00
aurel cordonnier
8eebe8841f Merge UE5/RET @ 16305968 to UE5/Main
This represents UE4/Main @ 16261013 and Dev-PerfTest @ 16259937

[CL 16306996 by aurel cordonnier in ue5-main branch]
2021-05-12 18:10:03 -04:00
ben woodhouse
cdeb752863 PerfReportTool 4.45
- make -csvLinkRootPath work any type of path. Links are made relative to the passed-in csv directory and then the new root is added

#ROBOMERGE-SOURCE: CL 16178660 via CL 16178671 via CL 16178673 via CL 16208175
#ROBOMERGE-BOT: STARSHIP (Release-Engine-Staging -> Main) (v796-16191650)

[CL 16208185 by ben woodhouse in ue5-main branch]
2021-05-05 11:51:38 -04:00
ben woodhouse
ea37cceb85 PerfReportTool 4.44
-add -reportLinkRootPath arg to make report links relative to a particular location
-add -csvLinkRootPath arg to make CSV links relative to a particular location
-add -listFiles arg to just list files that pass the filter instead of processing them

#ROBOMERGE-OWNER: ben.woodhouse
#ROBOMERGE-AUTHOR: ben.woodhouse
#ROBOMERGE-SOURCE: CL 16177813 via CL 16177817 via CL 16177818 via CL 16178057
#ROBOMERGE-BOT: STARSHIP (Release-Engine-Staging -> Main) (v792-16166999)

[CL 16178065 by ben woodhouse in ue5-main branch]
2021-05-01 15:23:22 -04:00
aurel cordonnier
50944fd712 Merge UE5/RES @ 16162155 to UE5/Main
This represents UE4/Main @ 16130047 and Dev-PerfTest @ 16126156

[CL 16163576 by aurel cordonnier in ue5-main branch]
2021-04-29 19:32:06 -04:00
ben woodhouse
e3ad811d6b Add missing file from CL 16110876
#ROBOMERGE-OWNER: ben.woodhouse
#ROBOMERGE-AUTHOR: ben.woodhouse
#ROBOMERGE-COMMAND: _robomerge ue5-main
#ROBOMERGE-SOURCE: CL 16110880 via CL 16110883 via CL 16110885 via CL 16110886
#ROBOMERGE-BOT: STARSHIP (Release-Engine-Staging -> Main) (v789-15992632)

[CL 16117146 by ben woodhouse in ue5-main branch]
2021-04-26 14:45:34 -04:00
ben woodhouse
d36b7af983 PerfReportTool 4.43
- Add support for >,<,>=,<= operators for metadata queries (for numerical fields)
- Use doubles for SummaryTable/SummaryTableColumn values, so we can accurately show display values (does not affect summary cache - this was already at double precision)

#ROBOMERGE-OWNER: ben.woodhouse
#ROBOMERGE-AUTHOR: ben.woodhouse
#ROBOMERGE-COMMAND: _robomerge ue5-main
#ROBOMERGE-SOURCE: CL 16110876 via CL 16110877 via CL 16110881 via CL 16110882
#ROBOMERGE-BOT: STARSHIP (Release-Engine-Staging -> Main) (v789-15992632)

[CL 16117137 by ben woodhouse in ue5-main branch]
2021-04-26 14:44:53 -04:00
ben woodhouse
867df3f118 PerfReportTool 4.42
- Add -requireMetadata option to ignore CSVs without metadata
- Clean up help text so the lines arent too long for standard console output

#ROBOMERGE-OWNER: ben.woodhouse
#ROBOMERGE-AUTHOR: ben.woodhouse
#ROBOMERGE-COMMAND: _robomerge ue5-main
#ROBOMERGE-SOURCE: CL 16105233 via CL 16105248 via CL 16105254 via CL 16105255
#ROBOMERGE-BOT: STARSHIP (Release-Engine-Staging -> Main) (v789-15992632)

[CL 16116861 by ben woodhouse in ue5-main branch]
2021-04-26 14:28:01 -04:00
ben woodhouse
41de178a81 PerfReportTool 4.41
- add -summaryTableCacheUseOnlyCsvID param to ignore the ReportType hash and just use the CSV ID for summaryTableCache ID entries. This is a good idea if you want these files to persist and you don't want them to be regenerated when report types change
- Add ReportType and ReportTypeID to the summaryTable ToolMetaData entries
- Use the reportType name as part of the report hash (this will invalidate existing cache files!)

#ROBOMERGE-OWNER: ben.woodhouse
#ROBOMERGE-AUTHOR: ben.woodhouse
#ROBOMERGE-COMMAND: _robomerge ue5-main
#ROBOMERGE-SOURCE: CL 16024419 via CL 16024452 via CL 16024531 via CL 16045954
#ROBOMERGE-BOT: STARSHIP (Release-Engine-Staging -> Main) (v789-15992632)

[CL 16116846 by ben woodhouse in ue5-main branch]
2021-04-26 14:27:22 -04:00
ben woodhouse
14fb564e7e PerfReportTool 4.40
- Refactor code to avoid use of the term Metadata to refer to summary table row data. The term metadata was easily confused with CSV metadata, and in most cases, it wasn't actually metadata.
- This refactor resulted in some minor breaking changes in XML format and command line params. The affected reports and scripts have been fixed up accordingly

The following were renamed:
SummaryMetadata -> SummaryTableRowData
SummaryMetadataValue->SummaryTableElement
SummaryMetadataTable->SummaryTable
metadataKey -> summaryStatName (also affects XML)
metadataPrefix -> summaryStatNamePrefix (also affects XML)
metadataCache -> summaryTableCache
Commandline params, e.g: -metadataCache -> -summaryMetadataCache


#ROBOMERGE-OWNER: ben.woodhouse
#ROBOMERGE-AUTHOR: ben.woodhouse
#ROBOMERGE-COMMAND: _robomerge ue5-main
#ROBOMERGE-SOURCE: CL 16023141 via CL 16023153 via CL 16023173 via CL 16045518
#ROBOMERGE-BOT: STARSHIP (Release-Engine-Staging -> Main) (v789-15992632)

[CL 16116817 by ben woodhouse in ue5-main branch]
2021-04-26 14:25:00 -04:00
ben woodhouse
5a8d2051f7 PerfReportTool 4.34
-5x faster metadata cache only reads:
 - Only read the initial CSV metadata entries before metadata filtering (since these always appear first)
 - Skip report type determiniation in metadataonly mode (this is slow and unnecessary)
 -crash fix (don't crash if old version metadata is encountered, just skip and log)

Querying a single device profile across 31k CSVs of mixed device profiles takes 2.1s instead of 10.2s previously

#ROBOMERGE-OWNER: ben.woodhouse
#ROBOMERGE-AUTHOR: ben.woodhouse
#ROBOMERGE-COMMAND: _robomerge ue5-main
#ROBOMERGE-SOURCE: CL 16015980 via CL 16015986 via CL 16015993 via CL 16043535
#ROBOMERGE-BOT: STARSHIP (Release-Engine-Staging -> Main) (v789-15992632)

[CL 16116807 by ben woodhouse in ue5-main branch]
2021-04-26 14:24:18 -04:00
ben woodhouse
08525cb9cf PerfReportTool 4.33
Open metadata and .bin files readonly

#ROBOMERGE-OWNER: ben.woodhouse
#ROBOMERGE-AUTHOR: ben.woodhouse
#ROBOMERGE-COMMAND: _robomerge ue5-main
#ROBOMERGE-SOURCE: CL 16013849 via CL 16013851 via CL 16013859 via CL 16043212
#ROBOMERGE-BOT: STARSHIP (Release-Engine-Staging -> Main) (v789-15992632)

[CL 16116794 by ben woodhouse in ue5-main branch]
2021-04-26 14:22:36 -04:00
ben woodhouse
177a5c926e PerfReportTool 4.32
- Add -metadataCacheIn mode for running without CSVs
 - Use a BlockingCollection to eliminate precache threading bottleneck and simplify precache thread producer/consumer code
 - Threading still hits a performance bottleneck at around 6 threads when running metadata queries currently, and gets slower after that as more threads are added.
    - Need further work to optimize GetReportTypeXML (too much unnecessary linq) and SummaryMetadata::TryReadFromCache

#ROBOMERGE-OWNER: ben.woodhouse
#ROBOMERGE-AUTHOR: ben.woodhouse
#ROBOMERGE-COMMAND: _robomerge ue5-main
#ROBOMERGE-SOURCE: CL 15974021 via CL 15974024 via CL 15974028 via CL 16027282
#ROBOMERGE-BOT: STARSHIP (Release-Engine-Staging -> Main) (v789-15992632)

[CL 16112788 by ben woodhouse in ue5-main branch]
2021-04-26 06:55:45 -04:00
Marc Audy
01b7c9f4f5 Merge UE5/RES @ 15958325 to UE5/Main
This represents UE4/Main @ 15913390 and Dev-PerfTest @ 15913304

[CL 15958515 by Marc Audy in ue5-main branch]
2021-04-08 14:32:07 -04:00
Marc Audy
0cbbc781ca Merge UE5/Release-Engine-Staging @ 15740152 to UE5/Main
This represents UE4/Main @ 15709114

[CL 15740605 by Marc Audy in ue5-main branch]
2021-03-18 15:20:03 -04:00
Marc Audy
8f73cd7fa9 Merge UE5/Release-Engine-Staging @ 15630841 to UE5/Main
This represents UE4/Main @ 15601601

[CL 15631170 by Marc Audy in ue5-main branch]
2021-03-05 19:27:14 -04:00
Marc Audy
ada7c144fa Merge //UE5/Release-Engine-Staging @14903491 to //UE5/Main
[CL 14906022 by Marc Audy in ue5-main branch]
2020-12-11 14:21:20 -04:00
Marcus Wassmer
3b81cf8201 Merging using //UE5/Main_to_//UE5/Release-Engine-Staging @14384769
autoresolved files
#rb none

[CL 14384911 by Marcus Wassmer in ue5-main branch]
2020-09-24 00:43:27 -04:00
Marc Audy
7379fa99c5 Merging //UE5/Release-Engine-Staging to Main (//UE5/Main) @ 14229157
[CL 14233282 by Marc Audy in ue5-main branch]
2020-09-01 14:07:48 -04:00
Marc Audy
a7c9001a94 Merging //UE5/Release-Engine-Staging to Main (//UE5/Main) @ 14075166
#rb
#rnx

[CL 14075271 by Marc Audy in ue5-main branch]
2020-08-11 01:36:57 -04:00