EcsRx/ecsrx

Priority is no longer adhered to

grofit opened this issue · 1 comments

Since the change to SystemExecutor to move handling logic to run off observable groups the priority aspect of things has been regressed.

There are 2 scenarios here which would have historically worked but right now will not:

  1. Systems setup ahead of entities would process the entity based on system type then priority order.
  2. Systems added at runtime would process future entities in priority order.

The first scenario should be easy enough to fix, but the latter scenario seems quite tricky to fix and as this is a niche scenario for now I am happy enough to live with it as it wont affect most consumers and can be looked at again once the entity changes have been run, as they may potentially change this entire process a bit.

The first issue has been fixed in latest release, as the 2nd one is such a niche one I am going to close it for now and potentially revisit in the future.