Project

General

Profile

Feature #857083

Show nation culture (2.6) and gain per turn in client

Added by Jacob Nevins 6 months ago. Updated 6 months ago.

Status:
Closed
Priority:
Normal
Assignee:
Category:
Client
Target version:
Start date:
Due date:
% Done:

0%

Estimated time:

Description

In the augmented2 modpack, city and nation culture really matters. With the 2.6 client, I found myself repeatedly opening the Demographics report to find out my nation's current culture.

Nation culture was added to the info popup in feature #821946, but only from 3.0+ because "client in S2_6 does not know the total amount of player culture".

I think that the evaluation in player_culture(), that uses history from server and evaluates performance effects on the client, would give a good enough result for most sensible rulesets? If not, I think it would be worth adding an optional capability to let new 2.6 servers send the real value to the client (like feature #820730 but without the embassies part), and show that in the client if available.

Also, on all branches, it would be useful to see the expected gain per turn (from history). Currently that has to be evaluated on the client in all branches, and I think these kind of forecasts are probably mostly fine like that, although it's possible to construct a ruleset where the forecasts are misleading due to incomplete client knowledge.

Attached patch uses client-side knowledge (for both on 2.6).

This patch only helps those clients that use get_info_label_text_popup() somewhere (i.e., not the SDL clients). (In the Gtk clients it's available by left-click on the left civilization info pane; in the Qt client it's a tooltip on the left sidebar gov/warming/etc icons.)

(Putting this in the tooltip is a compromise between making it available at all, and taking up valuable screen real estate in rulesets where culture isn't a thing. In future, I wonder if it is worth adding some booleans to the ruleset format, so that rulesets can indicate "culture is important", "plague is important" etc and the client can adapt its display accordingly. Or, probably, semantics of "suppress plague display".)

m-30-nation-culture-gain.patch (1.26 KB) m-30-nation-culture-gain.patch master, S3_0 (add nation history per turn) Jacob Nevins, 2020-01-23 12:11 AM
26-nation-culture-and-gain.patch (1.24 KB) 26-nation-culture-and-gain.patch S2_6 (display national culture and gain/turn) Jacob Nevins, 2020-01-23 12:12 AM

Related issues

Related to Freeciv - Feature #821946: Show player culture in info labelClosed

History

#1 Updated by Jacob Nevins 6 months ago

#3 Updated by Jacob Nevins 6 months ago

  • Status changed from Resolved to Closed

Also available in: Atom PDF