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. All of 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.

  1. Map-exploration total is completely wrong (35% + 36% = 56%?)
  • Notice you have to play a match with locked-teams in order to show up the teams score.
  • Only the market and miscellaneous tab have percent values.
  1. Kill-death-ratio team-score is identical to the score of the last player of that team, no sum, no average
Version 0, edited 8 years ago by elexis (next)

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.