Bug Reports and Technical Issues
- Thread starter Leoreth
- Start date
merijn_v1
Black Belt
The same problem occurs for me as well. I investigated it a bit and I think the following is happening. First of all, I think it is just a graphical bug. The road is indeed present. Opening the WB confirms this by making the roads visible. Exiting the WB makes them invisible again.
If you remove the road and placing it again, the road becomes visible. Setting plot.setRevealed() also makes the road visible. I investigated these functions and I noticed the plot.setRevealRouteType() function. Based on the name of the function, it could very possibly be connected to the error.
When the road is built in Resources.py, the plot.setRouteType(eTeam, eRoute) executes the plot.setRevealRouteType() function for all living teams. (Notice that it is a team parameter and not a player parameter) The Mongols are not yet alive, so the revealed route type is not updated for them, leaving it at the previous route type, which is usually NO_ROUTE.
Code:
@handler("BeginGameTurn")
def createRoutesBeforeSpawn(iGameTurn):
for iCiv in dSpawnRoutes:
if iGameTurn == year(dBirth[iCiv]) - 1 and data.isCivEnabled(iCiv):
for tile in dSpawnRoutes[iCiv]:
plot(tile).setRouteType(iRouteRoad)When a civ spawns, the code below is executed. plot.setRevealed() does also include a trigger of plot.setRevealedRouteType(). I expected that this would update the revealed route type, but it doesn't. However, as you can see in the code the initBirth is triggered during a range of turns. The first is the turn 2 turns before the proper spawn date and 1 turn before the road is build. Maybe it doesn't update the revealed route type because it is triggered before the road is build. The turn later the road is build (but the revealed route type is not updated for the Mongols as they aren't alive yet). However, I would expect that it would update during the next turns, as the initBirth should also trigger then. (I haven't seen something that prevents initBirth from executing when the civ has just spawned, so I expect it to execute during all turns in the range)
Code:
@handler("BeginGameTurn")
def checkBirths():
for iLoopPlayer in players.major().where(lambda p: dBirth[p] > scenarioStartYear()):
if year(dBirth[iLoopPlayer]) - turns(2) <= year() <= year(dBirth[iLoopPlayer]) + turns(6):
rnf.initBirth(dBirth[iLoopPlayer], iLoopPlayer)
[...]
def initBirth(self, iBirthYear, iPlayer):
[....]
# Leoreth: reveal all normal plots on spawn
for plot in plots.normal(iCiv):
plot.setRevealed(team(iPlayer).getID(), True, False, -1)A very simple fix for this is by placing the road not 1 but 2 turns before spawn. This way it is guaranteed that the road is build before plot.setRevealed() in initBirth() is triggered. I tested it and it works.
During this investigation I noticed that he plot.setRevealed() function uses iPlayer as the first argument, while it should be the teamID. I made a PR with this fix.
merijn_v1
Black Belt
I must have missed that part of the changelog.
I just checked it and it seems partially workiing. At spawn the roads are still invisible. When I build Karakorum, all roads outside the BFS do become visible, but the ones in the BFC are still invisible.
I just checked it and it seems partially workiing. At spawn the roads are still invisible. When I build Karakorum, all roads outside the BFS do become visible, but the ones in the BFC are still invisible.
merijn_v1
Black Belt
That is currently not the case. Before building a city the roads are NOT visible. After building the city only SOME (the ones outside the BFC) become visible.
The problem always is with the ones directly surrounding the capital tile. Before the capital is founded, the other tiles are not in your vision anyway (unless your capital is on a hill, but that is not the case for Mongolia).
Hickman888
Prince
This popup appeared when I built the Trading Company as the Spanish (Marathon speed).
Spoiler Python Exception :
Mosguito
Chieftain
- Joined
- Jul 21, 2020
- Messages
- 11
I have experimented on it recently and found the canadian can successfully play the other music into which I change on the XML, which means this problem may stem from the MP3 itself?
merijn_v1
Black Belt
You're right about San Martin using Peron diplo music. I made a PR with a fix.
Here is why. The name in the XML key (AS2D_DIPLO_SAN_MARTIN_INTRO) doesn't tell which audio file should be used. Instead, it points to an XML entry in Audio2DScripts.xml. (The one in the spoiler below.) It has the name San Martin in the tag because that makes it easy to see what it is used for. But it could be any name, as long as the key in both LeaderHeadInfos and Audio2DScripts are the same.
Spoiler :
The XML entry in the spoiler above also doesn't point to the audio file, but only has some settings. In it, you see <SoundID>DIPLO_SAN_MARTIN_INTRO</SoundID>. This tag points to the another XML entry, this time in audioDefines.xml. If we search for that we find what is in the spoiler below. Here is the path to the audio file defined. As you can see, the path of San Martin leads to the Peron audio file.
Spoiler :
This is why the game plays Peron's audio, while all keys in the XML are correctly called SOMETHING_SAN_MARTIN. The there is an error in the path to the audio file. As I mentioned, I made a PR (pull request) that fixes this.
The entry of Tokugawa is in fact the entry of Oda Nobunaga. The names in the XML keys can sometimes be a bit misleading. LEADER_JIMMU is actually Kammu, although that isn't obvious from the name in the keys.
merijn_v1
Black Belt
With the "replace in files" option of notepad++ that is a very quick task. I have one small question about one of the Indian leaders. Should it be Shuhaji (current name) or Shivaji (current XML keys and pedia)?
Mosguito
Chieftain
- Joined
- Jul 21, 2020
- Messages
- 11
Wow...that's a lot in my favor. Thanks a lot. I think I probably know the trick. When I searched for "Bumin" "Alparslan" or "canada" in the XML above I got none. Incredible instruction for me.
Dominus the Mentat
Prince
- Joined
- Oct 29, 2018
- Messages
- 301
This python exception has show up right after Protestantism was founded.
This seems to work fine. I end the turn in your save and plains farms are down to 2 food.2.) Progressing to the Renaissance era as the Moors doesn't take away the "+1on food improvements on plains tiles".
Hickman888
Prince
The hover icon shows the "correct" amount of food. But when you turn on tile yields, or check out how much food/turn the city is pulling in, it shows the bugged yield.
Spoiler Food Bug :
merijn_v1
Black Belt
The problem is that normally entering a new era does not change the yield, so they are not updated and therefor the yield symbols are also not updated. I made a PR that updates the yields (and therefor also the symbols) when you enter a new era, which solves this.
Last edited:
Sorry for checking this so late, but I am having no issues ending the turn. You win the UHV and the game ends as expected. Do you still want the turn after that?
I looked into this and it is technically working as intended, but this helped me discover a number of issues. What happened here is that in both cases, Poland and Italy collapsed and Holy Rome/Austria had a claim on their cities. In the first situation, they had the best claim to get Gdansk and Krakow, which brought them back. They then received Vienna and Budapest because they were independent at the time. Same situation with Italy, they had the best claim on Venica and could then also take Vienna and Budapest on top.Hello I believe I am using the latest update but I want to inquire if this is bugged or working as intended.
In this latest game, twice happening civs are "reborning" in someone else's faction core, while they are still alive. (or at moment of collapse, IDK)
The two examples I have:
1. Poland as a vassal of mine had low stability, and the HRE who was dead "reborn" into krakow as their capital. This killed the poland. (it had danzig and krakow as its only cities, and they both went HRE)
2. HRE who was dead, reborns into Italy civilization, with their capital in venice. This killed the italy. I think italy had low stability at the time, but I am not sure.
They just seem like very weird ways to reborn.
(also, in a byzantime game I went to like 1700s, and with the stability pointer being upward (so I had good stability), ottomans all of a sudden reborn itself in constantinople and most of its cities after I had already defeated them)
Should I try to get a save game for this second HRE reborn thing? or is it working as intended
The image is the turn hre reborn into italy
Spoiler :
I see multiple issues with this:
- Poland should not be in the Holy Roman respawn area
- Venice should not be in the Austrian respawn area
- in the Poland case, I don't think the initial respawn (not considering extra cities awarded later) should succeed if it has no own core cities and only core cities of the collapsing civ (this also applies to the Italy case)
- in the Italy case, I also don't think a respawn should succeed when it only has one city (unless the full respawn area has only one city)
I corrected these issues in 1.16.2, but they only further complicated the "respawn from collapse" code. The problem here is also in part that these limitations were already in place for "normal" respawns, but the code handling collapses wasn't using it. So on develop I will probably reimplement the respawn from collapse code to share more logic with the normal respawn code, including these new rules.
