PB 94 - Schaffen wir PAE?

[xist10]

LotC hat meine ich serverweit ? Wenn nicht, auf jeden Fall aktuell in jedem PB von uns.

Und Pie wollte ich nicht dazu nötigen, einen PB Spy Account anzulegen.

[Markus1978]

Kann ich die PB_PAE_7_final Version aus Post-606 herunterladen und installieren oder soll ich damit noch warten?

[xist10]

Grundsätzlich ja, das sind (erstmal) alle Pythonänderungen.

Es wird aber wohl noch eine weitere Version geben, da Zulan gerade Probleme hat, den Mod zum Laufen zu bringen. Scheinbar macht Linuxprobleme ? Eine Fehlerquelle wurde schon behoben, aber ich will heute für mich nochmal Civ auf Linux installieren und selber testen.

[Markus1978]

Grundsätzlich ja, das sind (erstmal) alle Pythonänderungen.

Es wird aber wohl noch eine weitere Version geben, da Zulan gerade Probleme hat, den Mod zum Laufen zu bringen. Scheinbar macht Linuxprobleme ? Eine Fehlerquelle wurde schon behoben, aber ich will heute für mich nochmal Civ auf Linux installieren und selber testen.

Danke für die Info werde mit der Installation dann noch warten bis die endgültige spielbare Final-Version für unser PAE-PB fertig ist.

[xist10]

Wie hier schon angesprochen funktioniert PB_PAE_7 im Moment nicht auf Linux. In meinen Augen liegt es an der neuen .dll.
Auf jeden Fall konnten wir den Fehler bisher nicht beheben.

Ich würde jetzt die neusten Änderungen in PB_PAE_6.17 einarbeiten und dann mit PAE VI das PB Starten wollen.
Das würde auch heißen, dass brettschmitt leider nochmal ein Startsave erstellen muss.

[Rob Anybody]

@Xist10:
Ist der Mod nicht im falschen Thread gelandet?

@alle
Ihr findet die neueste Version ihn dem Post https://www.civforum.de/showthread.p...=1#post9356413

[xist10]

Stimmt, sorry.

Hier nochmal der Direktlink zum Download.

[Rob Anybody]

Ich habe keine Fehler gefunden. Hat Zulan schon den neuen Link, um diesen Mod auf seinem Server zu testen?

Oder übernimt das wieder Ramk, jetzt wo er zurück ist?

[xist10]

Zulan hat den Link.
Keine Ahnung wann Zulan die Zeit hat.
Und wir brauchen ja noch ein neues Startsave.

Edit: Zulan hat sich gerade gemeldet, zum Testen wäre leider das Startsave nötig.

[Rob Anybody]

OK, ich schreibe brettschmitt eine PN


@brettschmitt
Bitte verwende diese .ini
Erklärung steht in der PN

[Pie]

Jetzt sind alle 4 Saves OK

Ich will jetzt kein Spaßverderber sein, aber wenn du das machst:

                        # RobA: Zuerst CtD ausfiltern 
                        iNewUnit = PAE_Unit.canUpgradeUnit(pUnit)
                        if iNewUnit == -1:
                                return True 

dann werden alle weitere Features nicht ausgeführt, zB: Unit zu Horse Upgrade, Sklaven der Stadt zuweisen etc.

[Rob Anybody]

Nein, das ist umgekehrte Logik
Das einzige was evtl. nicht ausgeführt wird, sind zB Ränge bei Einheiten die keine Praetorianer werden dürfen.
Dafür musste mann diese Zeilen ganz nach hinten schieben (direkt vor verlassen mit False). Da es aber wirklich nur sehr selten vorkommt, wollte ich damit nicht zusätzlich verwirren.

[xist10]

Schneller Test zeigt, Sklaven ansiedeln funktioniert.

Trotzdem hast du recht, da eigentlich sollte die Funktion vorher abbrechen. Da verhält sich etwas nicht 100% korrekt.

Aber wenn es bisher läuft...

[Rob Anybody]

Hier geht es allein um die KI. Alles was nicht in Python erledigt wird, versucht die KI im SdK zu erledigen, und dort kommt es zum CtD wenn falsche Werte übergeben werden. Rückgabe von "True" bedeutet: Ignoriere diese Einheit, das wurde schon behandelt.

Die Funktion canUpgradeUnit setzt eine Einheit (iNewUnit) nur auf -1 wenn diese Enheit eine Updateregel verletzt.
Die folgende If-Abfrage prüft mit ==-1 ob eine solche Verletzung festgestellt wurde. Alle anderen Einheiten werden durchgelassen.

In seltenen Fällen (so oft wie es zum CTD kam) kann das bedeuten, das eine Legion nun nicht mehr in den Rängen aufststeigt, weil sie kein Praetorianer werden darf.

Wir können aber gerne darüber reden, was Vorrang haben soll: doUpgradeVeteran oder doUpgradeRang

Edit:
Man muss Python bis zur Civ4-Ebene zurückverfolgen um die Auswirkung von True und False zu sehen.


Civ4/Assets/Phython/CvEntryPoints/CvGameInterface.py

def AI_unitUpdate(argsList):
	'AI moves units - return 0 to let AI handle it, return 1 to say that the move is handled in python '
	#CvUtil.pyPrint( "CvGameInterface.AI_unitUpdate" )
	return gameUtils().AI_unitUpdate(argsList)

[Rob Anybody]

Eigentlich liegt der Hund bereits in canUpgradeUnit begraben. Hier werden zwei verschiedene CtD-Gründe geprpüft und gemeinsam zurückgegeben.
(Zwei der CtD von Markus waren vor Grenzheer, zwei CtD waren danach). Und ja, ich haue mit dem Hammer drauf, damit es keine Schlupflöcher mehr gibt.
Es kann sein, das wenn eine KI-Einheit da rein fällt, sie für den Rest des Spieles inaktiv wird. Das kann ich aber erst nach einem Blick ins SDK beurteilen.

Für eine saubere Trennung müsste man die Schleife für jede Ursache einzeln durchlaufen lasen, aber das bedeutet doppelte Rechenzeit, weil normalerweise keiner der beiden Gründe vorliegt.

Ich könnte höchsten versuchen, doUpgradeVeteran und doUpgradeRang innerhalb von canUpgradeUnit abzuhandeln