TheLadiesOgre
Aspiring Codesmith
- Joined
- Jan 16, 2009
- Messages
- 505
interesting stuff here....
Requesting following features
- Thread starter j_mie6
- Start date
j_mie6
Deity
can you fix it so that it doesn't break saves? I presume that it breaks saves that were made before the dlll was introduced....
will this affect worldbuildersaves?
will this affect worldbuildersaves?
Asaf
Sleep Deprived
- Joined
- Mar 22, 2010
- Messages
- 1,326
In the unlikely case you're referring to anything I did - feel free to use it (just don't forget to credit
)Yep.
Theoretically yes. It's not worth the hassle, in my opinion.
I have no idea.
I'm working on the Python but having some problems with the English unhappiness under occupation. Because I'm using CyCity.changeExtraHappiness() to adjust the (un)happiness level of the city on conquest, but the CyCity.getExtraHappiness() method isn't returning what I'm expecting it to. It seems that the game is adjusting the extra happiness on its own, making it hard to predict what to add/subtract. At least it reads zero while the city is under occupation...
Asaf: I guess what I need is CyCity.setExtraHappiness() then...
Asaf: I guess what I need is CyCity.setExtraHappiness() then...
Asaf
Sleep Deprived
- Joined
- Mar 22, 2010
- Messages
- 1,326
I looked at the code and it seems as though it should work.
Can you post your Python code and point at what's not working?
Ok, I think I solved it. It turns out that a conquered city is actually a new CyCity instance. So the extra happiness is reset to zero by default. And the old, now invalid, CyCity instance displays a -1 value (as in None).
This should mean that the code is going to be really simple.
I'll be back once I get all of it working.
This should mean that the code is going to be really simple.
I'll be back once I get all of it working.Finally time for testing! 
I've attached the Python files - unpack the archive in the mods Python folder. Since I suggest you only try my code out at this stage - without any other custom Python interfering - you should back-up your CvEventManager.py and CvGameUtils.py and replace with mine. (This will disable any other Python mod component you currently have running.) Because we can merge the code later - in fact, I'd like to do it once you're satisfied with the powers code.
Just to make sure - these are the powers included:
Its actually your job to find the error(s) - so that i can fix it. (I trust that you have enabled Python exceptions, by the way.)
The most obvious thing creating trouble would be that the civilization short names doesn't match. Because I'm using the names as identifiers. (And yes, it should work with any language setting. Please test to make sure.
)
Also, you need to edit PythonCallbackDefines.xml:
And finally, this is what you edit in order to fine-tune the powers (in Powers.py):
Now I can start on the Rebels mod comp. (edit: first draft done )
I've attached the Python files - unpack the archive in the mods Python folder. Since I suggest you only try my code out at this stage - without any other custom Python interfering - you should back-up your CvEventManager.py and CvGameUtils.py and replace with mine. (This will disable any other Python mod component you currently have running.) Because we can merge the code later - in fact, I'd like to do it once you're satisfied with the powers code.Just to make sure - these are the powers included:
- Carthage: Power of the Sea - +1 movement for all naval units
- Greece: Power of Democracy - Enables Universal Sufferage from the start
- Brittania: Power of Rebellion - Conquored English cities suffer increased unhappiness and are more likely to revolt against foreign rulers
- Pictia: Power of the Highlands - Hills +1 food
- Gaul: Power of the Dun - Cities built on hills get +10% defence
Its actually your job to find the error(s) - so that i can fix it. (I trust that you have enabled Python exceptions, by the way.)The most obvious thing creating trouble would be that the civilization short names doesn't match. Because I'm using the names as identifiers. (And yes, it should work with any language setting. Please test to make sure.
)Also, you need to edit PythonCallbackDefines.xml:
Spoiler :
And finally, this is what you edit in order to fine-tune the powers (in Powers.py):
Spoiler :
(edit: first draft done )I'm working on the Rebels mod-comp and doing some improvements on the code. Like not check all player's cities on every turn, and since there seem to be 10 of them in the scenario this equates to a base 10% chance of any player's disordered city to be hit with the event. The good thing is that the lag caused by the mod-comp is also reduced to 10%.
One thing though: I also have to multiply the chance of rebellion probability by 10, which in turn means that 10% is the highest sensible value to use, because the event is only checked every 10 turns for any given player. Unless some Civs can be excluded from the event altogether.
I suggest we set the probabilities as 5% for civilized Civs and 10% for the uncivilized ones - and for any city founded by the Britons. How does that sound?
One thing though: I also have to multiply the chance of rebellion probability by 10, which in turn means that 10% is the highest sensible value to use, because the event is only checked every 10 turns for any given player. Unless some Civs can be excluded from the event altogether.
I suggest we set the probabilities as 5% for civilized Civs and 10% for the uncivilized ones - and for any city founded by the Britons. How does that sound?
j_mie6
Deity
yer thats fine. I should be able to test the code tonight
j_mie6
Deity
oh sorry didn't see that! yeah sounds fine... I just about to test the code
j_mie6
Deity
ok on first anylisis... Greek works.... Carthage does no longer works.... Pictish does not work... not sure how to find out if gaul works or briton for that matter
I put all of the files in the main python folder after removing all other python apart from my civlopedia python which just edits the name of one of the pedia tabs (BTS concepts -> New Concepts)
I put all of the files in the main python folder after removing all other python apart from my civlopedia python which just edits the name of one of the pedia tabs (BTS concepts -> New Concepts)
j_mie6
Deity
ok I have got my python exception!
Traceback (most recent call last):
File "CvEventInterface", line 23, in OnEvent
File "CvEventManager", line 187, in handleEvent
File "CvEventManager", line 333, in onGameStart
File "Powers", line 40, in pictia
Attribute Error: 'CyPlayer' object has no attribute 'setExtraPlotTypeYield'
Traceback (most recent call last):
File "CvEventInterface", line 23, in OnEvent
File "CvEventManager", line 187, in handleEvent
File "CvEventManager", line 333, in onGameStart
File "Powers", line 40, in pictia
Attribute Error: 'CyPlayer' object has no attribute 'setExtraPlotTypeYield'
No Python exceptions?
Found a Gaul hill city to see if the defense bonus appears? Switch players (alt + z) Conquere a conquere it to see if the bonus disappears.
Conquere a Briton city to see if you get the +2 "Aaarrggh!" unhappiness. (I couldn't find a way to add to the ethnic unhappiness.)
I could actually test your mod also. Where can I download the version you're currently using?
j_mie6
Deity
and where do I put that 
also there is no uploadable version for the one I'm using it's unfiinished but if you want it I can make a little zip for you... (can you unpack 7zip because I might use that just for this...)
also there is no uploadable version for the one I'm using it's unfiinished but if you want it I can make a little zip for you... (can you unpack 7zip because I might use that just for this...)
j_mie6
Deity
what about the rest? it says there is changed C++ sources in there does the whole folder get entered?
j_mie6
Deity
ok btw I have to send 4 emails to you to get the whole mod across lol...
use the first as a basis
second e-mail - take the unit art out of the folder named units and put in the units folder of the first e-mail (it's the other half)
Make a new folder in art named Leaderheads and put in the contents of e-mail 3 and 4 (only the ClovisFINAL folder and markantony or what after not the zip folders names...)
use the first as a basis
second e-mail - take the unit art out of the folder named units and put in the units folder of the first e-mail (it's the other half)
Make a new folder in art named Leaderheads and put in the contents of e-mail 3 and 4 (only the ClovisFINAL folder and markantony or what after not the zip folders names...)
j_mie6
Deity
I tried putting just the dll in the Jamie'sRomeMod1.3/Assets and it crashed the game.... can you post a screenshot of where it should be?
Similar threads
