fa icon indicating copy to clipboard operation
fa copied to clipboard

Units in transports are not counted as kills

Open JeroenDeDauw opened this issue 7 years ago • 9 comments

Issue: When killing a transport with a bunch of units in it, only the transport is registered as killed by the unit/player that made the kill.

I noticed this because my map has an "automatic reclaim" feature where players get a fraction of the mass value of the units they kill. This code looks at the "mass killed by player" stat, and this stat is not updated when the units in the transport die.

JeroenDeDauw avatar Oct 16 '18 22:10 JeroenDeDauw

yeah it is intended. An inte would get vet too easily if it get rewarded for the unit in the transport it killed.

shalkya avatar Oct 18 '18 18:10 shalkya

That makes sense. Though only means to not assign the kill the the unit that shot down the transport, or otherwise not have it count towards vet level. Assigning the kills to the player should still be possible no?

JeroenDeDauw avatar Oct 18 '18 19:10 JeroenDeDauw

I only remember that zock made the decision, balance wise that it won't be counted for vet level. i don't know the coding details about how to assign the kill without making unit vet out of it. Some other dev might help you.

shalkya avatar Oct 18 '18 20:10 shalkya

Could balance team discuss it again? I doubt couple of vetted inties or ASFs would kill the balance.

speed2CZ avatar Oct 20 '18 06:10 speed2CZ

Keep in mind that this likely can be fixed even without affecting the balance (though this might be a bit more difficult to do)

JeroenDeDauw avatar Oct 21 '18 02:10 JeroenDeDauw

@Tagada14 If this is still something that doesn't need looking into can you close please :)

MrRowey avatar Nov 09 '21 23:11 MrRowey

This defect has a noticeable impact on the gameplay in Survival Versus. Would be nice to have it fixed.

JeroenDeDauw avatar Nov 10 '21 00:11 JeroenDeDauw

You (Tagada) can put it on milestone 3727 if you want this addressed.

Garanas avatar Nov 10 '21 08:11 Garanas

There is already a cap of getting a maximum of one vet level per 1 unit kill (IIRC) so this wouldn't be a problem. If someone can pick this up it would be a nice QoL change.

Tagada14 avatar Nov 20 '22 21:11 Tagada14

https://github.com/FAForever/fa/pull/6091

  1. Veterancy distributed between multiple units as you would expect.
  2. Actual killcount is attributed to the unit that deals the killing blow (so a submarine could end up with ~150 kills if they get the last hit on a carrier, but that's unrelated to veterancy)

clyfordv avatar Apr 16 '24 21:04 clyfordv