[TAC] Bugdokumentation & Allgemeines Brainstorming

[Ronnar]

Ursache für den Absturz und die langen Rundenzeiten war eine holländische Karacke, die Pferde an die Ureinwohner verkaufen wollte. Die Karacke gilt aber als Militärschiff und kann keine Ureinwohnerdörfer betreten. Ich habe daher Militärschiffe, die Transportkapazitäten besitzen, von diesem Verbot ausgenommen.

Dann müssen wir das Coding für die Colopädie anpassen, oder?

In der CvPediaUnit.py geben wir einen Hinweis, welche Schiffe keine Ureinwohnerdörfer anlaufen können:

   if ((gc.getUnitInfo(self.iUnit).getDomainType() == 0) and not (gc.getUnitInfo(self.iUnit).isOnlyDefensive())):
    szSpecialText += "\n" + localText.getText("TXT_KEY_NO_INDIAN_HARBOUR", ()) 

Diese Logik würde dann ja jetzt nicht mehr passen?

[koma13]

Dann müssen wir das Coding für die Colopädie anpassen, oder?

Danke, daran hätte ich nun gar nicht gedacht.

Diese Logik würde dann ja jetzt nicht mehr passen?

Einfach ein

... and not gc.getUnitInfo(self.iUnit).getCargoSpace() > 0

hinten anhängen.

[Ronnar]

Ich sehe da kritische Exploitmöglichkeiten: Wenn man ein Kriegsschiff in einem Ureinwohnerdorf plaziert, kann man vorbeifahrende gegnerische Schiffe angreifen, ist selber aber dort im Hafen vor Angriffen geschützt. Ich nehme mal an, dass zu verhindern war der ursprüngliche Grund für diese Logik.

Die Pythondatei kann ich natürlich jederzeit ändern und gleich noch mein altes Coding fehlerunanfälliger machen (DomainTypes.DOMAIN_SEA statt 0). Gib mir dann einfach Bescheid, wenn deine Änderung in die nächste Version kommt und ich ändere die Pythondatei.

[BeckerC1972]

Zu 2.:
Eigentlich dachte ich, dass das schon so funktioniert ?
(Dass bei Auto-Bau die benötigten Resourcen dazugekauft werden ...)

Nachdem "1" so schnell Anklang fand, frage ich doch mal nach "2"

[Writing Bull]

Ich sehe da kritische Exploitmöglichkeiten: Wenn man ein Kriegsschiff in einem Ureinwohnerdorf plaziert, kann man vorbeifahrende gegnerische Schiffe angreifen, ist selber aber dort im Hafen vor Angriffen geschützt. Ich nehme mal an, dass zu verhindern war der ursprüngliche Grund für diese Logik.

Klingt plausibel!

Nur ganz kurz: Wenn wir über "Militärschiffe" oder "Kriegsschiffe" reden, dann meinen wir angriffsfähige Schiffe, nicht?

[koma13]

Ich sehe da kritische Exploitmöglichkeiten: Wenn man ein Kriegsschiff in einem Ureinwohnerdorf plaziert, kann man vorbeifahrende gegnerische Schiffe angreifen, ist selber aber dort im Hafen vor Angriffen geschützt. Ich nehme mal an, dass zu verhindern war der ursprüngliche Grund für diese Logik.

Na gut, wir haben aber das Problem, daß die KI die Karacke als Transportschiff verwendet. Und da sollten dann auch die entsprechenden Regeln gelten. Ein Transportschiff, welches keine Ureinwohnersiedlungen ansteuern kann, funktioniert ja nicht, wie wir gesehen haben.

[raystuttgart]

Na gut, wir haben aber das Problem, daß die KI die Karacke als Transportschiff verwendet. Und da sollten dann auch die entsprechenden Regeln gelten. Ein Transportschiff, welches keine Ureinwohnersiedlungen ansteuern kann, funktioniert ja nicht, wie wir gesehen haben.

Wie wäre es dann, wenn wir einfach noch eine Prüfung der Angriffsstärke einbauen ?
(Nur eine Überlegung)

Also Schiffe mit großer Angriffsstärke (z.B. Kaperschiff, Linienschiff, Fregatte) ausnehmen ?
(Die dürfen weiterhin keine Ureinwohnerdörfer befahren.)

Das würde den von Ronnar beschriebenen Exploit deutlich abmildern, oder ?
Wer würde schon versuchen, dem Gegner mit einer Karracke aufzulauern ?

Bringt uns natürlich wenig, wenn die KI auch Schiffe, die eine hohe Angriffsstärke haben, zum Handeln mit den Ureinwohner nutzen will.

[koma13]

Also Schiffe mit großer Angriffsstärke (z.B. Kaperschiff, Linienschiff, Fregatte) ausnehmen ?
(Die dürfen weiterhin keine Ureinwohnerdörfer befahren.)

Das würde den von Ronnar beschriebenen Exploit deutlich abmildern, oder ?
Wer würde schon versuchen, dem Gegner mit einer Karracke aufzulauern ?

Das ist schon so geplant. Nur Kriegsschiffe mit Frachtraum sollen Ureinwohnersiedlungen befahren können. Das betrifft also nur die Karacke und die Galeone.

[raystuttgart]

Vergiss meine Anmerkungen.

Wenn da eh nur Galleone und Karacke betroffen sind, seh ich die Gefahr eines Exploits eigentlich nicht.

[Writing Bull]

Nur Kriegsschiffe mit Frachtraum sollen Ureinwohnersiedlungen befahren können. Das betrifft also nur die Karacke und die Galeone.

Wie hast du das technisch umgesetzt? Durch eine DLL-Änderung, die das automatisch bewirkt, oder manuell über die Zuweisung dieser Eigenschaft in der UnitInfo.xml? Ersteres vermutlich, nicht?

[raystuttgart]

Wie hast du das technisch umgesetzt? Durch eine DLL-Änderung, die das automatisch bewirkt, oder manuell über die Zuweisung dieser Eigenschaft in der UnitInfo.xml? Ersteres vermutlich, nicht?

Ich habe koma so verstanden, dass er das durch eine DLL-Änderung umgesetzt hat.
Das sollte doch aber übers SVN ersichtlich sein ?

[Writing Bull]

Das sollte doch aber übers SVN ersichtlich sein ?

Die Änderung ist noch nicht auf dem Server.

[raystuttgart]

Die Änderung ist noch nicht auf dem Server.

Achso.

[Ronnar]

Wartet nur auf meine Killer-Galleonen-Armada

Wenn wir die Alternative haben den Bug zu beseitigen und dafür mit einer kleinen Exploitmöglichkeit leben zu müssen oder den Bug drin zu lassen, dann bin ich natürlich auch für Bug beheben.

Das Colopädie-Coding habe ich schon angepasst & getestet. Das lade ich dann hoch, sobald koma die DLL mit dem Fix hochgeladen hat.

[raystuttgart]

Wenn wir die Alternative haben den Bug zu beseitigen und dafür mit einer kleinen Exploitmöglichkeit leben zu müssen oder den Bug drin zu lassen, dann bin ich natürlich auch für Bug beheben.

koma hat wahrscheinlich mittlerweile sowieso eine Lösung gefunden, um sowohl Bug zu beheben, wie auch den Exploit zu verhindern.

Aber ja, der Exploit ist im Vergleich zum Bug - meiner Meinung nach - vernachlässigbar.