branchrust_reboot/main/indirect_instancingcancel
381 Commits over 396 Days - 0.04cph!
Improve Update() performance by caching is_in_motion state for every entity.
Improve Update() performance by caching is_in_motion state for every entity.
* 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 “µ”.
Some fixes and cleanups for the Indirect Instancing debug overlay
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)
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.
Fix a typo causing an access conflict between DamageTrackerRegionScanJob and TransformApplyJob
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.
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)
merge from indirect_instancing
* Moved fallback bounds transformation to a job
* Removed old fallback culling code
Cleanup: Remove old MotionList code
* Fixed objects in fallback renderer not updating correctly when in motion.
* Added profiler markers for draw call submission of instanced and fallback renderers.
* Fixed streaming_use_mesh_uv_distribution_metrics not actually doing anything.
Fix 2 NREs with Indirect Instancing enabled
* Convert INDIRECT_INSTANCING_ON to a global keyword because it kind of is global
* Make sure INDIRECT_INSTANCING_ON is always defined in the GCINCLUDE section (Fixes shipping containers)
* Allow changing indirect_instancing.enabled at any time in the Editor (FIxes entering play mode with it enabled)
Use LOD quality setting "Object Quality" in Indirect Instancing to more closely resemble the LOD switching behavior of the old system.
Try skipping the extra pass to remove null transforms for an extra 0.2 ms (toggleable via ConVar)
Perform application of the motion list transforms in a job for better performance
Set new _INDIRECT_INSTANCING_ON keyword when Indirect Instancing is enabled
Mark all Indirect Instancing ConVars as DeveloperOnly.
Don't include DeveloperOnly commands in autocomplete.
Merge from main (to pull in mac_foliage_tearing_fix)
* Fix randomly disappearing in-motion objects since switching to the new TransformMemory implementation.
* Fix randomly disappearing stationary and in-motion objects in the deep sea.
* Fix concurrent access exception when in-motion objects are destroyed.
add some more debug info to TransformMemory
