Bug Reports

Well, I need to get Revolution to use the improved player initialization code I came up to solve this problem in Better BTS AI (it seemed to be a broader issue, just more visible in Revolution because of the many civs spawned in game) ... once that's done, then yes I think it will solve the problem you've been seeing.

I'll post here again when I've got code for you, it should only be a few days.
 
In the late game, I was getting this error about every five turns:

CvEventInterface line30 onEvent
CvCustomEventManager line 149 handleEvent
CvCustomEventManager line 160 handle Default Event
RevEvents line 121 onEndGameTurn
RevEvents line 1111 checkForAssimilation

PyPopup not defined
 
I have an easy fix for that: Whichever file's at fault needs PyPopup imported and doesn't currently; when I get home, I can post the edited file if you want.

Awesome!

I felt sheepish posting this at all, since jdog has said that he intends to move away from popups as the vehicles for revolution messages, so it's really not pressing. I always like everything to run as smoothly as possible for my own mod, though, and those python errors stick in my craw somethin' awful ...
 
Ehehe...woops, forgot to post last night. *^.^* Well, if you don't wanna wait for my copy of it, all you need do is add the line "import Popup as PyPopup to the beginning section of either RevEvents or CvEventHandler, I think RevEvents.

I didn't post on it myself because I thought it was just happening because I was using an old game! :lol:
 
Alright, I mentioned this a while ago, but I figured I'd post it here since it is probably something Revolutions (more specifically barb-civ) enabled.

The Americans formed in Washington, and got a big horde of units. They moved it a few tiles into my territory, then moved north one tile, and then just kind of hung out there for a very long time (they're still there 30 turns after I took this screenshot).

frozenstackqc8.jpg


I've been experimenting with giving the military barb civs large conglomerates of units, and it doesn't seem to be working as well as it could. They usually keep their units patrolling around their cities, or their units get "frozen" in enemy territory (although, to be fair, I have no idea if this is happening in AI territory too, I've just seen it when being invaded in my own territory by a barb civ).

I'm using RevDCM with a few small xml changes, so it is possible that it's a bug with that.
 
User reports Python exceptions in these portions of the code:

Revolution.py:
Code:
                        else :
                            if( game.getSorenRandNum(100,'odds') > 50 ) :
                                pRevTeam.meet(k,False)
                                kPlayer = gc.getPlayer( kTeam.getLeaderID() )
                                if( kPlayer.AI_getAttitude(pPlayer.getID()) == AttitudeTypes.ATTITUDE_FRIENDLY ) :
                                    kPlayer.AI_changeAttitudeExtra( pRevPlayer.getID(), -2 )

BarbarianCiv.py
Code:
        monumentClass = CvUtil.findInfoTypeNum(gc.getBuildingClassInfo,gc.getNumBuildingClassInfos(),RevDefs.sXMLMonument )

There was a third python exception, but I introduced it with my mod's changes (and have fixed it). These are unrelated to anything I have done.
Neither of these caused any problems with gameplay, just the short pop up message. I have asked the user for a save of the game prior to these exceptions being thrown.
 
Exceptions.jpg


Got this when testing BST. I can't see how it could be related to anything I've done, pretty sure it's RevDCM intrinsic. After getting it, I copied over all my changed python with default RevDCM code, and still got it on the same turn, this exception is reproduceable. Let me know if you want a save, you'll need the lastest BST alpha build to load the save though. Also I can just click the OK button, it doesn't crash or hang or anything. Even though it doesn't break the game, I figure you'd want to track it down and figure out the cause.

Edit: It was really bothering me, because the exception popped up every turn after it first showed up. Noticed this occured after Hyuna Capac went Free Religion, I deleted his 2 inquisitors he had stockpiled, and the exception went away. Hope that helps.

Also I've gotten the previously reported:
monumentClass = CvUtil.findInfoTypeNum(gc.getBuildingClassInfo,gc.getNumBuildingClassInfos(),RevDefs.sXMLMonument )

python exception from that line a couple of times now.
 
Exceptions-1.jpg


Started showing up after Hirohito and Kim Jong Il made a permanent alliance. This is from the latest BST alpha build. Other then the aformentioned changes to City Distance, and taxes (both of wich are available in this forum), no python has been changed. So I highly doubt it's related. I'm pretty sure this is related to a revolution happening in a permanent alliance (since this exception started popping up after a permanent alliance was formed).
 
@Phungus
Thanks for the reports and printout Phungus. Are you sure you are using the latest build? Those errors were definetly in 1.00 but got fixed since then. The line numbers don't match up and it does not make sense this error. The line number points to the correct error problem for version 1.00 but not 1.01 any more. However I will keep an eye out for it.
Cheers.
 
Oh, I was under the impression the released python and source file you released patched 1.00 to 1.01 I haven't done a full install of 1.01, because I added those fixes you released. Anyway the Revolution.py line # wouldn't line up, because I've re written city distance and taxes, should be close to the mark though, and I figured since it tells you the call that's screwing up in in the exception, that would be good enough, I can post the actual code from the line if you want.

I'll see about upgrading to 1.01, it's kind of a pain though.... since I need to remerge. Any chance you could tell me what files (source and python) that changed from 1.00 to 1.01?
 
No probs Phungus. The files changed since 1.00 (excluding the SDK, readme's and maps) are:
autologEventManager.py (to get rid of autolog errors)
CvUtil.py (to get rid of inquisitor error above)
CvDiplomacy.py, CIV4DiplomacyInfos.xml, CIV4DiplomacyText.xml (new diplomacy interface option)
ChangePlayer.py (cheat mode change you requested)
BarbarianCiv.py, Revolution.py (Revolutions improvements)
GlobalDefinesAlt.xml (new entry or change to entry)
Cheers.
 
Thanks, finaly got around to doing this. One problem though, none of these files exist in RevolutionDCM 1.01, or at least they are very well hidden.
CvDiplomacy.py, CIV4DiplomacyInfos.xml, CIV4DiplomacyText.xml (new diplomacy interface option)

Edit:
also:
No probs Phungus. The files changed since 1.00 (excluding the SDK, readme's and maps)
Do you know what SDK files were changed, makes compiling a new core alot simpler if I only have to take into account one or two altered .cpp files, rather then dumping a whole new RevDCM source code list into my working dll.
 
Back
Top Bottom