search

EcsRx / ecsrx

A reactive take on the ECS pattern for .net game developers
MIT License
520 stars 36 forks source link

Priority is no longer adhered to #11

Closed grofit closed 5 years ago

grofit commented 5 years ago

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.

grofit commented 5 years ago

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.