[DISCUSSION][PLAYER] New behaviour for invalid replay assignments

Started by IchoTolot, August 15, 2019, 09:19:34 AM

Previous topic - Next topic

0 Members and 1 Guest are viewing this topic.

Which behaviour do you prefer?

V12.4.1 and earlier - invalid assignments are discarded
2 (50%)
V12.6.0 behaviour - invalid assignments are queued
1 (25%)
I have no preference either way
1 (25%)

Total Members Voted: 4

Simon

Quote from: IchoTolot on August 16, 2019, 07:34:01 AM
Quote from: Minim on August 16, 2019, 06:14:23 AM
lemming is building a long stair and you have 5 builders, assign the builder five times to the lemming quickly and fast forward a couple of times, voila! Your long stair is completed! 8-)
Skill queueing like in Lix is a desired feature that I am totally 100% in for!

Lix queuing will affect physics. NL 12.6 requires one frame of shrugging between two builders; player may only re-assign builder during this shrugging frame. Queued builders will skip this frame. Replays with such queuing will fail on NL 12.6. All NL 12.6 replays will play perfectly in (NL with queuing).

QuoteInvalid skill assignments that were not wanted before and that are now distrupting solving replays.

First hunch is that this should fail fast and noisily. NL should refuse the replay when opening the file. Nastily, such early detection would require a complete physics simulation; also, viewer would get no information about what the problem is, and would have to guess.

Thus, I have no proposal yet.

I still believe that it is correct for NL to disallow two assignments during same physics update.

-- Simon