fixed emergency protocol by removing sheduled wares requested before ...

[tost11]

When emergency protocol is activated and many buildings are already planned, the wares already scheduled for delivery are ignoring the protocol.

So now when emergency protocol is activated, those wares get removed (if not for the sawmill or woodcutter).

This also solved the issue for AI enemies on very low resources where they're not able to start because the first sawmill never got built.

[Flamefire]

LGTM,
trivial style issue in the NULL check which I fixed directly. git pull if you need to do further changes

[Flamefire]

You need to run clang-format first. If you have it installed when configuring you can run e.g. make clangFormat
Or use the diff as shown in the CI job.

Can you add a test for that? Might be a bit tricky though

[Spikeone]

Well, this is original behavior - you were also able to store out wares to get them going anway (a tactic used with very low goods frequently). This fix breaks this original behavior, doesn't it?

This sounds reading your description more like a problem the AI has, and should be fixed for the AI instead of for players (maybe the AI can use that system which is slight cheating but I'd opt for a smarter AI instead of an AI playing only by rules)

[Flamefire]

Well, this is original behavior - you were also able to store out wares to get them going anway (a tactic used with very low goods frequently). This fix breaks this original behavior, doesn't it?

What we have:

• When emergency program is active newly built buildings don't get wares except woodcutter or sawmills
• With this: Additionally wares ordered for such "blocked" buildings which are still in a warehouse, i.e. not carried out yet, are stopped, wares already on the way are not.

Not fully sure how the original handled that as it seems like a minor corner case happening rarely.
If in the original wares for other buildings are still carried out, then I'd also say that we guard this behavior for AI only. If it helps then I don't see an issue there as it is such a minimal QoL change for rare cases that it likely won't even be noticed in most cases.

[tost11]

The issue can also occur for the player, like On low resources, start when starting with multiple buildings (but the sawmill first, but on hilly terrain) and then multiple other buildings (fast before all boards are given out). Then all boards are scheduled for building. The sawmill and woodcutter are already planed. Then some time later the Terain Blow sawmill is leveled, and it requests boards. But all boards are already given out to other buildings. That results in no boards for the sawmill.

But yes, it is an edge case, and yes, I mainly built it for the AI to start proper play on some maps where it gets stuck on low-resource starts.

But actually it's an issue in the game, not only an AI issue.

So what now? I see multiple options that could be implemented to not block fast start for players.

• Fix it only on AI, but behavior stays broken on edge cases.
• Check what the behavior was in the original game.
• Improve emergency protocol with:

-- Only hold back needed boards (2 for the sawmill and 2 for the woodcutter) depending on what buildings are missing. Also maybe 1 more if the street gets removed and one is lost by the player or AI.
-- Unlock emergency protocol if the boards reach the building, not when finished (but on the other hand, without those building boards, production isn't possible anyway).
-- Map boards to exactly one needed building; for example, if two woodcutters build a target, one gets the boards. so other boards can be normally used.

What do you prefer?

For me it would be the last option, and I also would implement that more complicated but better emergency program behavior if wanted.

[Flamefire]

But yes, it is an edge case, and yes, I mainly built it for the AI to start proper play on some maps where it gets stuck on low-resource starts.

But actually it's an issue in the game, not only an AI issue.

That depends on the definition of "issue". We want (at least the default) behavior to match that of S2, so if "emergency mode" there meant that only newly built buildings do not get boards/stones then that is (usually) what we want.
AI may "cheat" and that's fine as a player could invoke this behavior by removing roads, so that's a shortcut.

So what now? I see multiple options that could be implemented to not block fast start for players.

* Fix it only on AI, but behavior stays broken on edge cases.

No issue with that in any case

* Check what the behavior was in the original game.

See above: That would be the most important thing to check first although it may be difficult to get into exactly this situation. But it looks like your description above should be reasonably easy to check. Could you do that?

* Improve emergency protocol with:

I think that is too complicated to avoid such a rare issue. In the end this is a resource management game and the emergency mode already a QoL improvement only. So it could be "incomplete" leaving more responsibility to the player.

However as queued wares do not turn up for the player anywhere in the UI it might seem like emergency mode is ignored for a short time. I didn't fully understand @Spikeone

Well, this is original behavior - you were also able to store out wares to get them going anway (a tactic used with very low goods frequently). This fix breaks this original behavior, doesn't it?

What exactly does this break? You build a building, wares for this building are queued, emergency mode activates, wares for building stop being carried out as-if the building was built/connected right after the activation.
If you can "force" delivering wares in emergency mode by out-storing then this isn't blocked by this change. Not sure if it was possible in RttR before, but if it was it still will be.

So I'd lean into accepting this change to enforce the visible semantics: When emergency mode is active no ware will leave the WHs (with the purpose) of going to a "disallowed" building site.
Or what do you thing @Spikeone ?

[tost11]

I have tested the behavior in the Siedler 2 Gold Edition.

It seems to be the cleverest approach. Boards are stored until only 4 are left. If a woodcutter builds it, it gets two boards. Another one would not receive any boards (like it's currently here); only the sawmill gets them (see screenshots).

When both buildings are finished, the protocol is deactivated.

I have also checked the emergency protocol for stones; if you run out of stones, you are doomed—no emergency protocol for it.

[Flamefire]

I verified with the original too in various situations and you are right: They are immediately stopped in warehouses.

We do have a difference though: S2 activates at 4 boards, RTTR at 10.

Anyway, some minor polishing here and it's good to go

[Flamefire]

Looks good!
I updated the branch and added a few small fixes to the new test: Lot's of unused includes and use of a fixture-suite and fixing the typo (moved parts to suite name)

The change I added now does the emergency protocol check in the tests similar to the real game. So that explicit check could be removed.
If I haven't missed anything and CI passes I'll merge, unless you notice anything

[tost11]

nice :D, i also think the issue: #953 can be closed then