#1582 closed enhancement (fixed)
[PATCH] Improve default player names.
Reported by: | wraitii | Owned by: | O.Davoodi |
---|---|---|---|
Priority: | Nice to Have | Milestone: | Alpha 12 |
Component: | UI & Simulation | Keywords: | |
Cc: | Patch: |
Description
The game uses the "Player X" template for player names when playing on a random map. While this is not a big letdown, it would be nice to use a system similar to AOE I/II, where the name of the AIs were historical rulers. (additionally, we could show the AI used in the endgame screen next to the player name).
Attachments (10)
Change History (33)
comment:1 by , 12 years ago
Milestone: | Backlog → Alpha 12 |
---|---|
Owner: | set to |
Status: | new → assigned |
comment:2 by , 12 years ago
It seems more logical to put it in the civ's representative .json files. Perhaps AIs could also defines names that would override the civ names if present but that's secondary.
comment:3 by , 12 years ago
Funny, I had the same idea :) I will post a patch later today that should do the trick.
comment:4 by , 12 years ago
I have attached a patch that adjust the player name, but only if it is set to a default (Player <number>). When a HistoricalRulers array is available within the civ JSON file it will take one of those names. Otherwise it sets the player name to the name of the civ. So an example game with 8 players could look like this:
- You
- Xenophon (Athenians)
- Cimon (Athenians)
- Macedonians
- Iberians
- Gauls
- Gauls 2
- Gauls 3
Currently I only added some historical rulers to the Athenians JSON file (taken from a list on WikiPedia). All other civs will default to their civ name, and if more than one player has this civ it adds a number (as with the Gauls above). The first gets no number (if desired I can change this behaviour).
I found that just a name of ruler is difficult, because Athenians, Macedonians and Spartans cannot easily be distinguished from their looks. Unless someone is an expert of classic Greek rulers his or her name is not of much help. For now, I add (Athenians) behind their names. An alternative would be something like "Xenophon of Athens," but it requires the civ name added in this form (Athens is not in the file, only Athenians).
by , 12 years ago
Attachment: | playernames.patch added |
---|
by , 12 years ago
comment:5 by , 12 years ago
Added a patch. Ruler names are in civs.zip file. If it selects an existing ruler, or a player already has the name, it will add roman numbers to the end of it.
comment:6 by , 12 years ago
Keywords: | patch review added |
---|---|
Summary: | Improve default player names. → [PATCH] Improve default player names. |
Can't we just have at least 8 historical names per civ? That way they would be guaranteed to be unique for each game. That seems better than adding arbitrary numbers, since numbers would imply historical figures who may or may not have existed :)
comment:7 by , 12 years ago
Well, there are not enough names for some civs. Also a player can choose a name similar to an AI and that won't be good. In my opinion, it doesn't matter if it isn't historically accurate because we just add a bunch of "names", not "rulers".
comment:8 by , 12 years ago
I'm not sure how we could detect and avoid name similarity or why we should try. Sounds like we need to find more historic names for those civs with less than 8 :P
comment:9 by , 12 years ago
It is almost impossible to find enough names for gauls, britons and iberians.
by , 12 years ago
Attachment: | civ_names.patch added |
---|
Adds a piece of code that adjusts default player names, if possible with names from a list
comment:10 by , 12 years ago
I updated my patch so it works with the civ files of Spahbod, plus a couple of fixes. It should work without issues now.
To avoid further double work: Our patches seem to do mostly the same, except that my code reverts to generic civilisation names if a list of possible AINames is not available (or is shorter than 8 names to avoid numbering; generic civ names are still numbered though, e.g. Hellenes III). What is the desired behaviour / which direction will be taken?
comment:11 by , 12 years ago
It depends on our ability to find more rulers for britons, gauls and iberians. If so, we can pick some names from the list.
comment:12 by , 12 years ago
Gauls: http://en.wikipedia.org/wiki/Catuvolcus http://en.wikipedia.org/wiki/Diviciacus_%28Aedui%29 http://en.wikipedia.org/wiki/Dumnorix http://en.wikipedia.org/wiki/Casticus http://en.wikipedia.org/wiki/Catamantaloedes
Iberians: Edeco "prince of the Edetani" - http://www.perseus.tufts.edu/hopper/text?doc=Perseus%3Atext%3A1999.01.0234%3Abook%3D10%3Achapter%3D34 http://en.wikipedia.org/wiki/Mandonius
There's some additions to sift through, hopefully they aren't duplicates. I didn't look for more Brittonic leaders because brit.json
already had enough, but I think we can easily get 8 names per civ (and hopefully more to add variety). If we want to pick random names uniquely, there is a shuffling technique I used in the random map generator (for random player positions) that should work.
follow-up: 15 comment:14 by , 12 years ago
Do names really have to be rulers? Couldn't we choose princes/whatever was in a position of power if we really lack names?
comment:15 by , 12 years ago
In my opinion, yes we can choose other roles, most names to have survived this long would have been important, powerful people :)
A few more Gallic leaders, these were titled vergobret or uergobretos, like Diviciacus above:
- Liscus
- Valetiacos
- Convictolitavis
- Celtillos
- Sedullos
comment:16 by , 12 years ago
And another Gallic chieftain: http://en.wikipedia.org/wiki/Viridovix
(originally found in A New Classical Dictionary of Greek and Roman Biography, Mythology and Geography)
by , 12 years ago
Attachment: | civs.2.zip added |
---|
by , 12 years ago
Attachment: | playernames.2.patch added |
---|
comment:19 by , 12 years ago
Milestone: | Alpha 12 → Alpha 11 |
---|
As we have postponed the release, i think we can get this into alpha 11
by , 12 years ago
Attachment: | playernames.3.patch added |
---|
comment:20 by , 12 years ago
Milestone: | Alpha 11 → Alpha 12 |
---|
by , 12 years ago
Attachment: | playernamespatch.patch added |
---|
by , 12 years ago
Attachment: | playernamespatch.3.patch added |
---|
comment:23 by , 12 years ago
Keywords: | player name patch review removed |
---|
I wanted to do this in near future. Thank you for creating the ticket :D One thing is: Should we put the names in each civ's representative .json file or in the AI's?