#2587 closed defect (fixed)
[PATCH] in team result screen: wrong map discovered total percentage
Reported by: | Tom Brewe | Owned by: | Itms |
---|---|---|---|
Priority: | If Time Permits | Milestone: | Alpha 18 |
Component: | UI & Simulation | Keywords: | |
Cc: | Patch: |
Description
When playing in a team the calculated amount of map discovered is being added, resulting in eg: Player 1: 60% Player 2: 70% Total: 130%
Better would be to have a total of 70%. So calculating the max() instead of the sum.
Attachments (1)
Change History (7)
comment:1 by , 10 years ago
comment:2 by , 10 years ago
Milestone: | Alpha 17 → Alpha 18 |
---|---|
Owner: | set to |
I'll push this to a18, but I'd like to take a look at it later, as I recently worked on SoD and FoW.
comment:3 by , 9 years ago
Keywords: | review patch added; GUI removed |
---|---|
Summary: | in team result screen: wrong map discovered total percentage → [PATCH] in team result screen: wrong map discovered total percentage |
Here is a proposal. I modified the way the range manager computes the exploration percentage: before it used a cache, now it computes it all when requested. However this could be bad for performance, so I made sure this is not performed too often: the AI was requesting the extended simulation state in one place, which is not needed if I'm not mistaken.
Now for a silly question: I couldn't find the total value in the summary screen, does it appear only in multiplayer games?
comment:4 by , 9 years ago
Here is a working version, tested in-game, and commented enough to avoid problems. I also added the barter prices to the basic simulation state so that AIs can use it.
by , 9 years ago
Attachment: | team_exploration.patch added |
---|
comment:6 by , 9 years ago
Keywords: | review patch removed |
---|
I'm not sure if 70% is the correct value here, as it could easily be 100% or anything between 100 and 70.