PB 94 - Schaffen wir PAE?

[Piccadilly]

Das kam glaube ich schonmal zur Sprache, aber ich schreibe es lieber nochmal:

Wenn das Spiel in naher Zukunft aufgesetzt wird, sollte vielleicht die Option "Neuer Random Seed bei erneuten Laden" ausgewählt werden. Das mache ich bei Mods inzwischen Standardmäßig, weil man so einige Saves noch retten kann, die sonst einen fatalen Bug hätten.

Abseits davon: Besten dank für den Aufwand, den ihr hier betreibt um dieses PB zu realisieren!

[shade]

Abseits davon: Besten dank für den Aufwand, den ihr hier betreibt um dieses PB zu realisieren!

Dem möchte ich mich anschließen.

[Rob Anybody]

Das kam glaube ich schonmal zur Sprache, aber ich schreibe es lieber nochmal:

Wenn das Spiel in naher Zukunft aufgesetzt wird, sollte vielleicht die Option "Neuer Random Seed bei erneuten Laden" ausgewählt werden. Das mache ich bei Mods inzwischen Standardmäßig, weil man so einige Saves noch retten kann, die sonst einen fatalen Bug hätten.

Abseits davon: Besten dank für den Aufwand, den ihr hier betreibt um dieses PB zu realisieren!

Das habe ich bei den Angaben für brettschnitt berücksichtigt.

[Markus1978]

Habe mit V4 in der Runde 883 folgende Fehlermeldung erhalten

Traceback (most recent call last):
  File "CvEventInterface", line 23, in onEvent
  File "CvEventManager", line 359, in handleEvent
  File "CvEventManager", line 3420, in onCombatResult
  File "PAE_Unit", line 2336, in doRankPromo
RuntimeError: unidentifiable C++ exception
ERR: Python function onEvent failed, module CvEventInterface

[Markus1978]

Habe in der Runde 913 eine weitere Fehlermeldung erhalten.
Fehlermeldung ist nicht wegklickbar und beim neu Laden verschwunden.

Traceback (most recent call last):
  File "CvGameInterface", line 146, in cannotTrain
  File "CvGameUtils", line 760, in cannotTrain
RuntimeError: unidentifiable C++ exception
ERR: Python function cannotTrain failed, module CvGameInterface

[Markus1978]

PB-PAE 7 V4:

Spielstand laden.
Runde beenden
Wenn ich dann die Runde erneut beende dann kommt es zu einem Crash to Desktop nach dem mir der Krieg erklärt wird.

[xist10]

Den Fehler aus 575 schaue ich mir eventuell mal an, genauso wie das Savegame.
Leider bin ich aktuell noch am lernen, also erst Dienstag abend.

[Rob Anybody]

PB-PAE 7 V4:

Spielstand laden.
Runde beenden
Wenn ich dann die Runde erneut beende dann kommt es zu einem Crash to Desktop nach dem mir der Krieg erklärt wird.

Der CtD ist beim Römer. Ich vermute, das ist das bekannte Problem.

"Probleme mit übergelaufenen Legionären": ist das der CTD von dem Legio III?

Ich denke mal. Kann ich aber nicht 100% bestätigen, aber der vorletzte Crash von Markus - #512 - war auch eine übergelaufnene Legion (in dem Fall meine ich Optio).


Spomtan denke ich, dass da irgendein Problem mit dem Ranksystem und fehlenen Techs ist ?
Beim Testen der berittenen Ränke der Römer hatte ich einen CtD, wenn die Tech für die 2. Einheit fehlt und deshalb die Rankaufstiege nicht zeitig gestoppt werden. Wobei Praefecte ja eigentlich schon am Ende sind...

Wurde das noch nicht gefixt?

[Markus1978]

Habe auch noch einen zweiten Spielstand 55 Runden vor Ende wo es ebenfalls beim Runde beenden zu einem Crash to Desktop kommt.

[Rob Anybody]

Es ist in beiden Fällen eine Einheit beim grünen (Ost-)Römer Konstantin.
Ich bin noch auf der Suche welche Einheit das genau ist.

[Rob Anybody]

Der Legio III Gallica ist eine tickende Zeitbombe. Je nach Kampfergebnissen führt er irgendwann zum CTD.



Auffällig: Wie schon bei #528 (Cyrenaica) handelt es sich um einen General.

[Rob Anybody]

In 4 verschiedenen Saves waren es immer Generäle
UnitName=Legio I Minervia
UnitName=Legio III Cyrenaica
UnitName=Legio III Gallica

Gemeinsam ist ihnen:
PromotionType=PROMOTION_LEADER
PromotionType=PROMOTION_LEADERSHIP
PromotionType=PROMOTION_FORM_FOURAGE
und mindestens PromotionType=PROMOTION_RANG_ROM_12

Einzeln sind diese PromotionType kein Problem, aber anscheinend führt genau diese Kombination früher oder später zum CTD.

Ich nehme an, die Summe der Boni überschreitet verbotene Grenzen: evtl. über 100% Heilung
Ich kann das aber nicht näher eingrenzen, möglicherweise wird der Effekt auf andere Einheiten auf dem Feld übertragen.

[Pie]

Ich habe es gefunden: Mit diesen beiden geänderten Dateien funktioniert der Updater

Habs übernommen, aber ausdokumentiert.

Habe mit V3 in der Runde 596 folgende Fehlermeldung erhalten.

Traceback (most recent call last):
  File "CvEventInterface", line 23, in onEvent
  File "CvEventManager", line 359, in handleEvent
  File "CvEventManager", line 3292, in onCombatResult
  File "PAE_Unit", line 3662, in doNavalOnCombatResult
  File "PAE_Unit", line 3671, in getNearestValidPlot
NameError: global name 'GC' is not defined
ERR: Python function onEvent failed, module CvEventInterface

zweite Fehlermeldung kommt in der Runde 599

Traceback (most recent call last):
  File "CvEventInterface", line 23, in onEvent
  File "CvEventManager", line 359, in handleEvent
  File "CvEventManager", line 5347, in onCityDoTurn
  File "PAE_City", line 2068, in provinceTribute
RuntimeError: unidentifiable C++ exception
ERR: Python function onEvent failed, module CvEventInterface

Das erste ist mein Fehler. Pie oder xist sollten das leicht beheben können.

Das zweite ist vermutlich eine Folge eines unter Umständen ungültig werdenden Pointers. Das kann ich mir nach dem Urlaub anschauen.

Erstes hab ich mit gc gefixt.
Zweites hab ich mit getNumRealBuilding statt isHasBuilding wahrscheinlich gefixt.

Habe mit V4 in Runde 603 folgende Fehlermeldung erhalten

Traceback (most recent call last):
  File "CvEventInterface", line 23, in onEvent
  File "CvEventManager", line 359, in handleEvent
  File "CvEventManager", line 3292, in onCombatResult
  File "PAE_Unit", line 3662, in doNavalOnCombatResult
  File "PAE_Unit", line 3671, in getNearestValidPlot
ArgumentError: Python argument types in
    CyMap.findCity(CyMap, int, int, int)
did not match C++ signature:
    findCity(class CyMap {lvalue}, int, int, int, int, bool, bool, int, int, class CyCity *)
ERR: Python function onEvent failed, module CvEventInterface

Das weiß ich nicht, ob ich das fixen kann. Ich habs mal probiert. Mir fehlt eine korrekte Variable für den letzten Wert. Ich versuchs mit None.

Habe mit V4 in der Runde 883 folgende Fehlermeldung erhalten

Traceback (most recent call last):
  File "CvEventInterface", line 23, in onEvent
  File "CvEventManager", line 359, in handleEvent
  File "CvEventManager", line 3420, in onCombatResult
  File "PAE_Unit", line 2336, in doRankPromo
RuntimeError: unidentifiable C++ exception
ERR: Python function onEvent failed, module CvEventInterface

Gefixt. Es fehlte and iNewPromo != -1 bei den Reitern (wie oben bei den Fusssoldaten).

Habe in der Runde 913 eine weitere Fehlermeldung erhalten.
Fehlermeldung ist nicht wegklickbar und beim neu Laden verschwunden.

Traceback (most recent call last):
  File "CvGameInterface", line 146, in cannotTrain
  File "CvGameUtils", line 760, in cannotTrain
RuntimeError: unidentifiable C++ exception
ERR: Python function cannotTrain failed, module CvGameInterface

Hab ich gefixt. Das Argument brauchte die UnitClassInfo und nicht die UnitInfo.

In 4 verschiedenen Saves waren es immer Generäle
UnitName=Legio I Minervia
UnitName=Legio III Cyrenaica
UnitName=Legio III Gallica

Gemeinsam ist ihnen:
PromotionType=PROMOTION_LEADER
PromotionType=PROMOTION_LEADERSHIP
PromotionType=PROMOTION_FORM_FOURAGE
und mindestens PromotionType=PROMOTION_RANG_ROM_12

Einzeln sind diese PromotionType kein Problem, aber anscheinend führt genau diese Kombination früher oder später zum CTD.

Ich nehme an, die Summe der Boni überschreitet verbotene Grenzen: evtl. über 100% Heilung
Ich kann das aber nicht näher eingrenzen, möglicherweise wird der Effekt auf andere Einheiten auf dem Feld übertragen.

Das müsstet ihr mir bitte noch weiter beobachten. Wenn dem so sei, dann muss ich an den Promos im XML herumschrauben....

----------

Ich fahre übermorgen eine Woche auf URLAUB auf die Insel Kos! Ich lad heute diese Änderungen ins git. Damit könnt ihr dann arbeiten, hoff ich.

[xist10]

Kurze Fragen, was sind die Änderungen bei
File "PAE_City", line 2068, in provinceTribute
File "CvGameUtils", line 760, in cannotTrain
genau ?

if not pCity.isHasBuilding(gc.getInfoTypeForString("BUILDING_PROVINZPALAST")):
->
if not pCity.getNumRealBuilding(gc.getInfoTypeForString("BUILDING_PROVINZPALAST")):


und

if pPlayer.getUnitClassCountPlusMaking(eUnit) > 10:
->
if pPlayer.getUnitClassCountPlusMaking(eUnit.getUnitClassInfo()) > 10:

Oder könnte ich einen Link zum git bekommen, dann suche ich mir das selber raus.
Im Zweifel muss ich Flunky dafür nerven.

CyMap.findCity() würde ich demnächst nochmal etwas rumprobieren und im Zweifel die Funktion hier auskommentieren. Wird aktuell nur für teleportierenes Treibgut verwendet.

Die Promos werden wir dann beobachten.

Sonst viel Spaß auf Kos.

[Rob Anybody]

Ich habe es gefunden: Mit diesen beiden geänderten Dateien funktioniert der Updater

Habs übernommen, aber ausdokumentiert.

Was genau hast du auskommentiert?

Alle Codezeilen mit "if PBMod .." werden mit "PBMod = True" oder "PBMod = False" gesteuert.
Das hattest du bereits so eingebaut gehabt und wurde von xist10 und mir nur ergänzt.
Damit "PBMod = True" den Updater vollständig einschaltet, dürfen Teile davon nicht zusätzlich auskommentiert sein.

Die Zeile 103 (PAEMod = "PiesAncientEurope" oder PAEMod = "PB_PAE_7") macht in diesem Zusammenhang keinen Sinn und kann mMn gelöscht werden.
Edit: Sorry ich habe das falsch gelesen.

Ich denke dabei an zukünftige PB-Spiele und PBmod-Versionen. Wer erinnert sich dann noch an die genaue Fuktionsweise wenn er den Updater mit "PBMod = True" einschaltten möchte?