[TAC] Ausbildung

[Akropolis]

Ich hätte zum Thema Ausbildung (egal ob bei Indios, Schule oder LBD) den Wunsch, dass diese Ausbildung auch in der Statistik auftaucht. Unter dem F9 Berater findet man doch, wie oft eine Einheit da ist und wie oft diese "errichtet" wurde. Hier zählt im Moment nur die Rekrutierung und die Auswanderung in Europa.

[Writing Bull]

Testanfrage zu "Korrekturen bzgl. Gamespeed bei LbD"
(Rev. # 176)

zu prüfen: (...)
2. Beim Spielen längerer Partien darauf achten, das LbD (Stadtberufe) immer noch passiert und die Frequenz passt

@ray, könntest du das bitte präzisieren? Wie funktioniert die Anpassung von LbD an die Spielgeschwindigkeiten? Gibt es da einen xml-Eintrag aus der CIV4GameSpeedInfo.xml, an den das gekoppelt ist?

Fürs Testen ist das wichtig. Gibt es eine einzige, zentral justierte Stellschraube oder gibt es kategoriespezifische (oder gar berufsspezifische) Stellschrauben?

[raystuttgart]

@ray, könntest du das bitte präzisieren? Wie funktioniert die Anpassung von LbD an die Spielgeschwindigkeiten? Gibt es da einen xml-Eintrag aus der CIV4GameSpeedInfo.xml, an den das gekoppelt ist?

Ja, es ist wie Dutzende anderer Features im Bereich Ausbildung und Einheiten an den XML-Eintrag

<iTrainPercent>

in CIV4GameSpeedInfo.xml gekoppelt.

Da das Ganze aber zufallsbasiert ist sollte es beim Test LEDIGLICH darum gehen:

1) Einfach nochmal überprüfen, das LbD bei Stadtberufen funktioniert -> das sollte ja schon zigfach passiert sein
2) rein gefühlsmässig (da ja dadurch auch Balancing geändert) -> passt die Häufigkeit -> das hätten die Tester sicher auch gemeldet, wenn es nicht mehr passt.

Fazit:

Ich glaube den Test kannst du mittlerweile als erledigt sehen.

[Writing Bull]

Aha!

Was wir testen sollten: dass die Anpassung grundsätzlich funktioniert. Also nicht, ob das Balancing stimmt - sondern ob die Ankopplung an die Spielgeschwindigkeiten grundsätzlich implementiert ist.

Wie sollte ein Tester hier vorgehen?

[raystuttgart]

Aha!

Was wir testen sollten: dass die Anpassung grundsätzlich funktioniert. Also nicht, ob das Balancing stimmt - sondern ob die Ankopplung an die Spielgeschwindigkeiten grundsätzlich implementiert ist.

Wie sollte ein Tester hier vorgehen?

Also folgendes ist eigentlich sicher:

1) Bei Spielgeschwindigkeit 100% hat sich an den LbD Häufigkeiten nichts getan
2) Bei den anderen Spielgeschwindigkeiten ändert sich LbD in EXAKT dem selben Verhältnis, wie z.B. normale Ausbildung.
3) Wenn LbD für Stadtberufe nicht mehr funktioniert hätte, wäre das mittlerweile aufgefallen.


Aber wenn ihr es nochmal explizit testen wollt, dann macht vielleicht folgendes:

1.) Vergleicht Spielgeschwindigkeit Marathon und Spielgeschwindigkeit Normal.
2.) Mit Worldbuilder jeweils 10 Städte und jeweils 5 Siedler.
3.) Klickt 100 Runden durch und schaut nachher nach, wieviele Siedler durch LbD zu Experten wurden.

[Writing Bull]

Danke. Ich habe deine Testanfrage entsprechend editiert.

[Akropolis]

Zur Testanfrage:

Ich habe Spiele auf episch und auf extra-Lager gespielt. Die Preise für Söldner waren bei episch höher.

LbD findet bei beiden Varianten statt. Da der Zeitpunkt auf Zufall steht, ist es mal schneller und mal langsamer. Ich lasse meist die Rohstoffsammler, die es nicht in Europa gibt, so ausbilden.

[RAB]

Dieser Beitrag stand ursprünglich im Sammelthread zu offiziellen Versionen. - W.B.

Hallo TAC-Team,

ich hatte gerade einen interessanten Bug: 3 Pioniere (Kleinkrimminelle) laufen gruppiert durch eine Stadt. In der Runde, in der 2 der Krimminellen durch Arbeit zu freien Kolonisten werden, wurden 300 Eisenwaren in der Stadt abgelegt - also die benötigten Eisenwaren für Pioniere. Allerdings sind sie weitergezogen und sind immer noch Pioniere. Ich habe also 300 Eisenwaren geschenkt bekommen.

Soweit ich es verstanden habe, wird ja bei "Learning by doing" der Siedler in der Stadt neu erzeugt - das macht Sinn, wenn die Beförderung in der Stadt erarbeitet wird. Bei Pionieren außerhalb der Stadt, sollte dies aber nicht geschehen, richtig?

Im Screenshot kann man es gut nachverfolgen:


Möglicherweise ist dieser Bug aber schon bekannt und behoben? Eine neue Version soll ja in den nächsten Tagen releast werden...

Viele Grüße!

[raystuttgart]

Dieser Beitrag stand ursprünglich im Sammelthread zu offiziellen Versionen. - W.B.

Soweit ich es verstanden habe, wird ja bei "Learning by doing" der Siedler in der Stadt neu erzeugt - das macht Sinn, wenn die Beförderung in der Stadt erarbeitet wird. Bei Pionieren außerhalb der Stadt, sollte dies aber nicht geschehen, richtig?

Es wird bei LbD auch für Berufe ausserhalb der Stadt eine Einheit neu erzeugt.
Das ist technisch notwendig.
(Alte vernichtet, neue erzeugt.)

Allerdings sollte die alte Einheit einfach komplett mit ihrer Ausrüstung vernichtet werden und die neue Einheit im selben Beruf (also auch mit der selbn Ausrüstung) neu erzeugt werden.

Das auf einmal Ausrütsung in der Stadt landet, kann ich mir nicht erklären.

Ich prüfe das mal.

[raystuttgart]

Sorry, ich habe was falsches getestet.

Ich habe den Aufstieg zum Spezialisten getestet.

Achtung Spoiler:

Habe es mehrfach getestet:

Der beschriebene Fehler tritt bei mir nicht auf.

Es werden KEINE Werkzeuge in einer Stadt erzeugt, wenn ein Pionier durch LbD zum Spezialisten wird.

In der Logik gibt es auch nichts, dass dieses Verhalten erklären könnte.

Ich vermute folgendes:

Irgendein Event hat da zugeschlagen.
Oder es wurde ein Gebäude beschleunigt, wodurch Werkzeuge gekauft wurden.

Was die Fehlermeldung angeht muss aber folgendes getestet werden:

Kleinkrimineller im Beruf Pionier -> Freier Siedler im Beruf Pionier

Sehe es mir nochmal an.

Update:

Auch hier taucht der Fehler bei mir NICHT auf.
Auch im Code gibt es nichts, was den Fehler erklären könnte.

Ich bleibe bei meiner Vermutung:

Irgendein Event hat da zugeschlagen.
Oder es wurde ein Gebäude beschleunigt, wodurch Werkzeuge gekauft wurden.

[Writing Bull]

Das wäre prima, wenn du das testen könntest!
Und danke für deine Meldung, RAB!

Freier Siedler im Beruf Pionier

Nur kurz, damit keine Verwirrung entsteht: "Freie Siedler" gibt es nicht bei TAC. Es gibt die Einheit "Freier Kolonist", und es gibt den Beruf "Siedler".

[raystuttgart]

Ah Stop !

Ich habe schon wieder was beim Test übersehen !
Das ist passiert, als die Einheit AUF einer Stadt stand.

Ich hab das getestet, als die Einheit NEBEN der Stadt gearbeitet hat.

Sehe mir das nochmal an.

[RAB]

Also ein Gebäude wurde nicht gekauft - die Stadt produziert Politikpunkte. Im Lager waren 150 Einheiten und 300 kamen dazu -> daher war das Lager überfüllt...

[raystuttgart]

Also ein Gebäude wurde nicht gekauft - die Stadt produziert Politikpunkte. Im Lager waren 150 Einheiten und 300 kamen dazu -> daher war das Lager überfüllt...

Ja, du hattest Recht, ich kann den Fehler nachvollziehen.
(Sorry, hatte bei deiner Beschreibung nicht genau gelesen.)

Fehler tritt unter folgenden Bedingungen auf:

* Schuldknecht oder Kleinkrimineller im Beruf Pionier wird zum Freien Kolonisten im Beruf Pionier
* Während das passiert steht er auf einer Stadt.

-> Die Ausrüstung die ein Pionier braucht, wird in der Stadt gelagert.

Anmerkungen:

1. Diese Kombination ist extrem selten.
2. Der Fehler ist nicht schlimm, es geht nichts kaputt, man bekommt halt einfach ein paar Werkzeuge.
3. Ich kann es mir vom Code her noch nicht genau erklären. Ich vermute aber es liegt an der Konvertierung und das ich gleichzeitig die neue Einheit ebenfalls im selben Beruf erzeugt habe (siehe Codestellen in Rot)

-> Sprich ich habe 2 mal Werkzeuge erzeugt !

Hier der Codeauszug:

Achtung Spoiler:

UnitTypes DefaultUnitType = (UnitTypes)GC.getCivilizationInfo(getCivilizationType()).getCivilizationUnits(GC.getDefineINT("DEFAULT_POPULATION_UNIT"));
	FAssert(DefaultUnitType != NO_UNIT);
	CvUnit* DefaultUnit = initUnit(DefaultUnitType, convUnit->getProfession(), convUnit->getX_INLINE(), convUnit->getY_INLINE(), convUnit->AI_getUnitAIType());
	FAssert(DefaultUnit != NULL);
	DefaultUnit->joinGroup(convUnit->getGroup());
	DefaultUnit->convert(convUnit, true);

[raystuttgart]

Ich habe den Fehler behoben und erfolgreich getestet.

Hier der Fix. (2 Mal eine Zeile austauschen.)


1.) Dies ist die Stelle für KK / SK wird Freier Kolonist.

Fehlerherbeiführung:
* Schuldknecht oder Kleinkrimineller im Beruf Pionier wird zum Freien Kolonisten im Beruf Pionier
* Während das passiert steht er auf einer Stadt.

In CvPlayer.cpp siehe Code in Rot.

Achtung Spoiler:

int randomValue = GC.getGameINLINE().getSorenRandNum(1000, "LbD Free Map");
	
	// no Success if randomValue larger calculatedChance
	if (randomValue > calculatedChance)
	{
		return false;
	}
	
	// convert Unit to Free Settler
	UnitTypes DefaultUnitType = (UnitTypes)GC.getCivilizationInfo(getCivilizationType()).getCivilizationUnits(GC.getDefineINT("DEFAULT_POPULATION_UNIT"));
	FAssert(DefaultUnitType != NO_UNIT);
	//CvUnit* DefaultUnit = initUnit(DefaultUnitType, convUnit->getProfession(), convUnit->getX_INLINE(), convUnit->getY_INLINE(), convUnit->AI_getUnitAIType());
	CvUnit* DefaultUnit = initUnit(DefaultUnitType, (ProfessionTypes) GC.getUnitInfo(DefaultUnitType).getDefaultProfession(), convUnit->getX_INLINE(), convUnit->getY_INLINE());
	FAssert(DefaultUnit != NULL);
	DefaultUnit->joinGroup(convUnit->getGroup());
	DefaultUnit->convert(convUnit, true);

2. Für den Fall Freier Kolonist wird Experte gibt es den equivalenten Fehler.

Fehlerherbeiführung:
* Freier Kolonist im Beruf Pionier wird zum Experten
* Während das passiert steht er auf einer Stadt.

Also auch hier ebenfalls in CvPlayer.cpp siehe Code in Rot.

Achtung Spoiler:

int randomValue = GC.getGameINLINE().getSorenRandNum(1000, "LbD Expert Map");
	
	// no Success if randomValue larger calculatedChance
	if (randomValue > calculatedChance)
	{
		return false;
	}
	
	int expert = GC.getProfessionInfo(convUnit->getProfession()).LbD_getExpert();
			
	UnitTypes expertUnitType = (UnitTypes)GC.getCivilizationInfo(getCivilizationType()).getCivilizationUnits(expert);
	FAssert(expertUnitType != NO_UNIT);
	//CvUnit* expertUnit = initUnit(expertUnitType, convUnit->getProfession(), convUnit->getX_INLINE(), convUnit->getY_INLINE(), convUnit->AI_getUnitAIType());
	CvUnit* expertUnit = initUnit(expertUnitType, (ProfessionTypes) GC.getCivilizationInfo(getCivilizationType()).getDefaultProfession(), convUnit->getX_INLINE(), convUnit->getY_INLINE());
	FAssert(expertUnit != NULL);
	expertUnit->joinGroup(convUnit->getGroup());
	expertUnit->convert(convUnit, true);

koma muss also 2 Mal lediglich eine Zeile austauschen.
(Habe es mehrfach getestet.)

ACHTUNG !:
Die zwei Zeilen sind in den beiden Fällen UNTERSCHIEDLICH !

@Team:
Ok, Sorry für den Fehler, aber der Fall ist so selten, dass er mir einfach nie aufgefallen ist.

@RAB:
Respekt und Danke fürs aufmerksame Testen.

Tipps für den Test:
(nachdem koma meinen Fix übernommen hat)

Achtung Spoiler:

1.) Einstellungen im XML setzen, damit die Ereignisse (Expertenaufstieg und Freiheit) sehr schnell und sehr häufig kommen.
2.) Mit Worldbuilder Freie Kolonisten, Kleinkrimineller und Schuldknecht setzen, Pioniere setzen, die Werkzeuge der Pioniere auf die anderen Einheiten verteilen
3.) Die Einheiten AUF der Stadt stehen lassen und ein paar Runden durchklicken
4.) Wenn ein Aufstieg bzw. die Freiheit erfolgt ist, in die Stadt schauen, obs da mehr Werkzeuge gibt, was nicht passieren soll.