userDaniel Pcancel

1,658 Commits over 427 Days - 0.16cph!

5 Months Ago
Bugfix: avoid out-of-bounds access when scanning for alloc-only threads Fixes perf snapshot export failing to generate while processing worker threads. There's still a rare case of main thread crashing - investigating Tests: Exported a snapshot in editor
5 Months Ago
Update: ProfileBinViewer can now grok the new profile .bin files Tests: opened bin profile from editor
5 Months Ago
Update: rewrite ServerProfile exporters to work with new data layour - expanded unit test - Core binary should be the same, but copied from the freshest release build - purposefully borked ProfileBinViewer - will fix next Tests: unit tests & took a snapshot on craggy
5 Months Ago
Update: Rewrote how ServerProfiler.Core stores profiling data - got a 2% improvement on 1mil empty call benchmark It will be easier to add new features to the profiler (user scopes, user counters, continuous capture). Need to rewrite the exporters now. Tests: unit tests
5 Months Ago
Tests: perf test to measure ServerProfiler overhead Going to try a different internal storage approach to make future changes easier (and potentially faster) Tests: ran the new tests
5 Months Ago
Merge: from parallel_validatemove - Removes PlayerCache.ValidPlayers allocs Tests: took a snapshot on Craggy in editor
5 Months Ago
Optim: PlayerCache.ValidPlayers no longer allocates garbage Tests: took snapshot on Craggy in editor to confirm
5 Months Ago
Merge: from main Tests: none, no conflicts
5 Months Ago
Merge: from profiling_improvements - Reduces overhead of serializing to/from ProtoBuf by not tracking BufferStream calls Tests: snapshot on Craggy in editor
5 Months Ago
Update: further filtering of methods - Dropping BaseEntity.Is* methods that are just HasFlag wrappers - Dropping new Rust.Data BufferStream and RangeHandle Should reduce overhead on serialization Tests: snapshot on Craggy in editor
5 Months Ago
Merge: from main Tests: none, no conflicts
5 Months Ago
Merge: from relationshipmanager_leaks - Server and Client-side bugfixes for pooling around RelationshipManager types Tests: local 2 player session with flicking open-closed the contacts screen and with pooling tracking
5 Months Ago
Merge: from main Tests: none, no conflicts
5 Months Ago
Bugfix: return ProtoBuf.PlayerRelationships back to pool after client rpc is executed Think it was the only PlayerRelationships leak on client Tests: local session on craggy, opened&closed contacts - saw pool metrics stay stable
5 Months Ago
Update: using new RelationshipManager.ClearRelations to avoid potential pooling leaks Tests: none
5 Months Ago
Bugfix: clear PlayerRelationshipInfo when returning PlayerRelationship to pool Tests: none, since it turns out that we don't have any code that stresses this path - one more thing to fix then
5 Months Ago
Bugfix: return PlayerRelationshipInfo back to pool when forgetting a player 2 more places to fix Tests: none
5 Months Ago
Bugfix: return to pool ProtoBuf.RelationshipManager.PlayerRelationships + nested types after client rpcs There's still one more server leak and a client leak Tests: local 2-player session. Made sure changed code is being stepped through.
5 Months Ago
Merge: from main Tests: none, no conflicts
5 Months Ago
Merge: from reduce_appmarkersellorder_allocs - Reduces pool spillage and misses of ProtoBuf.AppMarker.SellOrder Tests: started server on procgen map
5 Months Ago
Optim: Increase ProtoBuf.AppMarker.SellOrder pool capacity to 2k - Should reduce creation and spillage of SellOrders Tests: booted server on procgen.
5 Months Ago
Merge: from profiling_improvements Tests: snapshot on craggy in editor
5 Months Ago
Merge: from main Tests: none, no conflicts
5 Months Ago
Update: Further removal of about 10% methods from perf snapshot - Built using e70c083b Tests: took a snapshot on craggy
5 Months Ago
Merge: from listcompare_optim - ListHashSet can now be pooled and supports Compare - Optim Networkable.UpdateSubscriptions via ListHashSet Tests: unit tests
5 Months Ago
Merge: from main Tests: none, no conflicts
5 Months Ago
Clean: simplify code via ?. notation Tests: unit tests
5 Months Ago
Optim: Constain List<T>.Compare to only works on List<T> This removes remaining 4 boxing allocations Tests: unit tests
5 Months Ago
Optim: Use ListHashSet<T>.Compare in UpdateSubscriptions and UpdateHighPrioritySubscriptions Tests: none, trivial change
5 Months Ago
Update: Network.Visibility.Provider works on ListHashSet Tests: none, simple changes
5 Months Ago
Update: Facepunch.Pool now supports ListHashSet - gave ListHashSet a default ctor Tests: unit tests
5 Months Ago
Update: ListHashSet has it's own specialized static Compare method - Replciated tests from ListExtensionTests for ListHashSet Tests: ran unit tests
5 Months Ago
Optim: List.Compare now uses pooled hashsets instead of hashing-and-sorting 2x faster. Now, just need to see if I can get rid of these remaining allocs (just 4 per stable run) Tests: unit tests
5 Months Ago
Merge: from profiling_improvements - Reduce capture size by ~19% by filtering out more methods Tests: snapshot on craggy in editor
5 Months Ago
Update: further reduce capture scope by ~19% - Built from c4679e41 Tests: snapshot in editor on Craggy
5 Months Ago
Merge: from main
5 Months Ago
Merge: from parallel_validatemove - Bugfix: GamePhysics.OverlapCapsules no longer skips 0-length-capsule queries - Additional unit tests Tests: unit tests + staging demo playback
5 Months Ago
Tests: add GamePhysics.CheckShhere, CheckCapsule and related consistency tests Contains divergence cases that are constantly warned about - this is to document current behavior Tests: ran unit tests
5 Months Ago
Update: Adding GamePhysics.CheckSpheres Tests: unit tests (next submit)
5 Months Ago
Merge: from main Tests: none, no conflicts
5 Months Ago
Update: Made GamePhysics.CheckCapsules and related consistent with CheckCapsule around sphere queries - Added a defaulted param(true) that controls whether we should scan for sphere-like capsule queries or not - Added a couple comments to clarify things This adds ~1ms on 10k player perf test scenario, but should silence Unity's warnings Tests: unit tests + staging demo playback
5 Months Ago
Update: GamePhysics.OverlapCapsules and OverlapSpheres now check for invalid commands and report errros Tests: unit tests
5 Months Ago
Clean: added a comment expanding the reason for previous test Tests: none, trivial change
5 Months Ago
▉▅▄█▆: ▊▌▋▉█▊▇▇▄▌▋▅▄▄▇▍▆▍▍▉█▊▆▆▅▇▆▉▉▋▆▅ - ▆▌▄▍ ▅▉▌▅ ▇▄▋▅▆▄▉▇▊ ▇-▇▆▅▌▌▄ ▉█▇▅▇▌▌ ▇▉▉▌▅ ▍▇▌▊▄ ▋▅█▍ ▄▄▅▌▍▌▅█▄ ▄█ ▍▄█▊ ▅ ▋▌▉▆█▊▊▅▇ ▇▆▉ ▊▆█▋ █▅▉▇ ▆▆█▍ ▊▌▉ ▉█▅▍ ▍▅ █▆▍ ▊▅ ▉▇▇▍█: ▉▊▉ ▋▌▊ ▋▉▌▌▇
5 Months Ago
Bugfix: ServerDemoPlayer - skip trying to spawn invalid prefabs from the server demo stream Tests: played back server demo
5 Months Ago
Update: make ServerDemoPlayer complient with recent Protobuf changes Tests: played back a staging server demo
5 Months Ago
Merge: from main Tests: none, no conflicts
5 Months Ago
Merge: from loading_entity_leak - Fix zombie ProtoBuf.Entity objects never being returned to the pool after loading a game save Tests: loaded craggy save, editor default procgen save and staging server save
5 Months Ago
Bugfix: don't leak ProtoBuf.Entity during loading of a save Loading a 4.3k staging server save drops ~129k objects (almost all get spilled, but that's okay after a load) Tests: loaded craggy save, loaded default editor procgen save and stagign server procgen save
5 Months Ago
Merge: from fix_treetoolrenderer - Brings back tree rendering on editor start with a scene with trees without triggering database refresh Tests: variosu editor open scenarios, various scene switching scenarios with domain reload