This makes Direction no longer follow a cartesian trig circle but oh well. Also helper methods to work with this.
Math tests currently fail, pushing this so Acruid can see about fixing SpriteComponent.
* Added documentation to Clyde on the sprite rendering calls.
* Added a rotation debug entity.
* Non-directional RSIs and raw textures are now rotated properly.
* Directional RSIs and Sprite Smoothing work.
* Remove the Directional flag usages.
* Supports layers with different numbers of directions.
* Fixes window rendering.
* Moved IPauseManager from server to shared.
* Moved ITimerManager from Timers to Timing.
* Added missing IConsoleHost to server/client RegisterIoC. Tests work again.
* Add SerializedType attribute to serialize types without the !type tag
* Fix nulls in tests
* Fix null in tests maybe
* Return to type tags
* Fix imports
* Removed the Interfaces folder.
* All objects inside the GameObjects subfolders are now in the GameObjects namespace.
* Added a Resharper DotSettings file to mark the GameObjects subfolders as not providing namespaces.
* Simplified Robust.client.Graphics namespace.
* Automated remove redundant using statements.
Lidgren NetTime.Now is now synchronized to IGameTiming.RealTime.
NetChannel is now a nested class of NetManager.
Add IGameTiming.ServerTime, INetChannel.RemoteTimeOffset, INetChannel.RemoteTime
* Renamed shared ICommand to IConsoleCommand.
* Lots of refactoring into a shared context.
* Removed ICommonSession from server concmd Execute.
* Added argStr parameter to concmd execute.
* The execute function of client concmds now returns void, use the new shell.RemoteExecuteCommand function to forward commands.
# Conflicts:
# Robust.Client/Console/Commands/Debug.cs
* Finally move shells and commands into shared.
* Console commands can now be registered directly without a class in a shared context.
* Pulled up ConsoleHost and Console shell into a shared context.
* Pulled up half the functions of ConsoleHost into a shared context.
* Repair rebase damage.
* Make LoadConsoleCommands function not remove any previously registered commands.
1. Set timeBeginPeriod(3) on the server to reduce scheduler latency in the lidgren thread.
2. Add 16ms of guaranteed lag bias to client prediction calculations to account for scheduler latency.
Both of these changes are to account for how the windows scheduler seems to handle time periods in related to socket polls. See this Discord conversation for why, details down below as well: https://discord.com/channels/310555209753690112/770682801607278632/798309250291204107
Basically Windows has this thing called time periods which determines the precision of sleep operations and such. By default it's like 16ms so a sleep will only be accurate to within 16ms.
Problem: Lidgren polls the socket with a timeout of 1ms.
The way Windows seems to handle this is that:
1. if a message comes into the socket, the poll immediately ends and Lidgren can handle it.
2. If nothing comes in, it takes the whole 16ms time period to actually process stuff.
Oh yeah, and Lidgren's thread needs to keep pumping at a steady rate or else it *won't flush its send queue*. On Windows it seems to normally pump at 65/125 Hz. On Linux it goes like 950 Hz as intended.
Now, the worst part is that (1) causes Lidgren's latency calculation to always read 0 (over localhost) instead of the 30~ms it SHOULD BE (assuming client and server localhost).
That 30ms of unaccounted delay worst caseis enough to cause prediction undershoot and have messages arrive too late. Yikes.
So, to fix this...
On the server we just decrease the tick period and call it a day. Screw your battery life players don't have local servers running anyways.
On the client we bias the prediction calculations to account for this "unmeasurable" lag.
Of course, all this can be configured via CVars.
* Defer MoveEvent out of TransformComponent.HandleComponentState
* Imports
* Make the update loop more readable and call ToArray
* Fix tests
* Fix tests HALLELUJAH
It's now parallelized which cuts off ~200ms on its own for me.
Config is now shared between multiple loads which saves a lot as well.
All in all, pretty good.
Fixes#1434
This means that adding support for new architectures (e.g. ARM) is MUCH easier.
It removes download_natives.py which simplifies the build process.
It's also way less painful to maintain.
* Numerics helpers
* Comments
* Reuse naive methods for remainders.
* Do naive operation if array length is under 4.
* NumericsHelpers takes in Span, can store elsewhere
* Make some spans read-only
* Enabled static property to disable/enable hardware accelerated paths.
* AVX support.
* welp
* .
* environment variable to disable simd
* Adds ARM support (AdvSimd)
* Fix AVX horizontal sum
* Add unit tests
* NET 5.0 moment
* RemoteExecutor moment
* Add nuget.config to the UnitTesting project only
* Add "ROBUST" prefix to env vars
* Better naming for array length utils
* Revert "Add nuget.config to the UnitTesting project only"
This reverts commit 1d474ff33d.
* Ignore ARM tests, remove #if NET5_0
Pieter-Jan Briers