One problematic scenario is when all components from an actor are unregistered and one of them is the navigation parent of the others. In this case, the parent may be unregistered before some of its children and then calls to UpdateNavOctreeParentChain from a child may queue back the unregistered parent. UpdateNavOctreeParentChain will now consider the actual status of the parent (Added or queued for add) before registering it or its children.
[at]mieszko.zielinski [at]guillaume.guay
#rb mieszko.zielinski, guillaume.guay
#ROBOMERGE-SOURCE: CL 8552772 via CL 8557168
#ROBOMERGE-BOT: (v406-8472469)
[CL 8557314 by yoan stamant in Main branch]
in the navigation sytem without setting flag bRestrictBuildingToActiveTiles yet. This is required for the initial virtual call to RestrictBuildingToActiveTiles to perform the ActiveTiles initialization.
#rnx
[at]mieszko.zielinski
#rb mieszko.zielinski
#ROBOMERGE-SOURCE: CL 8548670 via CL 8555979
#ROBOMERGE-BOT: (v406-8472469)
[CL 8556169 by yoan stamant in Main branch]
#rnx
[at]maxime.mercier
#rb maxime.mercier
#ROBOMERGE-SOURCE: CL 8364914 via CL 8368369
#ROBOMERGE-BOT: (v402-8361577)
[CL 8368535 by yoan stamant in Main branch]
#rb Yoan.StAmant
Yoan.StAmant
#ROBOMERGE-SOURCE: CL 8210655 via CL 8212294
#ROBOMERGE-BOT: (v401-8057353)
[CL 8212296 by mieszko zielinski in Main branch]
This approach allowed for easy NavigationSystem's config update with NavSystemConfigOverride's properties. NavSystemConfigOverride can now specify if it wants to fully override the pre-existing nav sys instance or if it just wants to append new information (like supported agents) to the existing navigation system instance. There's also an option to do nothing if there's already a navigation system present.
This CL rolls-back a bunch of temp fixes done in past couple of days.
#rb Yoan.StAmant
Yoan.StAmant
#ROBOMERGE-OWNER: mieszko.zielinski
#ROBOMERGE-AUTHOR: mieszko.zielinski
#ROBOMERGE-SOURCE: CL 8187661 via CL 8193395 via CL 8207613
#ROBOMERGE-BOT: (v401-8057353)
[CL 8207758 by mieszko zielinski in Main branch]
#rb Yoan.StAmant
Yoan.StAmant, Maxime.Mercier
#ROBOMERGE-SOURCE: CL 8154693 via CL 8154694 via CL 8154817
#ROBOMERGE-BOT: (v401-8057353)
[CL 8161935 by mieszko zielinski in Main branch]
#jira UE-78811
[at]guillaume.guay [at]mieszko.zielinski
#ROBOMERGE-SOURCE: CL 8119051 via CL 8122721
#ROBOMERGE-BOT: (v401-8057353)
[CL 8124722 by yoan stamant in Main branch]
RecastNavMeshGenerator initialization should be called right after object construction to ensure virtual methods are properly called and derived classes are fully constructed before rebuild navigation mesh.
#rnx
#rb yoan.stamant
[fyi] [at]Mieszko.Zielinski
#ROBOMERGE-SOURCE: CL 7981977 via CL 7996225
#ROBOMERGE-BOT: (v396-7974030)
[CL 7996495 by guillaume guay in Main branch]
- Added navigation system config 'bGenerateNavDataWhenNoCompatibleNavBound". When set to TRUE (default) it will still generate navigation data if any NavBoundVolume is compatible with its NavDataConfig, if false it gets rid of that navigation data;
- Make sure that when "bSpawnNavDataInNavBoundsLevel" is TRUE, it builds the navigation data in the first level containing a NavBoundVolume that actually supports this navigation data config (not any first one);
- Added virtual method on NavigateSystem "OnNavSystemOverriden" to give a chance to inspect previous NavigationSystem when getting overriden by a nav config override;
- Added some missing NavigationSystem clean up logic.
#tests client/server standalone Creative AI, Phoebe and regular + editor
#cr maxime.mercier
[FYI] mieszko.zielinski, stephen.holmes, yoan.stamant
#ROBOMERGE-OWNER: guillaume.guay
#ROBOMERGE-AUTHOR: guillaume.guay
#ROBOMERGE-SOURCE: CL 7679442 via CL 7679443 via CL 7688466
#ROBOMERGE-BOT: (v383-7686620)
[CL 7688470 by guillaume guay in Main branch]
#rb stephane.delmer
#rnx
#ROBOMERGE-SOURCE: CL 7678434 via CL 7678435 via CL 7683318
#ROBOMERGE-BOT: (v382-7680231)
[CL 7683469 by maxime mercier in Main branch]
Submitting to Main as a follow up to 7308751
#jira none
#rb Yoan.StAmant
#review-7308772 Yoan.StAmant
#okforgitub public
[CL 7309638 by Mieszko Zielinski in Main branch]
#rnx
[at]maxime.mercier
#rb guillaume.guay
#ROBOMERGE-SOURCE: CL 7289773 via CL 7306493
#ROBOMERGE-BOT: (v370-7290619)
[CL 7306546 by yoan stamant in Main branch]
