PB 94 - Schaffen wir PAE?

[Rob Anybody]

Wir haben eine Definitionslücke. Die KI braucht eine Else-Anweisung, wenn sie eine Einheit nicht updaten darf.

Combat4 ist Vorrausetzung für Praetorianer. Wenn wir nach Grenzheer jeder CtD-Einheit Combat4 wieder wegnehmen, besteht das Problem nicht mehr.

If Update
   if  Update ohne CtD
       Update ausführten
   else 
       Combat4 löschen
   Return True

If yxz

Am besten wird Combat4 erst gar nicht vergeben, wenn das beim anschließenden Update zum CtD führt.
Die entsprechende Combat4-Beförderung braucht also ebenfalls diese Abfrage.

[Rob Anybody]

Ich habe mir die verschiedenen Varianten in PAE7, PB6.17 (v25) und PB 7 nochmal angesehen und verstehe immer besser was da passiert.
Die folgende Änderung für PAE7 verhindert alle 4 CtD von Markus. Aber es löst das dahinterliegende Logikproblem nicht.



Ich prüfe jetzt noch innerhalb von canUpgrade wo genau die CtD (vor Grenzheer) durchschlüpfen und fange sie dort auf.
Die über dem Bugfix stehende Prätorianerabfrage ist doppelt gemoppelt mit der Abfrage in canUpgradeUnit. Mit doUpgradeVeteran innerhalb von canUpgrade könne diese Zeilen gelöscht werden.

Schon wieder ein Satz mit X

[Rob Anybody]

Ich habe die Zeile mit iNewUnit=-1 übersehen
Pie, du hast recht. Ich blockiere den CtD weil ich alle noch nicht behandelten Fälle blockiere.

Aber was ist unser noch nicht behandelter Fall?
Was versucht die KI mit den Generals-Legionen zu machen?
Was immer es ist, es muss noch in "AI_unitUpdate" aufgenommen werden.

Edit:
Oder, ist das doch erst in einer nachfolgenden Funktion?

[Rob Anybody]

Ich habe jetzt neue Log-Einträge gefunden.

Player 52 Unit 42238056 (TXT_KEY_LEADER_BARBARIAN's Rebell) moving from 17:28 to -2147483647:-2147483647
Player 7 Unit 28057691 (Tyrrhenus der Etrusker's Sklaven) moving from -2147483647:-2147483647 to 17:28
Rand = 2041593911 on 192499 (AI Unit Birthmark)
Player 7 Unit 26230912 (Tyrrhenus der Etrusker's Scorpio) moving from 16:29 to 17:28
Player 7 Unit 26230912 (Tyrrhenus der Etrusker's Scorpio) moving from 17:28 to 18:27
Player 7 Unit 28065997 (Tyrrhenus der Etrusker's Hipparchos) moving from -2147483647:-2147483647 to 24:31
Rand = 623266212 on 192499 (AI Unit Birthmark)
Player 7 Unit 21692569 (Tyrrhenus der Etrusker's Legio II Adiutrix (Praefectus)) moving from 24:31 to -2147483647:-2147483647
Player 7 Unit 28074137 (Tyrrhenus der Etrusker's Hipparchos) moving from -2147483647:-2147483647 to 24:31
Rand = 345342733 on 192500 (AI Unit Birthmark)
Player 7 Unit 16613393 (Tyrrhenus der Etrusker's Legio III Gallica (Praefectus)) moving from 24:31 to -2147483647:-2147483647
Player 7 Unit 28082193 (Tyrrhenus der Etrusker's Hipparchos) moving from -2147483647:-2147483647 to 24:31
Rand = -89853246 on 192501 (AI Unit Birthmark)
Player 7 Unit 25608213 (Tyrrhenus der Etrusker's Legio IV Scythica (Praefectus)) moving from 24:31 to -2147483647:-2147483647
Player 7 Unit 28090389 (Tyrrhenus der Etrusker's Hipparchos) moving from -2147483647:-2147483647 to 24:31
Rand = -29447981 on 192502 (AI Unit Birthmark)
Player 7 Unit 25436368 (Tyrrhenus der Etrusker's Legio XVIII (Praefectus)) moving from 24:31 to -2147483647:-2147483647

Die schuldie CtD-Einheit ist nicht als letztes aufgeführt. Das Feld 24:31 ist eine Stadt. Sagt das irgendetwas aus?

[xist10]

Combat4 müsste doch die Stärke4 Beförderung sein ?

Sonst findet laut Log wohl ein Upgrade Praefectus->Hipparchos statt, oder ? Eigentlich sollte da nichts möglich sein, dachte ich. Praefectus ist doch schon das höchste, was die Ränge der Römische Legion hat.
Aber mir sagt das leider garnichts.


Ich kenne mich nicht 100% in Python aus, aber spontan würde ich das ganze in dieser Reihenfolge einbauen:

def fun():
   ...
    iNewUnit = PAE_Unit.canUpgradeUnit(pUnit)
    if not iNewUnit == -1:
       do Rang Promo etc
    ...
    if iNewUnit == -1:
       return True

Also das return True später durchführen und vorher nur verhindern, dass man in die kritische Funktion rutscht.
Hast du schonmal versucht, aber ich denke, if not funktioniert etwas anders als !=.

[brettschmitt]

Soll ich jetzt das Startsave erstellen, oder ändert sich nochmal was?

@Rob
Deine startPitboss.bat verweist ja noch auf PAE_7, soll ich das auf PB_PAE_6,17 ändern? Danach den Durchlauf machen?

[xist10]

@brettschmitt, erstelle mal das Startsave, diese Änderung ist Savegamekompatibel. Dann können wir eventuell auch mal den Modupdater testen.
Und ja, ändere bitte alle Hinweise auf PB_PAE_7 auf PB_PAE_6.17.

[brettschmitt]

Zulan hat das neue Save bekommen

[Flunky]

Ich habe jetzt neue Log-Einträge gefunden.

Player 52 Unit 42238056 (TXT_KEY_LEADER_BARBARIAN's Rebell) moving from 17:28 to -2147483647:-2147483647
Player 7 Unit 28057691 (Tyrrhenus der Etrusker's Sklaven) moving from -2147483647:-2147483647 to 17:28
Rand = 2041593911 on 192499 (AI Unit Birthmark)
Player 7 Unit 26230912 (Tyrrhenus der Etrusker's Scorpio) moving from 16:29 to 17:28
Player 7 Unit 26230912 (Tyrrhenus der Etrusker's Scorpio) moving from 17:28 to 18:27
Player 7 Unit 28065997 (Tyrrhenus der Etrusker's Hipparchos) moving from -2147483647:-2147483647 to 24:31
Rand = 623266212 on 192499 (AI Unit Birthmark)
Player 7 Unit 21692569 (Tyrrhenus der Etrusker's Legio II Adiutrix (Praefectus)) moving from 24:31 to -2147483647:-2147483647
Player 7 Unit 28074137 (Tyrrhenus der Etrusker's Hipparchos) moving from -2147483647:-2147483647 to 24:31
Rand = 345342733 on 192500 (AI Unit Birthmark)
Player 7 Unit 16613393 (Tyrrhenus der Etrusker's Legio III Gallica (Praefectus)) moving from 24:31 to -2147483647:-2147483647
Player 7 Unit 28082193 (Tyrrhenus der Etrusker's Hipparchos) moving from -2147483647:-2147483647 to 24:31
Rand = -89853246 on 192501 (AI Unit Birthmark)
Player 7 Unit 25608213 (Tyrrhenus der Etrusker's Legio IV Scythica (Praefectus)) moving from 24:31 to -2147483647:-2147483647
Player 7 Unit 28090389 (Tyrrhenus der Etrusker's Hipparchos) moving from -2147483647:-2147483647 to 24:31
Rand = -29447981 on 192502 (AI Unit Birthmark)
Player 7 Unit 25436368 (Tyrrhenus der Etrusker's Legio XVIII (Praefectus)) moving from 24:31 to -2147483647:-2147483647

Die schuldie CtD-Einheit ist nicht als letztes aufgeführt. Das Feld 24:31 ist eine Stadt. Sagt das irgendetwas aus?

Der CtD passiert schon beim Upgrade von Legio XVIII, nicht bei der rot markierten. Sonst würde das Log nicht weitergehen. Ich schaue nachher mal rein.

[Rob Anybody]

Combat4 müsste doch die Stärke4 Beförderung sein ?

Sonst findet laut Log wohl ein Upgrade Praefectus->Hipparchos statt, oder ? Eigentlich sollte da nichts möglich sein, dachte ich. Praefectus ist doch schon das höchste, was die Ränge der Römische Legion hat.
Aber mir sagt das leider garnichts.

Nun ein Update, das nicht möglich ist, könnte den CtD auslösen.
Derr Ärger geht eindeutig von der roten Einheit aus. Kann die danach noch einmal an der Reihe gewesen sein?
Oder, kann Civ4 mehere Kerne benutzen? Könnte ein Kern noch Einheiten herumlaufen lassen, während der andere Kern sich am Update verschluckt?
Evtl. habe ich das Log auch aus einem Durchlauf ohne CtD ...., nein dann wären die Einheiten von einer anderen Civ

[Rob Anybody]

Zulan hat das neue Save bekommen

Vielen Dank

[brettschmitt]

Ich hoffe mal ihr könnt denn jetzt endlich anfangen!
Los gehts, ich will Storys lesen!

Achtung Spoiler:

und ferner hoffe ich, dass ich nicht nochmal ein Save erstellen muss, weil sich wieder was ändert

[Flunky]

Haben wir aktuell ein 6.17-CtD-Save?

[brettschmitt]

Haben wir aktuell ein 6.17-CtD-Save?

Ja, habs heut mittag an Zulan geschickt...

[EDIT] Äh nee, nicht CtD, sondern Startsave. CtD will ich nicht

[Rob Anybody]

Haben wir aktuell ein 6.17-CtD-Save?

Es gibt zwei Saves von Marcus für 6.17 und zwei für 7. Ich baue meine Änderungen immer in beide Versionen ein, um alle vier Saves zu testen.

Die Saves für 6.17 findest du hier: https://www.civforum.de/showthread.p...=1#post9334844
und hier: https://www.civforum.de/showthread.p...=1#post9341584