PB 94 - Schaffen wir PAE?

[Rob Anybody]

Hier https://www.civforum.de/showthread.p...=1#post9464021 wurde eine noch offene Änderung vorgeschlagen.

Ich halte auch die kleine Korrektur für eine Änderung der Spielmechanik und würde es nicht übernehmen wollen. Wie seht ihr das?

[Flunky]

Das ist eine Korrektur der Spielmechanik as intended. So wie es implementiert ist, hat die Zufallszahl überhaupt keinen Einfluss. Der ganze Absatz

                                bUndoCorp = False
                                if lCorp and lReli:
                                        if CvUtil.myRandom(2, "undoCorp") == 1:
                                                bUndoCorp = True

                                # Kult
                                if lCorp or bUndoCorp: 

ist identisch zu:

                                # Kult
                                if lCorp: 

[Pie]

Das Rammenlimit sollte aber keinen OOS verursachen.

Und in removePagans sollte es so aussehen:

                    if lCorp and lReli:
                        if CvUtil.myRandom(2, "undoCorp") == 1:
                            bUndoCorp = True

                    # Kult
                    if lCorp and bUndoCorp: 

[xist10]

Beide Codeteile sollte ich einfügen können.
Das habe ich wohl vorher übersehen und da stimme ich Flunky zu, dass dies ein Codefehler ist.
Hat Pie ja auch bestätigt.

Bei den Rammen wäre wohl die Lösung, die Funktion zu verlassen, wenn Spieler ein Mensch ist ?

ich bezweifle aber trotzdem etwas, dass dies ein Problem darstellt.

[Rob Anybody]

Bei den Rammen wäre wohl die Lösung, die Funktion zu verlassen, wenn Spieler ein Mensch ist ?

ich bezweifle aber trotzdem etwas, dass dies ein Problem darstellt.

Der OOS kommt, weil ich etwas anklicke, was auf dem Server nicht geht.
Ich kann mir das so vorstellen: Lokal bin ich Mensch und darf Rammen bauen. Auf den Server bin ich mit ID=3 "KI" und darf keine Rammen bauen.

[xist10]

Also für den Mod müsste ich dann entweder die Rammen beim Bauen für alle begrenzen oder für alle freigeben ?

Das würde aber den OSS Fehler erklären.

[Flunky]

Oder einfach den Fix für Händler auch für Rammen übernehmen. Code folgt.

[Flunky]

Der Rammenteil von cannotTrain:

                # KI Rammen Problem
                if eUnit in L.LRammen:
                        iPlayer = pCity.getOwner()
                        pPlayer = gc.getPlayer(iPlayer)

                        iAnz = pPlayer.getUnitClassCountPlusMaking(gc.getUnitInfo(eUnit).getUnitClassType())
                        # check Order Queue in dieser Stadt
                        for iOrderCurrentCity in range(pCity.getOrderQueueLength()):
                                pOrder = pCity.getOrderFromQueue(iOrderCurrentCity)
                                if pOrder.eOrderType == OrderTypes.ORDER_TRAIN and pOrder.iData1 == eUnit:
                                        iAnz -= 1
                                        break
                
                        if iAnz >= 10:
                            return True
                        # if pPlayer.getUnitClassCountPlusMaking(eUnit) > 10:
                        #         return True 

€: korrigiert

[Pie]

Habs für PAE7 übernommen.

[Rob Anybody]

Ich verstehe nicht, warum wir mit der Fehlerbehebung nicht voran kommen.

Ich habe jetzt die beiden Änderungen in ein Patch.zip gepackt und bitte Zulan sie mir auf den Server zu kopieren, damit ich testen kann, ob der OOS-Fehler weiterhin auftritt ...

[xist10]

Es gibt keinen Grund.

Gut, ich wollte Zulan eigentlich nicht anschreiben, da dies PB schon wieder Probleme macht.
Ich hatte zumindest kurz darüber nachgedacht, mich mal zu informieren wie schwer es sein sollte, einen eigenen Server anzumieten / bereitzustellen.

Dann könnte ich wohl etwas schneller die vermutlich noch zu liefernen Patches veröffentlichen.

Hier wären v39 in der Dropbox hochgeladen.
Wie immer die Vollversion. In BtS/Mods/ entpacken und Dateien überschreiben.
Oder nutzt Robs Version als Patch. Im selben ordner entpacken, es werden aber nur die 2 geänderten Dateien überschrieben.

[Markus1978]

Kann ich Version 39 schon installieren oder soll ich noch damit warten?

[Rob Anybody]

Ich teste gerade, ob der OOS immer noch auftritt. Ich melde mich gleich dazu.

[Rob Anybody]

Der Fix funktioniert. Es gibt keinen OOS mehr.

Allerdings habe ich gegenüber der Version von xist10 eine etwas andere Einrückung bei

                        if iAnz >= 10:
                            return True 

Vielleicht kann sich jemand (Flunky/xist10?) das noch anschauen, ob die unterschiedliche Einrückung einen Unterschied ausmacht.


-----

Anmerkung:
Ich bin beim ersten Test (bei mir noch mit v38) wie damals unter dem Limit von 10 Rammen geblieben und konnte das Verhalten von #1060 nicht reproduzieren (der Phyton-Code auf dem Server ist jetzt ja auch ein anderer).
Nach dem 2. Einwählen, waren diese "weniger als 10 Rammen" noch vorhanden, aber als ich versucht habe mehr als 10 Rammen einzustellen, gab es wieder OOS und die überzähligen Rammen fehlten beim 3. Einwählen.

Sowohl mit meinem Patch als auch mit v39 ist es nun möglich 10 Rammen zu bauen, aber nicht mehr möglich überzählige Rammen in die Bauliste aufzunehmen.
Ich sehe es damit als erwiesen an, das Flunky den Fehler richtig erkannt und mit seiner Phyton-Änderung behoben hat.

[xist10]

Das macht kein Problem.

Die 2 Tabs von dir sind etwas besser, weil das mehr oder weniger konsistent mit dem Rest des Codes ist, aber dies dient nur der Lesbarkeit.

Wichtig für Python ist nur, dass de rteil nach einem : mehr eingerückt ist als vorher und alle Codezeilen gleichviel Eingerückt sind.