fa icon indicating copy to clipboard operation
fa copied to clipboard

Veterancy values are unintuitive

Open Basilisk3 opened this issue 1 year ago • 6 comments

Scenario 1: A T4 artillery destroys a Paragon. The result is that it will not gain the full 250k mass cost of the paragon as veterancy, but only around 83k. This is incorrect, and the unit would need to destroy several Paragons to even gain a single level of veterancy.

Scenario 2: A land T4 destroys a paragon. The UI will now show that the unit has killed 14k mass (in the case of the Galactic Colossus). This results in it being impossible to instantly gain full vet, of a single unit. Balance wise, this makes sense in my opinion.

Nevertheless, the value displayed in the UI is still wrong and misleading. I have noticed that a lot of players use the veterancy UI to learn how much mass a unit has killed. However, as shown here, in some cases this is very inaccurate.

Another example could be: A tech 1 bomber killing a tech 2 mass extractor.

One possible solution would be to implement two separate indicators, one for the veterancy values, and one for how much mass the unit destroyed.

Basilisk3 avatar Mar 27 '23 14:03 Basilisk3

Re 1: Will need to investigate this. Re 2: Units not being able to gain more than 1 vet level per unit kill is intentional but as you said it has some undesirable effects. Perhaps a solution with two indicators or something else would be something that the Game team would be interested in picking up. @Garanas

Tagada14 avatar Jun 06 '23 19:06 Tagada14

@clyfordv or @lL1l1 would you be interested in taking a look at this?

MrRowey avatar Apr 22 '24 08:04 MrRowey

With paragon this happens because it takes damage 3 times from a mavor: 5000 damage from the shell (it only has 5000 HP), 5000 damage from the inner nuke explosion, and 5000 damage from the outer nuke explosion. This means it took 15000 damage and 5000 came from the mavor, so the mavor gets 1/3 of its mass as vet, 83400 as shown in the bug.

lL1l1 avatar Apr 22 '24 08:04 lL1l1

I'll take a swing at it.

clyfordv avatar Apr 22 '24 20:04 clyfordv

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

clyfordv avatar Apr 23 '24 02:04 clyfordv

I think two values by default (Experience and MassValueKilled) is probably a UI hazard, but some way of accessing total mass killed seems very reasonable. The overhead of tracking that for every unit is not nothing, so separate PR here: https://github.com/FAForever/fa/pull/6111

clyfordv avatar Apr 23 '24 11:04 clyfordv