Bugs

[rucivfan]

Kann bitte jemand nachschauen, warum das Spiel nach Runde beenden hängen bleibt?

Bin beim debuggen.

Assert Failed

File:  CvGlobals.cpp
Line:  3645
Expression:  strcmp(szType, "NONE")==0 || strcmp(szType, "")==0
Message:  info type IMPROVEMENT_ORBITAL_STATION not found, Current XML file is: xml\GameInfo/CIV4EspionageMissionInfo.xml

----------------------------------------------------------

Wann wird das mal entfernt, Mono? Zu Faul? Du musst doch nur mal alle Pythonaufrufe dafür entfernen. Durchsuche einfach mal die Pythondateien nach IMPROVEMENT_ORBITAL_STATION.

Da es diese Modernsisierung nicht gibt, kann sie auch nicht gefunden werden. Es gibt nur eine Einheit UNITCLASS_ORBITAL_STATION, aber das ist was ganz anderes.

Assert Failed

File:  CvUnitAI.cpp
Line:  11612
Expression:  canDefend()
Message:  

----------------------------------------------------------

Dieser FAssert sagt aus, dass eine Einheit ohne Kampffähigkeit KI-Code für Kampffähige Einheiten ausführt. Sowas führt schon in PAE zu vielen Problemen.

Es geht genau um UNITAI_ATTACK_CITY als KI. Eine Einheit ist aber auch nicht Angriffsfähig, wenn sie auf den falschen Terrain ist. In dem Fall ist es falsches Terrain. Diese Einheit wird trotz Transport für bestimmten KI-Code ausgewählt.



Das ist alles noch nicht die Problemursache für den Hänger. Die Ursache konnte ich bisher nur einkreisen.

Achtung Spoiler:

2 Einheiten auf diesen Feld sind die Ursache.

edit: Das ist sehr Merkwürdig. Eine der 2 Einheiten will sich in einer Gruppe formieren und weckt damit die andere Einheit auf. Diese war im Interceptionmodus. Wenn der Modus vom Interceptionmodus zu Missionsmodus wechselt, wird die Gruppe wieder in den Idlemopdus versetzt und genau das läuft dann endlos.

[rucivfan]

Habe es noch nicht 100% verstanden, aber ich würde mal sagen, ich verstehe das Problem.

Welche Einheit verbirgt sich hinter der Type-Nummer 100?

edit: Die Ursache für das Problem ist im Zusammenhang mit Wasserstädten zu sehen. Eine Einheit will zu einer anderen Wasserstadt als Verteidiger.

edit: Das ist ein Astreiner CoC-Schrottcode-Fehler.

Es gibt hier eine Änderung der CvUnitAI::AI_plotValid. Diese ist für den Fehler verantwortlich.

case DOMAIN_LAND:
		//if (pPlot->getArea() == getArea() || canMoveAllTerrain())
		if (pPlot->isValidDomainForLocation(*this))
		{
			return true;
		}
		break;

Das auskommentierte ist der BTS Code. Was das Problem daran ist? Nun, der Problemcode geht teilweise davon aus, wenn die Funktion plotValid true wieder gibt, dann gibt es auch einen Weg. Den gibt es hier aber nicht.

Es gibt zwar noch eine zweite Instanz, die den Fehler verhindert, aber ich weiß nicht warum es nicht verhindert wird. Eine Landeinheit kann nun mal nicht über Wasser laufen.

edit: Können hier Landeinheiten unter bestimmten Gegebenheiten mit Ausnahme Stadtfelder über Wasser laufen? Die Ursache habe ich schon gefunden. In einer wichtigen Funktion wird für Wassereinheiten das laufen auf Plots mit Wasser nicht mehr versperrt. Jedoch wurden alle anderen Funktionen nicht angepasst. Code denkt, man kann über Wasser laufen und die Einheit merkt später, es geht doch nicht.

case DOMAIN_LAND:
		if (pPlot->isWater() && !canMoveAllTerrain())
		{
			if (!pPlot->isCity() || 0 == GC.getDefineINT("LAND_UNITS_CAN_ATTACK_WATER_CITIES"))
			{
				/*if (bIgnoreLoad || !isHuman() || plot()->isWater() || !canLoad(pPlot))
				{
					return false;
				}*/
				if (!bIgnoreLoad && isHuman() && canLoad(pPlot))
				{
					break;
				}
				if (GET_TEAM(getTeam()).isWaterWork() && pPlot->getBlockadedCount(getTeam()) == 0 && !bAttack && pPlot->getTeam() == getTeam())
				{
					break;
				}
				if (pPlot->getTeam() == getTeam() && flatMovementCost())
				{
					break;
				}

				return false;
			}
		}
		break;

Woher stammt die Änderung aus der bool CvUnit::canMoveInto(const CvPlot* pPlot, bool bAttack, bool bDeclareWar, bool bIgnoreLoad) const?

edit: OK, das ist mal wieder eine typischer KI-Fehler. Die canMoveInto lässt mit den Code oben was zu, was in der bool CvPlot::isValidDomainForAction(const CvUnit& unit) const nicht zugelassen wird. Daher kommt das Problem.

Lösung: Die Änderung oben rückgängig machen.

edit: Es wäre Alternativ auch möglich int pathDestValid(int iToX, int iToY, const void* pointer, FAStar* finder) in CvGameCoreUtil.cpp zu ändern, aber die Auswirkungen kann ich nicht beurteilen.

if ((!(pSelectionGroup->canFight())) || (pSelectionGroup->getDomainType() == DOMAIN_LAND && !pToPlot->isValidDomainForLocation(*pSelectionGroup->getHeadUnit())))

Das dicke müsste weg.

edit: Oder man fügt GET_TEAM(getTeam()).isWaterWork() && pPlot->getBlockadedCount(getTeam()) == 0 && pPlot->getTeam() == getTeam() von oben der CvPlot::isValidDomainForAction bei Landeinheiten hinzu. Die Folgen sind aber auch nicht absehbar. Auf jeden Fall würden dann Landeinheiten auf bearbeiten Wasser gegen Schiffe verteidigen.

[Monaldinio]

Erstmal wieder ein dickes danke das du sich so intensiv mit dem Problem auseinander gesetzt hast!

Du hast ja nun gleich 3 Lösungsansätze präsentiert, was ist deiner Meinung nach der beste?



Edit sagt zu deinen Post heute morgen, ja...Mona ist echt zu faul...weil er meint das dieser Fehler schnell zu beseitigen wäre und er es immer aufschiebt...und dann vergisst! Im neuen Update wird es diesen Fehler nicht mehr geben.

[rucivfan]

Du hast ja nun gleich 3 Lösungsansätze präsentiert, was ist deiner Meinung nach der beste?

Das kommt drauf an, was if (GET_TEAM(getTeam()).isWaterWork() && pPlot->getBlockadedCount(getTeam()) == 0 && !bAttack && pPlot->getTeam() == getTeam()) oben bezwecken sollte.

Du hast die dickgedruckte Frage noch nicht beantwortet. Sollen Land-Einheiten über Wasser laufen? Dürfen Landeinheiten über Wasser gegen Seeeinheiten verteidigen? Beides ist nicht BTS-Standard.

[Monaldinio]

Ab einer bestimmten Technologie ist es soll es möglich mit landeinheiten über bestimmte wasserfelder zu laufen.
Die Frage möchte ich bejaen.

[rucivfan]

Dann musst du Lösung 3 nehmen, wobei es das Problem gibt, dass sich Landeinheiten dann auf Wasser verteidigen.

bool CvPlot::isValidDomainForAction(const CvUnit& unit) const
{
	switch (unit.getDomainType())
	{
	case DOMAIN_SEA:
		return (isWater() || unit.canMoveAllTerrain());
		break;

	case DOMAIN_AIR:
		return false;
		break;

	case DOMAIN_LAND:
	case DOMAIN_IMMOBILE:
		return (!isWater() || unit.canMoveAllTerrain() || (unit.getTeam() == getTeam() && (unit.flatMovementCost() || (GET_TEAM(unit.getTeam()).isWaterWork() && getBlockadedCount(unit.getTeam()) == 0))));
		break;

	default:
		FAssert(false);
		break;
	}

	return false;
}

Habe es noch nicht getestet mit den Save.

[Monaldinio]

Mhmm, ich guck mir das nochmal an wie es momentan ist wenn man die Tech erforscht hat, die das laufen über bedtimmte Wasserfelder erlaubt...

[rucivfan]

Habe es getestet, Runde läuft durch!

edit:

[Monaldinio]

Ja prima!

Edit: Jetzt muss ich den Code aus deinen post #201 nur noch einbauen und neu kompilieren.


Der Screenshot zeigt den Wiederbelebungsfehler, der in meiner Arbeitsversion schon korrigiert sein sollte, durch deine zusätzliche Abfrage!

Ich habe dir die Version gegeben die aktuell zum Download zu Verfügung steht.

[Monaldinio]

IST ZUSTAND!

Also ich habe jetzt nochmal ein bischen genauer getestet, mit der Technologie Cenauri Hydrologie können Einheiten über Wasserfelder im eigenen Gebiet laufen. Sollte so auch bleiben.
Einheiten die sich auf Wasserfelder befinden und von Seeeinheiten angegriffen werden, verteidigen sich nicht. Grundsätzlich finde ich das gut, da Landeinheiten nun mal Landeinheiten sind.

WÜNSCHENSWERTER ZUSTAND

So wie ich dich verstanden habe, würde deine Codekorrektur den Bug beheben...aber auf kosten das sich dann Landeinheiten die sich auf Wasserfeldern im eigenen Gebiet befinden, gegen Seeeinheiten verteidigen können.
Das finde ich nicht so gut. Am liebsten wäre mir das man das so beibehalten könnte.

Wenn es aber zu schwierig ist bzw die Codekorrektur unabsehbare Folgen haben könnte, wäre ich auch damit zufrieden wenn sich besagte Einheiten nur mit, sagen wir mal 10-20% ihrer eigentlichen Stärke, auf Wasserfeldern verteidigen.

[rucivfan]

Dann musst du Lösung 2 nehmen. Das hat aber den Nachteil, dass die KI ihr teuren Land-Einheiten einfach mal so verlieren kann und das nicht berücksichtigt. Daher wäre die 20% Stärke-Lösung wohl am sinnvollsten.

[Monaldinio]

Gut, dann die 20% Stärke Variante auf Wasserfeldern im eigenen Gebiet.

Edit: wobei mir aber auch zwei Ausnahmen einfallen, schwebepanzer (hovertank) und Rotor!

[Monaldinio]

rucivfan ich möchte dich nicht hetzten, du hast nicht zufällig den Code mit den 20% auf der Pfanne?!?

[rucivfan]

Nein, aber das ist doch nicht schwer. Ich schreibe vielleich was Dienstag nachmittag, aber nur vielleicht. Base hat erstmal vorrang. Vorher habe ich auch keine Zeit.

[SMAC-Fan]

Unterseeboote von der Fraktion Wellen können bis zu 4 Landeinheiten transportieren? Ist das so gewollt?
Jedenfalls seh ich von ihm nur ein Unterseeboot und er landet mit 4 Armeeeinheiten auf dem Land vor meiner Basis...