Opened 8 years ago

Closed 8 years ago

Last modified 8 years ago

#3561 closed defect (fixed)

Summary screen - team totals wrong

Reported by: elexis Owned by: Imarok
Priority: Should Have Milestone: Alpha 21
Component: UI & Simulation Keywords:
Cc: Patch:

Description

If you have match with a lot of trade, you will see that the value of the team barter efficiency can become bigger than 100%. The percent-values are actually added instead of averaged.

For example if there is a team with two players and they both have 50% barter efficiency, then the team value should display 50% too instead of 100%, which is quite misleading.

Attachments (1)

t3561_fix_summary_screen_meh_WIP_v0.1.patch (5.9 KB ) - added by elexis 8 years ago.
Not finished, but addresses most of the issues partially.

Download all attachments as: .zip

Change History (12)

comment:1 by elexis, 8 years ago

Summary: Summary screen - team barter efficiency is summed up instead of averagedSummary screen - team totals wrong

There are actually multiple issues here:

  1. The team percentage values are summed up instead of averaged (not only barter efficiency)
  2. Having infinite kill death ratio is not correct when having lost 0 units. Until we lost a unit, we don't know if the resulting value be close to 0 or close to infinity. It should display "-" instead. The team-total-value needs to take this into account correctly.

Feminization and vegetarian ratio, "barter efficiency" have the same problem. They display "0" instead of infinity.

(We decided to keep the infinity peculiarity)

  1. Map-exploration total is completely wrong (35% + 36% = 56%?) (not wrong as areas explored by both players are not taken into account twice)
  1. Kill-death-ratio team-score is identical to the score of the last player of that team, no sum, no average
Last edited 8 years ago by elexis (previous) (diff)

by elexis, 8 years ago

Not finished, but addresses most of the issues partially.

comment:2 by Stan, 8 years ago

refs #3150

in reply to:  1 comment:3 by leper, 8 years ago

Replying to elexis:

  1. Map-exploration total is completely wrong (35% + 36% = 56%?)

See #3150 as pointed out by Stan.

comment:4 by elexis, 8 years ago

Milestone: Alpha 19Backlog

comment:5 by Imarok, 8 years ago

what is missing in this patch?

comment:6 by Imarok, 8 years ago

Owner: set to Imarok

comment:7 by Imarok, 8 years ago

Cc: jon12@… added

comment:8 by Imarok, 8 years ago

Cc: jon12@… removed

in reply to:  5 comment:9 by Itms, 8 years ago

Replying to Imarok:

what is missing in this patch?

I'm not sure things are missing, it's just that it was a hacky patch that tried to fix issues that weren't issues :p (for instance the exploration total)

The patch is also clearly outdated. If you want to work on the issue I'd suggest to write a brand new patch (you can take some inspiration from elexis' one of course).

comment:10 by Itms, 8 years ago

Resolution: fixed
Status: newclosed

In 18026:

Fixes the handling of teams in the summary screen, along with some cleanup. Patch by Imarok, fixes #3150, #3561.

comment:11 by Itms, 8 years ago

Keywords: simple removed
Milestone: BacklogAlpha 21
Note: See TracTickets for help on using tickets.