userDaniel Pcancel

850 Commits over 243 Days - 0.15cph!

15 Days Ago
Bugfix: EventRecord.AddField(bool) now respects it's param Lucky for us, wasn't used anywhere outside of tests. Tests: none, trivial change
15 Days Ago
Update: adding a couple perf tests for EventRecord - also removed one of profilign scopes since I don't need it anymore Used them to check if packing EventRecordField would give any perf benefit, and it's a no - indistinguishable from noise. Tests: ran the new tests
15 Days Ago
LOD0 and prefabs setup condenser tanks smalls
15 Days Ago
Optim: Allocate scratch buffer on the stack instead of thread local mem Perf tests showed same performance, so we can save on the global allocation Tests: ran editor on craggy
15 Days Ago
Optim: avoid scratch buffer round trip After checking internals, GUID serialization is also alloc-free, so routing through that. Tests: profiled in editor
15 Days Ago
Optim: eliminate float/double related allocs in EventRecordField.Serialize Need to run a couple additional experiments(stackalloc, tagged union), but this part is basically done. Tests: ran in editor, observed in profiler that no more allocs are happening in Serialize for small records
15 Days Ago
Optim: EventRecordField - use thread local scratch to avoid GUID serialization allocs Tests: validated value via debugging, unity profiler showed no allocs during Serialize(CSV) call
15 Days Ago
Update: hacky EventRecord profiling setup to track allocations Will need to discard this before merge Tests: ran in editor
16 Days Ago
Bugfix: Use valid index in WaterTestFromVolumes Tests: detected during staging demo playback with useparallelupdatejobs - reran the demo, no more NREs
16 Days Ago
Merge: from parallel_validatemove - Fixers a couple rare bugs leading to missing data from FullServerDemo recordings - More work on BasePlayer.SErverUpdateParallel, still disabled - Editor-only: Added a couple unit tests - Editor-only: ServerDemoPlayer - disable error spam during demo playback, improve log format - Editor-only: ServerDemoPlayer - automatically authenticate connections during demo playback Tests: played back demo from staging server, recorded a couple new demos in local editor
16 Days Ago
Merge: from main Tests: none, no conflicts
16 Days Ago
Bugfix: Initialize WaterSystem coarse grid on clients - Also make it safe for scenes that don't have a water setup (like playground) Tests: tested with a separate client connecting to server - saw that Client was initializing everything correctly in the right order
16 Days Ago
Bugfix: ensure WaterCollision grid is setup with right terrain dimensions Tests: In editor, started on craggy - size matched. Started on procgen - matched. Played demo - matched. In all cases validated initialization order to confirm no colliders/volumes were added too early. Exported PNG of the grid.
16 Days Ago
Bugfix: FullServerDemos - transient entity recording fixes - Use DemoCount instead of ChunkIndex when determing to send snapshots - ChunkIndices reset, so they can rarely overlap for an entity and cause it to not send a snapshot - Reset counter on prefab pooling - previously it could cause skipping of transient entity snapshotting Tests: recorded a bunch of demos in editor
16 Days Ago
Bugfix: ServerDemoPlayer - skip the auth flow when playing the demo - also index packet logging messages - makes it easier to sort out order of things in editor logs Seems the simpler thing to do for now. Tests: played back staging demo - much less error spam
17 Days Ago
Update: ServerDemoPlayer - add Ready message logging Tests: none, trivial change
17 Days Ago
Update: ServerDemoPlayer - log more message types Tests: ran demo from staging server
17 Days Ago
Update: ServerDemoPlayer - switch to control error reporting Tried the demo from staging, the error spam is too much and not super helpful, so adding a toggle to disable it. Tests: Played back the staging demo
17 Days Ago
Optim: Avoid handling null cases in a batch with only non-null values - Also updated the test to spawn entities, since now it's a requirement for the func. This removes the need to juggle data to setup batch operations. Should save a bit of time. Tests: Ran the updated unit tests
17 Days Ago
▄▇▅▆▇▊: ▇▆▆▊▉▍ ▄▋ ▊▉▆▇▅▉▋▇▉▄▅▋█▉▆ ▋▇▆▍█ ▅▊ ▅▅▋▌▅▅▄▍▄▆▍█▅▆▋█▇▉▆▋ ▌▅▄▊ ▅▌▍▉▆▇▍▊▉▄▇▄ ▉▆▇▊▆: ▉▄▌▌▄▇ ▍▄▌▌ ▇ ▅▇▋▄ ▅▅▊▇ ▅▉▋▊▇▇▌▅▌▋▌▄▅▊▉▆▄▆▋ ▅█▄ █▇ █
17 Days Ago
Merge: from main Tests: none, no conflicts
17 Days Ago
Merge: from profiling_improvements - makes the Linux binaries compatible with more distros (Ubuntu 20.04, Debian 12) Tests: ran in Ubunti 20.04 and 24.04 via WSL. Took snapshots and opened them in perfetto.
17 Days Ago
Update: ServerProfiler now usable on older Linux distros - Binaries built using revision 45d79338 Should work on Debian 12. Tests: Ran on WSL Ubuntu 20.04 and Ubuntu 24.04 and took snapshots - all worked.
17 Days Ago
Merge: from main Tests: none, no conflicts
20 Days Ago
Merge: from profiling_improvements - Fixes ProfilerBinViewer to display all available threads and fix invalid callstack depth calculation - Fixes a bug that would prevent json from being generated on busy servers - Fixes a bug with timelines being very-slightly out of sync - Optim/Bugfix to filter out all constructors from being profiled Tests: a lot of exports in the edittor and a bit of forced "bad" cases
20 Days Ago
Bugfix: ServerProfiler - properly filter out constructors - Due to a typo (missing .) it would never match constructors and never filter them out Using release libs built from ec8c5522 Tests: snapshot in editor on Craggy - confirmed no contructors were recorded.
20 Days Ago
Bugfix: ServerProfiler - sync up non-main-thread timelines to main thread Previously it was possible to have a small gap at the very start of the snapshot on non-main thread views. Tests: exported a snapshot in the editor.
20 Days Ago
Bugfix: ServerProfiler - properly handle sheared callstack at the start of worker threads during export This revealed that I have a bug with non-main thread timing (things are slightly offset). Will fix next. Tests: injected data that was tripping up the export originally. Validated that it does trip up before fix, and now exports correctly after fix. Exported 10 snapshots in the editor - no failures, and all look correct.
20 Days Ago
Bugfix: ProfileBinViewer - fix callstack depth calculation - Also replace RadioBoxGroup for threads with a Dropdown - turns out 50 threads radio buttons don't vertically fit my monitor. Who knew. Hoping it's the same issue in the json exporter. Tests: Opened a debug snapshot from official server.
20 Days Ago
Merge: from main Tests: none, no conflicts
20 Days Ago
Bugfix: CollectionUtil.SortInplace now correctly sorts - Also renamed these utils as previous names were confusing Tests: ran the old + new unit tests - all pass
20 Days Ago
Update: Adding unit tests for CollectionUtil Discovered that my SortInplace is borked, so I'll fix it in the next update. Tests: ran the new unit tests
20 Days Ago
Update: consolidate various scatter code used in batched WaterLevel and GamePhysics checks Will cover them with tests just to be extra safe. Tests: ran WaterLevel and GamePhysics unit tests
20 Days Ago
Update: Name a couple magic constants Tests: none, trivial change
20 Days Ago
Clean: rogue newline Tests: trivial change
20 Days Ago
Update: TerrainCollision.GetIgnore(Span...) now uses the ignore grid broadphase Tests: ran new unit tests that go through this path
20 Days Ago
Update: new unit tests for GamePhysics.HandleIgnoreCollisions - Also removed an unnecessary branch Tests: ran the new unit tests
21 Days Ago
Merge: from main Tests: none, no conflicts
21 Days Ago
▍▌█▇█: ▅▅▇▍ ▋▅▊▄▊█_▌▋▌▊█▌▊▋▌▉▋_▄▅▆▇▆▉▉ ▄▍▅▇▅▌ ▆▇▇ ▆▌▉█▅▊ ▊▋▅▌▌▉ ▍▇▅▊▌▊▅ ▍▅▊█▉: ▅▄▋▇ █▉▍█ ▅▉▇▌▆▄ ▉▇▉ ▉ ▅▇▋▍▄, ▊▍▊ ▇▅▄█ ▅▊▇▍▇▅▍ ▇▆▉▍▇█ ▇▍▆█▋▅ ▌▇▊█ ▅▌ ▅▄▄▇▌▆▌ █▊▇ ▊▉▇▋█▇.
21 Days Ago
█▉▋▅▍▆: ▊▌▌ ▇█▌▇▅▋▅ ▇▆▊▄▋▇▉██▄▅ ▉▍▉▌▇▅▍ ▉▄█ █▊▊ ▊▄▋▋▄▉▌▊ - ▉▉▍█▅ ▇▋▄▄▇ ▋▌ ▅▄▍▇█▅ ▋█▉ ▊▊▌▉▄█▊ ▊▌█▌▆ █▄ ▇▉█ ▅▋▆▉ ▄▅▋▌▉▉▅ █▆ ▌▌█▅▌ ▌▅▍▅▌▄▇▅ ▌▌▊ ▊▄▉▋, ▄▍▍▅▍▉▉ ▄▋ ▉▍ ▉▆▉▉▄▌ ▇▅█▇▉. ▄▍▅▅ ▌▆▌▅▌█ ▉▊▅▍ ▌▆▅▇▇▅ ▅▊▌█▅▊ ▇▌▌▆▍▄█ ▉▌▄▍ ▌▊▆▅▉▊ ▉▍▊▇▊▅▉▆▋▅ ▆▇█▄ ▋▄▆▍▄▇▊█ ▉▅▊▄ ▋▆ ▇▊▅+ ▍▉▍▇. ██▆▋▊: ▆▍▅▍▆▆ █▍▆▋ █▇▅▆ ▄▅██ - ▋▉█ ▆▍▅▍ ▆▋ █▋▆▊▄▆▇▅ ▊▆▉ ▍█▊▌ ▍▉▅▋▌. ▇▍▅ ▄▍█▅ ▆▍ ▍▄▍▍█▄▅ ▄▄ █▅▋ ▄▇▍▍▉▇▍ ▉▇█▄▅█ ▇█▉ ▆▊▉ ▄▅▅▊▆▊. ▍▍▊▊ ▍█ ▋ ▊▅▇▊▆▄▇▅ ▆▊▊▉▄▋ ▉▄▉ ▉▌ █▌▄▉▍▊▊▆▊ ▇▇▍ ▇▍▆ █ ▇▄▌▇ ▅▆▇▋ ▉▉ ▅▅▌▇▍▄▄▅▄▌ ▊▉▇▍▉▌.
21 Days Ago
Merge: from parallel_validatemove * Modifies Full Server Demo recording to grab more data + timestamps per packet * Adds editor-only "DemoServer" server backend that we can use in editor to play back full server demos. Switch editor to ServerMode and put path in GameSetup object * New server-side batch-update of players routine - disabled by default as it's not validated yet. Controlled by `server.UsePlayerUpdateJobs` * Added GamePhysics batch versions of OverlapCapsule and OverlapSphere * Couple unit tests to check Water's batch queries against non-batched versions Tests: A lot of server demos recorded and played back in the editor doing core activities (looting, harvesting, interacting). Made sure standalone server and client builds locally.
21 Days Ago
Merge: from main Tests: none, no conflicts
21 Days Ago
Merge: from main Tests: none
21 Days Ago
Clean: ServerDemo - fix up names of entities that get reported via errors Tests: played back procgen demo
21 Days Ago
Bugfix: ServerDemoPlayer - invert the error reporting logic for DestroyEntity Previously, we considered it an error if we couldn't find an entity that demo stream tried to destroy. But, if the server behaves correctly, then it should be already destroyed - so now we report an error when we do find an entity. Tests: played back the complex demo from procgen - errors still present, but harmless
21 Days Ago
Update: FullServerDemo - wrap transient entity snapshotting in try-catch - Also correctly clean up NetWrite I've checked a bunch of code related to saving, and I'm 100% certain there are landmines that I haven't spotted. This way I've neutered the danger of recording the demo, and I'll be fixing up those cases as we discover them in the wild. Tests: recorded a demo on Craggy
21 Days Ago
Bugfix: FullServerDemo - use save for disk instead of network when snapshotting transient entities - Also couple safety checks for BasePlayer.Save, since now I'm triggering a path that usually isn't triggered for a class of entities (NPCs) Tests: while recording a server demo, on a procgen map in a local server did all basic activities (looting, harvesting, crafting and building) + drove a bike and attacked a ScientistNPC at outpost. Then was able to play back both demo chunks. Tests: on local standalone server and procgen world, went to pier monument and descended to find aggressive NPCs - no more exceptions in server log
22 Days Ago
Update: ServerDemoPlayer - report errors if failing to find entities Need to verify if my previous changes are working in more complex scenarios. Tests: none, trivial change
22 Days Ago
Update: FullServerDemo - detect full network ID range used in demo Replaces old invalid system of just reserving a constant chunk Tests: played back all demos that I have - all valid.
22 Days Ago
Update: FullServerDemo - support client RPCs for transient entities Implemented via an extra parse of demo stream, but should be okay since it's an editor-only process. Tests: Played the same demo where I collected wood - previously it quietly failed, now player got attributed resources.