[modmodmod] The Sword of Islam Reborn

[HNation]

Whoops. Apologies, a little out of it tonight. You definitely need an extra tile to the east, for my file I mean?

Yes, I duplicated the easternmost column so that the map would by 125 tiles instead of 124, because of my goal to extend the map in the south eventually and 100x125 is a nice clean aspect ratio

 

[DrLoboto]

Alright let's try this again

 

[HNation]

Alright let's try this again

Thank you! Now Maps.py is the one that isn't working, it says line 104, but I don't see anything broken with it.

I think you fixed the City Name Manager, though.

 

[DrLoboto]

Thank you! Now Maps.py is the one that isn't working, it says line 104, but I don't see anything broken with it.

I think you fixed the City Name Manager, though.

Post it here and I can take a look. Also try screenshotting the error popups, may help

 

[HNation]

Post it here and I can take a look. Also try screenshotting the error popups, may help

Apparently there's a syntax error

Here are the error popups, transcribed
Traceback (most recent call last)
File "<string>", line 1 in ?
File "<string>", line 52 in load_module
File "CvEventInterface", line 13, in ?
File "<string>", line 52, in load_module
File "CvCustomEventManager", line 22, in ?
File "<string>", line 52, in load_module
File "CvRFCEventHandler, line 22, in ?
File "<string>", line 52, in load_module
File "Dataloader", line 5, in ?
File "<string>", line 35, in load_module
File "<string>", line 13, in _get_code
File "
Maps
", line
104
artStyles = (
Failed to load python module CvEventInterface

 

[DrLoboto]

Apparently there's a syntax error

Here are the error popups, transcribed
Traceback (most recent call last)
File "<string>", line 1 in ?
File "<string>", line 52 in load_module
File "CvEventInterface", line 13, in ?
File "<string>", line 52, in load_module
File "CvCustomEventManager", line 22, in ?
File "<string>", line 52, in load_module
File "CvRFCEventHandler, line 22, in ?
File "<string>", line 52, in load_module
File "Dataloader", line 5, in ?
File "<string>", line 35, in load_module
File "<string>", line 13, in _get_code
File "
Maps
", line
104
artStyles = (
Failed to load python module CvEventInterface

Looks like some " ) " are missing at the end of the 'regions' and 'artstyles' tuples

 

[HNation]

Looks like some " ) " are missing at the end of the 'regions' and 'artstyles' tuples

Thanks! The mod loads without python exceptions now, but it still won't open the map. Previously, I would receive python exceptions while loading the mod. Now, I only get them when I try to open the map in the mod.

 

[HNation]

@DrLoboto did you add any new types of resources? Perhaps one of those is in the map and that's why it won't load

 

[DrLoboto]

@DrLoboto did you add any new types of resources? Perhaps one of those is in the map and that's why it won't load

Nothing in the dll, just the FEATURE_TERRACE in the xml files. I'm kinda stumped here, but the things I'd check are:

-If you're using my 'regions' section in maps.py, that all the new regions have entries in Consts.py and CIV4GameText_MEM_Regions.xml.
-that any 'TeamReveal' or 'Owner' entries in the WBsave don't reference player #'s greater than the amount in this version of SOI. I think after the new civs I added, my independents were in teams 42-45

Not sure what here is actually necessary to load into the map but this is what comes to mind. That python error seems to be referencing revealed map plots.

 

[HNation]

Nothing in the dll, just the FEATURE_TERRACE in the xml files. I'm kinda stumped here, but the things I'd check are:

-If you're using my 'regions' section in maps.py, that all the new regions have entries in Consts.py and CIV4GameText_MEM_Regions.xml.
-that any 'TeamReveal' or 'Owner' entries in the WBsave don't reference player #'s greater than the amount in this version of SOI. I think after the new civs I added, my independents were in teams 42-45

Not sure what here is actually necessary to load into the map but this is what comes to mind. That python error seems to be referencing revealed map plots.

The regions in maps.py is the original one from SoIR just with the northernmost and easternmost lines duplicated so it fits the size of the merged map.
I found some TeamReveal=42 and deleted it, now the map is loading and I can select a civ but as soon as I'm done setting up the game (difficulty, speed, etc...) it crashes. No python errors either.

 

[HNation]

Nothing in the dll, just the FEATURE_TERRACE in the xml files. I'm kinda stumped here, but the things I'd check are:

-If you're using my 'regions' section in maps.py, that all the new regions have entries in Consts.py and CIV4GameText_MEM_Regions.xml.
-that any 'TeamReveal' or 'Owner' entries in the WBsave don't reference player #'s greater than the amount in this version of SOI. I think after the new civs I added, my independents were in teams 42-45

Not sure what here is actually necessary to load into the map but this is what comes to mind. That python error seems to be referencing revealed map plots.

I removed FEATURE_TERRACE, but it still won't load. Does it load in your version? Also, can you please post a screenshot of FEATURE_Terrace?

 

[HNation]

@GregFred187 I finally got the merged map working, thanks to @DrLoboto 's help. I'm not quite sure I merged it right.

DrLoboto, could you please post screenshots of your Crimea, North Caspian Sea, Aral Sea, and all those rivers in Central Asia so I can compare and make sure I didn't mess something up in the merge?

 

[HNation]

The merged map is finally available and working. Besides combining, @GregFred187 and @DrLoboto maps, I also incorporated @Nizou Greek Islands. I also shifted mainland Greece 1S, so it lined up better with Asia Minor, and made a couple of minor terrain changes. Played 25 turns on it and so far, everything spawns in its correct location. The one big change I'd still like to make is adding the eastern coastline of India, there should be some ocean in the southeast corner of the map, not just land there, but that will be more work and I'll do it later. I also need to make the northwest corner of the map more than just empty plains.

I'm thinking that eventually all of DrLoboto's civs could be added, alongside Serbia, Bulgaria, and Hungary. Perhaps Wallachia or some other Romanian state as well as an NPC Poland (since most of Poland is not on the map).

 

[DrLoboto]

The merged map is finally available and working. Besides combining, @GregFred187 and @DrLoboto maps, I also incorporated @Nizou Greek Islands. I also shifted mainland Greece 1S, so it lined up better with Asia Minor, and made a couple of minor terrain changes. Played 25 turns on it and so far, everything spawns in its correct location. The one big change I'd still like to make is adding the eastern coastline of India, there should be some ocean in the southeast corner of the map, not just land there, but that will be more work and I'll do it later. I also need to make the northwest corner of the map more than just empty plains.

I'm thinking that eventually all of DrLoboto's civs could be added, alongside Serbia, Bulgaria, and Hungary. Perhaps Wallachia or some other Romanian state as well as an NPC Poland (since most of Poland is not on the map).
View attachment 690458View attachment 690459

Hey, sorry I've been away. That looks real good, nice work. About India - I think I extended the map to either just before India's eastern coast, or right to it. It looks the way it does because I didn't know what to put in Andhra Pradesh so I walled it off with mountains...

Things can definitely be done in the area but it would likely require a handful of tiles to the east to safely get to the coast, as well as an additional civ or two. Cholas, Kakatiyas, etc.

 

[HNation]

Hey, sorry I've been away. That looks real good, nice work. About India - I think I extended the map to either just before India's eastern coast, or right to it. It looks the way it does because I didn't know what to put in Andhra Pradesh so I walled it off with mountains...

Things can definitely be done in the area but it would likely require a handful of tiles to the east to safely get to the coast, as well as an additional civ or two. Cholas, Kakatiyas, etc.

Thanks! I cross referenced your CityName manager against Google Maps, and it does look like there should be a few tiles of coastline in Eastern India, but I will add those in soon. Obviously, the little bit of Ocean there won't be useful for ships unless the map is extended south to include the tip of India.

 

[HNation]

I've been testing the new map. It's working fine, except for the fact that the new areas are almost completely empty. The new parts of India get settled, and Georgia built a few cities on the Northern Black Sea coast, but other than that, nothing. DrLoboto's settler maps will have to be merged with the currently existing one, and the two sets of city name maps will have to be merged too.

These are the civs DrLoboto added

1. Khazars
2. Oghuz
3. Khanids
4. Kievans
5. Alans
6. Chalukya
7. Kypchaks
8. Khitai
9. Ilkhanate
10. Golden Horde
11. Chagatai

GregFred is planning to add Serbia and Bulgaria

It would also be nice to add Hungary and Wallachia, now that we have pretty much all of the Balkans. Perhaps Poland, too, but most of it would be off the map. Ditto with Muscovy, which also comes with the problem of only appearing late in the timeline.

@DrLoboto would you be willing to share more info about the extra civs you added? Maybe a screenshot of their Civilopedia pages showing their unique units, buildings, UHVs, and spawn dates?

 

[DrLoboto]

I've been testing the new map. It's working fine, except for the fact that the new areas are almost completely empty. The new parts of India get settled, and Georgia built a few cities on the Northern Black Sea coast, but other than that, nothing. DrLoboto's settler maps will have to be merged with the currently existing one, and the two sets of city name maps will have to be merged too.

These are the civs DrLoboto added

1. Khazars
2. Oghuz
3. Khanids
4. Kievans
5. Alans
6. Chalukya
7. Kypchaks
8. Khitai
9. Ilkhanate
10. Golden Horde
11. Chagatai

GregFred is planning to add Serbia and Bulgaria

It would also be nice to add Hungary and Wallachia, now that we have pretty much all of the Balkans. Perhaps Poland, too, but most of it would be off the map. Ditto with Muscovy, which also comes with the problem of only appearing late in the timeline.

@DrLoboto would you be willing to share more info about the extra civs you added? Maybe a screenshot of their Civilopedia pages showing their unique units, buildings, UHVs, and spawn dates?

Ya know what, I'm just gonna upload my version somewhere to make this easier for everyone. Going to try to use this site first and if I can't figure it out, somewhere else.

 

[HNation]

Ya know what, I'm just gonna upload my version somewhere to make this easier for everyone. Going to try to use this site first and if I can't figure it out, somewhere else.

Thanks! Github or Sourceforge are options if Civfanatics doesn't work.

 

[DrLoboto]

Alright folks, to make modding easier for those who wish to integrate any of my personal SOI edits with this mod, I 'm posting a link to my version. It's fully playable on its own but not completely polished and is heavily tailored to my own playstyle and interests. Dawn of Man and Civilopedia text aren't fully done (or localized), my scripting is nowhere near as clean and efficient as I'd like, I had to improvise with a bunch of flags, and there's a bunch of subjective changes that make it more than a little different from the base mod - scripted secessions, collapses, and stability hits, changes to victory conditions, resource yields, and unit stats, better production and resource yields leading to bigger armies and cities, etc.

Again, it is fully playable and fairly balanced (for Epic only), barring one non-fatal python error I'm still figuring out. Hopefully enterprising modders can make use of some of it wrt The Sword of Islam Reborn, and maybe some of you will find it fun on its own, I dunno. Bear in mind that I'm not really certain I want to support or publicly develop this thing thing at the moment.


Sword of Islam Modded

 

[Nizou]

The merged map is finally available and working. Besides combining, @GregFred187 and @DrLoboto maps, I also incorporated @Nizou Greek Islands. I also shifted mainland Greece 1S, so it lined up better with Asia Minor, and made a couple of minor terrain changes. Played 25 turns on it and so far, everything spawns in its correct location. The one big change I'd still like to make is adding the eastern coastline of India, there should be some ocean in the southeast corner of the map, not just land there, but that will be more work and I'll do it later. I also need to make the northwest corner of the map more than just empty plains.

I'm thinking that eventually all of DrLoboto's civs could be added, alongside Serbia, Bulgaria, and Hungary. Perhaps Wallachia or some other Romanian state as well as an NPC Poland (since most of Poland is not on the map).
View attachment 690458View attachment 690459

Nice ! Do you have the new upscaled XMLs for _Maps and _CityNameManager ? So that I can start to work on these