search

reeseschultz / ReeseUnityDemos

Unity packages and demos—emphasizing ECS, jobs and the Burst compiler—by Reese and others.
https://reese.codes
MIT License
516 stars 45 forks source link

[BUG]NavPlanSystem use entityInQueryIndex and nativeThreadIndex together will cause bug #93

Closed VincentSai closed 2 years ago

VincentSai commented 2 years ago

Bug description:

NavPlanSystem use entityInQueryIndex and nativeThreadIndex together will cause bug, https://forum.unity.com/threads/nativethreadindex-and-entityinqueryindex-mixed-up-by-burst-compiler.1049879/

Bug reproduction:

Try Debug.Log entityInQueryIndex and nativeThreadIndex then will see they will be same number

Expected behavior:

entityInQueryIndex should be exactly entity sort key, nativeThreadIndex should be exactly thread index.

Describe alternatives or workarounds you've considered:

https://forum.unity.com/threads/nativethreadindex-and-entityinqueryindex-mixed-up-by-burst-compiler.1049879/ It is DOTS codegen's bug, so just like the post move nativeThreadIndex to last parameter with in.

reeseschultz commented 2 years ago

Closing per #95.