keeperfx icon indicating copy to clipboard operation
keeperfx copied to clipboard

Working creatures can damage allied players.

Open PlebSupporter opened this issue 1 year ago • 4 comments

A creature working in a room in which the room animation involves attacking (training/workshop), then those strikes can damage creatures from another, but allied, faction.

PlebSupporter avatar Nov 26 '24 01:11 PlebSupporter

@PlebSupporter

Allies attacking the same door hit each other. This is a lot easier to test for:

  1. Make some corpses and open a path for enemy allied imps to collect them for their graveyards.
  2. Build a door before they can reach the corpses.

https://github.com/user-attachments/assets/c0ecfe2e-54b4-42b2-99fb-fcd7fdfbeae6

classic-testmap.zip

elemanzer avatar Nov 26 '24 19:11 elemanzer

At this point maybe the question need to be asked because I think all of this is probably by design, KeeperFX didn't add the alliance feature. So the question is, do we really want to change this?

walt253 avatar Nov 26 '24 19:11 walt253

@walt253

I believe so because it is much, much worse on KFX than it is in DK. I had to triple the imp count on my testmap just to make it more prominent on DK and even then the imps on DK seem to readjust so as not to get hit. You can see the dk vs kfx comparison below.

I recall running into this in kfx: post undead keeper and the allied keepers then were able to make unlimited vampires on an already tough level simply by killing off each others' imps.

DK: https://github.com/user-attachments/assets/f8ed8b26-7fd0-4470-9475-6d0f4fd8e37a

KFX: https://github.com/user-attachments/assets/ff37edc0-2d30-4995-b03d-973f0deed5a0

Testmap: classic-x3.zip

elemanzer avatar Nov 26 '24 21:11 elemanzer

I think we do want this fixes yes. Alliances are not really new, but were barely functional in DK1 and we did expand upon it. Making them work better is nice. This issue requires implementing the extra hittypes.

The alliance handling became extra important with roaming heroes in keeper colors, which makes mapmakers combine 2 factions into one.

Loobinex avatar Dec 01 '24 10:12 Loobinex