Commit Graph

13 Commits

Author SHA1 Message Date
paul chipchase
b9c03ad503 Move utility function inside the anonymous namespace.
#rb trivial
#rnx
#preflight 63bfcd344b018bfa80f6439c

[CL 23660394 by paul chipchase in ue5-main branch]
2023-01-12 05:49:36 -05:00
paul chipchase
7c4a834261 Change TryVirtualizePackages/TryRehydratePackages to return the corresponding results structure rather than an enum value.
#rb Per.Larsson
#jira UE-169626
#rnx
#preflight 63bd670a71079a8d1c0e837b

- Since the API was forcing the caller to pass in a results structure to be filled in, we might as well make it the return value.
- Added a ::WasSuccessful method to the results structures that can be used instead of checking if the result had errors or not.
- Remove the reset method from FVirtualizationResult/FRehydrationResult as they no longer need it.
- The older deprecated methods still use the results enum, so we cannot easily deprecate those enums yet.

[CL 23626072 by paul chipchase in ue5-main branch]
2023-01-10 09:15:11 -05:00
paul chipchase
66b32a743f Add a new overload to IVirtualizationSystem::TryRehydratePackages, which takes additional options (via a bitfield enum) and returns more info about the resulting process. The original version has been deprecated.
#rb Per.Larsson
#jira UE-169626
#rnx
#preflight 63bd0ebfd862fdd347bce1fe

### VA System
- This allows us to provide the user with more ways to customize the rehydration and return more detailed info about it if the calling code wishes to log additional info. In both cases we can extend the options and the data returned without changing the API.
- At the moment the only flag we support is 'Checkout', which requests that the rehydration process checkout any package that it needs to modify rather than giving an error. This means that the user does not need to check packages out from revision control before running the rehydration process.
-- We still check if packages can be modified and warn the user if they can't, as package files could be locked in other ways.
- The rehydration process will now long the time taken if verbose logging is set for the category 'LogVirtualization'

### UnrealVirtualizationTool
- The virtualize command now reports how many packages were checked out if the flag was set.
- The rehydration command now supports a '-Checkout' commandline flag, which when enabled will use the new api to checkout the packages that need to be checked out when rehydrated.

[CL 23625132 by paul chipchase in ue5-main branch]
2023-01-10 06:27:20 -05:00
paul chipchase
50ce43df37 Add a new commandlet GeneratePayloadManifest that will dump all of the payload info about given set of packages.
#rb Per.Larsson
#jira UE-169895
#rnx
#preflight 639204815624e6da5ef1ba93

- This doesn't have great potential for production use but can be useful for teams when trying to work out which filters and limits to use when turning on asset virtualization.

[CL 23461404 by paul chipchase in ue5-main branch]
2022-12-09 03:39:15 -05:00
henrik karlsson
fa90b399a4 Added includes for future change. This changelist only contains added #include and a couple of empty placeholder files
Tested compiling fortnite, unrealeditor, lyra, qagame with non-unity/pch

#preflight 63635997876630122adeab9f
#rb none

[CL 22958990 by henrik karlsson in ue5-main branch]
2022-11-03 14:18:47 -04:00
paul chipchase
d6298779b2 Add more profiling scopes to the PrecachePayloads commandlet
#rb trivial
#rnx
#preflight 635b9fef7515720a5fb1bfe7

[CL 22827291 by paul chipchase in ue5-main branch]
2022-10-28 08:34:16 -04:00
paul chipchase
e42dbfc7bc Drastically reduce the amount of time payload precaching spends trying to extract payload information from packages.
#rb Per.Larsson
#jira UE-167457
#rnx
#preflight 635a75130d3a231123c51720

- Use ParallelForWithTaskContext to iterate ove rthe list of packages and extra the payload information.
- This has been shown to givea speed up of 5x-10x depending on the machine being tested.

[CL 22810930 by paul chipchase in ue5-main branch]
2022-10-27 11:33:53 -04:00
paul chipchase
446d9b7918 Add two new paths to the PreCachePayloads commandlet that take advantage of the new batch API.
#rb Per.Larsson
#jira UE-163093
#rnx
#preflight 63528929ae33b04ec1f4d892

- Iteration method 4 requests all found payloads in one single large batch. Useful for testing but would easily OOM systems when used on real projects.
- Iteration method 5 is similar to method 3, it splits the work into smaller jobs of 64 payloads at a time and spreads them out over our cores. This is the new method and in tests has managed to reach about 800mbit/s from HordeStorage.

[CL 22707967 by paul chipchase in ue5-main branch]
2022-10-21 22:28:45 -04:00
bryan sefcik
483f93dc1a Fixed misc include paths
#jira
#preflight 634f491669246074dbb2719d

[CL 22633214 by bryan sefcik in ue5-main branch]
2022-10-19 14:38:31 -04:00
paul chipchase
a4fbae3f12 Add a very basic way to precache all virtualized payloads in a project and store them in the cache storage backends (usually the local DDC in our internal set up)
#rb Per.Larsson
#jira UE-164010
#rnx
#preflight 631f33368c5f7af11bc2a6b5
The commandlet is run via the cmdline '-run=VirtualizationEditor.PrecachePayloads' (the VirtualizationEditor part will load the module before executing the PrecachePayloads commandlet)
At the moment this is very slow to run as the VA API only allows one payload to be pulled in a single call and although we can make requests on many threads, there is a lot of overhead when requesting data from some backends and a batch pull API would be substantially quicker. This will be looked at in a future work item.
There are currently 4 different ways we can iterate over the payloads and precache them, since the commandlet is still too slow to use in general production, the different ways of iterating are provided so we can experiment later.
There is no way to customize what is precached, we just iterate over all the packaged in a project. We might want to improve this in the future but it would perhaps make more sense to move the package iteration options from the resave commandlet and make it something general purpose that different commandlets can also use. This would help build up a consistent way of choosing which packages are affected by a commandlet.

[CL 22050364 by paul chipchase in ue5-main branch]
2022-09-16 11:35:57 -04:00
paul chipchase
57db613f63 Move the rehydration context menu functionality from the statistics panel to its own code file
#rb Per.Larsson
#jira UE-162600
#rnx
#preflight 6317334c7b21ad1e0a1448be

- When the statistics panel lived in the DDC editor module it was easier to keep all virtualization functionality in one place. Now that we have our own module we can move it to its own code file for better readability.
- Added a utility function ::SetupMenuEntries called when the module is started, we should add all future menu entries here.

[CL 21816542 by paul chipchase in ue5-main branch]
2022-09-06 10:13:34 -04:00
paul chipchase
8dd3df9f7e CIS fix - Add missing undef of LOCTEXT_NAMESPACE
#rb trivial
#jira UE-162600
#rnx
#preflight 6316f7a57b21ad1e0a0d917f

[CL 21808827 by paul chipchase in ue5-main branch]
2022-09-06 04:35:06 -04:00
paul chipchase
3eb268abbb Create a new module 'VirtualizationEditor' and move the existing VA editor code there from the 'DerivedDataEditor'
#rb Per.Larsson
#jira UE-162600
#rnx
#preflight 631597667562a90dfa991913

- Up until now we have been piggybacking off the DDC editor module as the VA statistics panel was originally based on the DDC statistic panel but now we are starting to do more VA editor/UX work so we should probably have our own module.
- This does mean we need to remove the the VA statistics panel from the DDC status bar, but it didn't really belong there and the panel can still be opened via the menu 'Tool->Audit->Virtual Assets'

[CL 21790583 by paul chipchase in ue5-main branch]
2022-09-05 06:19:34 -04:00