1.6 feedback

[Deliverator]

If you check out the Mountains Back to Service modpack thread, david has raised this issue there. Basically, the peak related tags don't do what we want - it is an issue with the modpack.

 

[keldath]

hey guys, i have some solution to our mountais passable problem,

but i need one of you python experts,

we ned to create a dummy feature, with some invisible art, that will be set, to appear on every peak on the map,
thus,
we can limit the entering of units into that feature,
and like this we can control who can go over a peak and who cant!

i have tested it through the world builder, and it works.

its basically going around the problem and cheat it...

my only problem now 0 is how to get the mapscript to create tge "test" feature i created on each peak.

so, who's gonna help?

 

[davidlallen]

hey,

this wasn't adresed yet, but some vehicles like quad can go over some of the peaks,

i think i know why, peak isnt a regular terran, its placed over a regular terrain , so theres one terrain that it allows the quad to go over and another - that wont allow it to pass.

See proposed solution in this post. I can't find the thread reference quickly, but the last time we discussed this there was some concern that fixing this will have a big negative effect on the game. Suspensors may have to take a long detour to reach a city for attacking.

This is in the issue spreadsheet and I hope to fix it someday, but it gets pushed down due to the time needed to properly test.

 

[keldath]

yehah saw that,

but how about my solution? it might be easier and saver ,

can you tell me how can i make a feature to appear on the map?

im really sure that what i proposed here can serve us for the time being,
dont you think?

 

[davidlallen]

That is a good idea. To implement it, you have to add a feature, which is an invisible feature like wind trap water. Then you have to write new python code in both of the mapscript files, which loops over all the plots and adds the feature if the plot is a peak.

Adding the flag into the sdk involves less code and is probably a better solution.

 

[keldath]

yeah sdk is mostly better,

i tried to add the feature to the mapgenerator - i added a line:

self.featureTest = self.gc.getInfoTypeForString("FEATURE_Test")

and the xml section, but the test (gave it an rock arch art for now) didnt apeared on the whole map.

is there anywhere else i need to add a line in the python?

 

[Deliverator]

It seems messy to add a dummy feature. Why not wait for a proper solution, particularly since we are not sure whether this change is desirable for gameplay as david says?

 

[keldath]

ok,
but i think we shouldnt delay dealing with this, its a major flaw in game play strategy in my opinion.

have you got the new 1.6.2?

 

[davidlallen]

@ deliverator, I am afraid that many changes are starting to "single thread" through me. If it is possible for others to also fix the bugs or make and test experimental changes, it is better overall.

@ keldath, you can try this change in publicmaps/archipelago.py, but I would have to study arrakis.py to find the corresponding location. Add the three red lines:

def addFeatures():
	[...]
	peak = gc.getInfoTypeForString("TERRAIN_PEAK")
	[COLOR="red"]peakfeat = gc.getInfoTypeForString("FEATURE_Test")[/COLOR]
	for ix in range(xmax):
		for iy in range(ymax):
			pPlot = map.plot(ix, iy)
			[...]
			# Make all Graben terrain into Sink/Graben unless Coastal in which case make it Mesa
			elif pPlot.getTerrainType() == plains:
				pPlot.setPlotType(PlotTypes.PLOT_HILLS, false, false)
			[COLOR="Red"]if pPlot.isPeak():
				pPlot.setFeatureType(peakfeat, 0)[/COLOR]

Make sure that the tabs in the lines you add stay as tabs. Python is very fussy about white space.

 

[Gisz]

To emphasize David's point; did you install another patch after saving the game?

No. I don't change anything. I have: Vista Home Prem. Edit. CIV4:BtS 3.19 with Dune Wars 1.6.1.

Problem is: I can not start a game at all. During loading a mod (not a saved game) game is crash when load XML.

 

[Deliverator]

@Gisz: Try right clicking and selecting Run as Administrator. Multiple Vista users have said the mod only works this way.

 

[davidlallen]

This vista problem bothers me. I have posted on this thread to learn more.

 

[keldath]

HEY DAVID!

AWESOME thanks a lot!

well it 95% works, some mountains are still left without the feature, but most have it:

Spoiler :

any ideas?

 

[davidlallen]

This section may also need the line "setFeatureType" added:

			# Replace Hills with Peaks (to make Peaks=Mesa more prevalent)
			elif pPlot.isHills():			
				pPlot.setPlotType(PlotTypes.PLOT_PEAK, false, false)	
				pPlot.setTerrainType(peak, false, false)

Please do test carefully to see if the AI is still able to reach cities to attack them. We may decide this hurts gameplay too much.

 

[keldath]

you mean this?

if pPlot.isPeak():
pPlot.setFeatureType(peakfeat, 0)

and sure ill make the proper tests

 

[Ahriman]

Please do test carefully to see if the AI is still able to reach cities to attack them. We may decide this hurts gameplay too much.

Agreed.
Note that we can probably capture many of the desired features if we can give a strength penalty adjustment on mesa tiles.

 

[keldath]

hey,
added:

if pPlot.isPeak():
pPlot.setFeatureType(peakfeat, 0)

again beneath the section :

# Replace Hills with Peaks (to make Peaks=Mesa more prevalent)
elif pPlot.isHills():
pPlot.setPlotType(PlotTypes.PLOT_PEAK, false, false)
pPlot.setTerrainType(peak, false, false)


it seems that mostly the remaining peaks without the feature are the ones on rugged terrain,

 

[Deliverator]

it seems that mostly the remaining peaks without the feature are the ones on rugged terrain,

This could be related to another issue with Archipelago.py - you get Rugged/Sink terrain which should not be a possible combination since there is a code block which should replace it.

 

[Deliverator]

This vista problem bothers me. I have posted on this thread to learn more.

The guy who had Vista 64 and had to give up said he could play FFH2 no problem.

 

[davidlallen]

you mean this?

if pPlot.isPeak():
pPlot.setFeatureType(peakfeat, 0)

and sure ill make the proper tests

No, just the one line pPlot.setFeatureType(peakfeat,0)