921 Commits over 730 Days - 0.05cph!
merge from indirect_instancing
Mark Indirect Instancing as DeveloperOnly
Fixed NRE in FallbackMemory.SubmitDrawCalls and other problems by using a free space bitmap and keeping fallback indices stable
Fix occasional wrong position of objects which use Animator snapping upon spawn when Instancing is active
Attempt to fix release build crash with Indirect Instancing enabled by avoiding (potentially buggy) NativeBitArray altogether.
merge from indirect_instancing
Advanced on-screen profiling using FrameTimingManager (WIP)
Improve Update() performance by caching is_in_motion state for every entity.
Improve Update() performance by caching is_in_motion state for every entity.
merge from indirect_instancing
* Tuned execution order to avoid waiting for jobs the main thread.
* Split TransformApplyJob into TransformGatherJob that only extracts matrices to reduce any locks we might place on transform data.
* Added a ton more instrumentation to the CrudeProfiler to get more live information about Indirect Instancing, Debug Draw and also some global timings.
* Made certain expensive debug features toggleable.
* The debug overlay can now render “µ”.
merge from indirect_instancing
Some fixes and cleanups for the Indirect Instancing debug overlay
merge from indirect_instancing
Properly cull underground objects when Indirect Instancing is enabled
Fix culling volumes not working with Indirect Instancing by allowing to inject culling decisions from gameplay code. (pre_culled_mask)
merge from indirect_instancing
Fix deep sea island terrain not rendering properly when Indirect Instancing is enabled by not instancing all MeshTerrains for now.
Fix command count mismatch error spam due to destroyed materials.
merge from indirect_instancing (exception fix)
Fix a typo causing an access conflict between DamageTrackerRegionScanJob and TransformApplyJob
merge from indirect_instancing
Fix object popping caused by orphaned motion list entries in multiple ways:
* Remove seq_nums from the motion list just before they are unassigned. This piggy-backs off the existing remove_queue to guarantee container safety when objects are destroyed at inconvienient times. No need for a separate motion_stop_queue.
* Store parent entity for every InstancedLODComponent. When objects are returned to the pool they are unparented first, but they still need to know who their parent was so they can properly remove their seq_nums from that entity.
Bonus: Fix some NREs when Indirect Instancing is disabled in the editor.
merge from indirect_instancing
Keep Indirect Instancing disabled by default (until rare wrong object bug is fixed)
* Improve job scheduling to make leaving the job phase more robust through loading screens
* Fix NRE when trying to remove instances before Indirect Instancing was initialised
* Made switching Indirect Instancing at runtime more robust
* Prevent some other possible exceptions
* Avoid some unnecessary calls to MotionList
* Renamed TransformMemory to MotionList (as it should be)
* Use managed HashSet instead of NativeHashSet because there seems to be no performance benefit to using native at the moment.
* Fixed exception caused by removing elements from the set while iterating
Use negative handles for fallback memory
Remove InstanceHandle and use a simple integer instead
* Added generic NativeMultiList to replace InstanceHandleList (WIP)
* Store Transform.InstanceIDs (and actual nullable Transform references) in PerInstanceMemory, indexed by seq_num
* Simplified MotionList add/removal procedures to HashSet operations
* Cleared up function naming in BaseEntity.Client.Motion.cs
* Consider all entities as a potential motion parent and utilise existing parent/child relations to call MotionStart / MotionStop
* Removed IMotionParent interface because it caused more confusion than anything else
* Correctly dispose of the hash set storing child instances when an entity is destroyed
(WIP) Large refactor: Make BaseEntity responsible for calling StartMotion / StopMotion with (potentially) a single array of numbers to mitigate spikes when large entities set into motion.
* Make instances identifiable by a single never changing integer
* Speed up adding/removing instances by using various look up tables to find instances instead of staged linear searching
Convert seq_num uint → int
Enable Indirect Instancing by default
No longer restrict Indirect Instancing (`indirect_instancing.enabled`) to developer accounts. I'm looking at you, staging players. 👀
Allow Indirect Instancing to be toggled in-game any time (Also correctly applies config value when in editor now)
* Remove the need for HasValidMesh() on LOD components
* Renamed a bunch of things for clarity
Disable Indirect Instancing by default to unblock editor issues (I'll be back)
