[Release] Muddy's D2NT

Page 108 of 495

108

04/30/2011 12:14 Muddy Waters#1606

Naja, was mir daran nicht gef�llt ist, dass das Problem eigentlich nur umgangen wird, anstatt es wirklich zu l�sen, was in dem Fall aus meiner Sicht aber machbar w�re.

Ich werde nochmal versuchen, dass bei mir zu reproduzieren und dann genau die Ursache nachvollziehen und das Problem an der Wurzel anpacken.

An dieser Stelle nochmal ein allgemeiner Hinweis:

Ich hatte hier irgendwo eine Modifikation der Funktion NTTMGR_IDItems() genannt, weil einige Probleme mit verkauften unid Items hatten.
Danach kamen Meldungen, dass Items gedroppt w�rden, anstatt sie beim H�ndler zu verkaufen. Gestern Abend habe ich dies auch bei einer meiner Sorcs enddeckt, sodass ich mir die Sache eben nochmal angeschaut habe.

Dabei musste ich feststellen, dass ich ziemlichen Unsinn von mir gegeben habe; die modifizierte Funktion enth�lt grobe Denkfehler - ich entschuldige mich daf�r nochmal an dieser Stelle, derart dumme Fehler sollten mir eigentlich nicht passieren... :o

Ich bitte darum alle, die diese modifizierte Variante der oben genannten Funktion verwendet haben, diese durch folgende Funktion zu ersetzen:

Code:

function NTTMGR_IDItems(npc)
{
	var _tome, _scroll;
	var _result, _check;
	var _items = NTT_GetUnidItems();
		
	if(!_items || _items.length < 1)
		return;

	_tome = NTT_GetTome();

	if(_tome && _tome.GetStat(70) < _items.length)
		NTT_FillTome(npc);

	for(var i = 0 ; i < _items.length ; i++)
	{
		switch(NTSI_CheckItem(_items[i], true))
		{
		case 1:
			MWI_WriteManagerLog(_items[i], 0, true);
		case 2:
			break;
		case 0:
			if(_items[i].GetItemCost(1) >= 35E3)
				break;
		case -1:
			if(_tome)
				_result = NTT_IdItem(_tome, _items[i]);
			else
			{
				_scroll = NTT_GetScroll();

				if(!_scroll)
				{
					NTT_BuyScrolls(npc);
					_scroll = NTT_GetScroll();
				}

				_result = NTT_IdItem(_scroll, _items[i]);
			}

			if(!_result)
				break;
			
			_check = NTSI_CheckItem(_items[i], true);
			
			if(_check == 1)
				MWI_WriteManagerLog(_items[i], 0, true);
			else if(_check == 0 || _check == 3)
			{	
				if(MWConfig_UseXMLItemlog)
					MWI_WriteLog(_items[i], 1);
				if(MWConfig_LogSoldItems)	
					MWI_WriteManagerLog(_items[i], 3, true);
				NTT_ShopItem(_items[i], npc, 1);
			}
			break;
		}
	}
}

Dann klappts auch mit dem Verkaufen. ;)

LG
Muddy

04/30/2011 12:33 ldevil#1607

Quote:

Originally Posted by Muddy_Waters

Naja, was mir daran nicht gef�llt ist, dass das Problem eigentlich nur umgangen wird, anstatt es wirklich zu l�sen, was in dem Fall aus meiner Sicht aber machbar w�re.

Ich werde nochmal versuchen, dass bei mir zu reproduzieren und dann genau die Ursache nachvollziehen und das Problem an der Wurzel anpacken

Seh ich nicht so. Momentan macht der Bot ja folgendes:

1. Suche das magische Item f�r das Rezept (unlimitiert)
2. Suche Rune, Jool und Gem (unlimitiert, aber nur wenn 1. erledigt)

Das Problem ist, dass diese Reihenfolge zwar f�r die "komplizierten" Items wie Amu (hohes ilvl erforderlich) oder so ca. alles ausser Ringe (selten ansich) funktioniert. Weil der Bot die Items schlicht selten findet.

Entweder muss man nun f�r alle Rezepte eine "Priorisierung" machen (was evtl. geht aber wohl ziemlich kompliziert ist da es vom Level des Charakters und den Scripts abh�ngt) oder man macht es halt eben so, dass man gewisse Items limitiert. Ich denke aber nicht, dass die Limitierung ein "Umgehen" des Problems ist, auch wenn ich verstehen kann, dass du gerne eine L�sung h�ttest die das dynamisch ohne Limits hinbekommt.

Der Fehler tritt halt erst auf, wenn der Char ein genug hohes Level hat und somit eigentlich alle Ringe die er findet die Level Anforderungen erf�llen, was wohl bei den Amus nicht ganz so schnell der Fall ist.

Und danke f�r den Fix :)

04/30/2011 12:53 Gaterrrr#1608

Quote:

Originally Posted by Muddy_Waters

Das ist ein Bug in der Release Version. Wie das ausbessern kannst, erf�hrst du in [Only registered and activated users can see links. Click Here To Register...].
Ausserdem findest du [Only registered and activated users can see links. Click Here To Register...] nochmal ein paar erkl�rende Worte zum Rush Script.

LG
Muddy

Vielen Dank, der zu rushende spricht jetzt mit Warriv. Allerdings st�rzt dann, wenn er in den n�chsten Akt wechseln will, also nach A2, immer Diablo 2 ab (Fatal Error), und ich wei� nicht, woran das liegt. (Videos habe ich installiert)

[Only registered and activated users can see links. Click Here To Register...]

04/30/2011 12:56 Shrimps in Bierteig#1609

Quote:

Originally Posted by Gaterrrr

Vielen Dank, der zu rushende spricht jetzt mit Warriv. Allerdings st�rzt dann, wenn er in den n�chsten Akt wechseln will, also nach A2, immer Diablo 2 ab (Fatal Error), und ich wei� nicht, woran das liegt. (Videos habe ich installiert)

[Only registered and activated users can see links. Click Here To Register...]

einfach im ntstarter bei Nosound h�ckchen wegmachen. Das sollte das problem beheben.

mfg

04/30/2011 13:10 Gaterrrr#1610

Quote:

Originally Posted by Shrimps in Bierteig

einfach im ntstarter bei Nosound h�ckchen wegmachen. Das sollte das problem beheben.

mfg

Vielen Dank, das hat geholfen - der Bot erreicht jetzt sicher Akt 2. Allerdings tritt hier das n�chste Problem auf: Beim Zauberer macht der Rusher ein Tp auf, der zu rushende geht durchs Tp, und dann steht der Rusher dort dumm rum wie 50m Feldweg

04/30/2011 13:16 Muddy Waters#1611

Quote:

Originally Posted by Gaterrrr

Vielen Dank, das hat geholfen - der Bot erreicht jetzt sicher Akt 2. Allerdings tritt hier das n�chste Problem auf: Beim Zauberer macht der Rusher ein Tp auf, der zu rushende geht durchs Tp, und dann steht der Rusher dort dumm rum wie 50m Feldweg

Beweg den Quester mal 1, 2 Schritte oder sorg daf�r, dass er get�tet wird. :D

Klingt komisch, aber speziell dort neigt die API dazu, die Unit des Questers nicht zu finden, nachdem dieser direkt durchs TP gekommen ist. :rolleyes:

@ldevil
Ich werde die Ursache jetzt mal schnell analysieren, deiner Argumentation kann ich nicht folgen, vom Cubing Algorithmus her ist die Sache klar limitiert, das Problem liegt in der itemloc der Items bei der Initialisierung - soweit habe ich es jedenfalls gesehen, als ich das letzte mal einen fl�chtigen Blick drauf geworfen habe.

Irgendwas war auch besonders wichtig bei der Initialisierung, bl�derweise habe ich es mir nicht dazugeschrieben, ich bin aber sicher, dasman auf irgendwas achten musste. :D

LG
Muddy

04/30/2011 13:35 AndreundNina#1612

Also irgent wie schmeist meine Sorc immer noch alles raus nach dem identifizieren mhm dabei meine ich alles ersetzt zu haben was du eine seite voher gepostet hast.

04/30/2011 13:39 Muddy Waters#1613

Quote:

Originally Posted by AndreundNina

Also irgent wie schmeist meine Sorc immer noch alles raus nach dem identifizieren mhm dabei meine ich alles ersetzt zu haben was du eine seite voher gepostet hast.

Edit:
Kommando zur�ck, verlesen.

Kannst du mal ein bisschen mehr Infos geben?
Unter welchen Umst�nden passiert das? Welcher Akt? Wieviele Items hat der Char in etwa dabei?
Wie sehen deine Configs aus?

LG
Muddy

04/30/2011 13:41 ldevil#1614

Quote:

Originally Posted by Muddy_Waters

@ldevil
Ich werde die Ursache jetzt mal schnell analysieren, deiner Argumentation kann ich nicht folgen, vom Cubing Algorithmus her ist die Sache klar limitiert, das Problem liegt in der itemloc der Items bei der Initialisierung - soweit habe ich es jedenfalls gesehen, als ich das letzte mal einen fl�chtigen Blick drauf geworfen habe.

Irgendwas war auch besonders wichtig bei der Initialisierung, bl�derweise habe ich es mir nicht dazugeschrieben, ich bin aber sicher, dasman auf irgendwas achten musste. :D

LG
Muddy

Hm, ok ich seh meinen Denkfehler, muss es mir wohl doch noch einmal ansehen ^^

edit:

Eigentlich geht's doch nur um den Abschnitt:

Code:

[B][I][COLOR="Navy"]if[/COLOR][/I][/B](_classid == NTConfig_CubingItem[i][1]) [COLOR="DarkGreen"]// Check Magic Item (Amu, Ring, ...)[/COLOR]
{
	[B][I][COLOR="Navy"]if[/COLOR][/I][/B]((result == [COLOR="Red"]0[/COLOR] || result == [COLOR="Red"]3[/COLOR]) && item.quality == [COLOR="Red"]4[/COLOR] && _calclevel >= _NTCU_CraftRecipe[_index][[COLOR="Red"]0[/COLOR]])
	{
		[B][I][COLOR="Navy"]if[/COLOR][/I][/B](NTConfig_CubingItem[i][[COLOR="Red"]2[/COLOR]] == null)
		{
			NTConfig_CubingItem[i][[COLOR="Red"]2[/COLOR]] = _gid;
			_result = [B][I][COLOR="Navy"]true[/COLOR][/I][/B];
		}
		[B][I][COLOR="Navy"]else[/COLOR][/I][/B] [B][I][COLOR="Navy"]if[/COLOR][/I][/B](NTConfig_CubingItem[i][[COLOR="Red"]2[/COLOR]] == _gid && NTCU_CheckItemCount(_classid,[COLOR="Red"]4[/COLOR],_NTCU_CraftRecipe[_index][[COLOR="Red"]0[/COLOR]]))
			_result = [B][I][COLOR="Navy"]true[/COLOR][/I][/B];
		[B][I][COLOR="Navy"]else[/COLOR][/I][/B] [B][I][COLOR="Navy"]if[/COLOR][/I][/B](MWConfig_UseAdvancedCubingAlgorithm)
		{
			[B][I][COLOR="Navy"]if[/COLOR][/I][/B](_checkedGids.indexOf(NTConfig_CubingItem[i][[COLOR="Red"]2[/COLOR]]) > [COLOR="Red"]-1[/COLOR])
			{
				NTConfig_CubingItem[i][[COLOR="Red"]2[/COLOR]] = _gid;

				[B][I][COLOR="Navy"]return[/COLOR][/I][/B] [B][I][COLOR="Navy"]true[/COLOR][/I][/B];
			}
			[B][I][COLOR="Navy"]else[/COLOR][/I][/B]
				_checkedGids.push(NTConfig_CubingItem[i][[COLOR="Red"]2[/COLOR]]);
		}
	}	
}

Wenn wir jetzt annehmen, der Bot findet immer nur Ringe (als Beispiel) sprich vercubt keine, dann l�uft das doch so ab:

Den ersten Ring beh�lt er, weil ja noch keine GID eingetragen ist und tr�gt die GID ein. Nun passiert nix bis ein weiterer gefunden wird.
Wird ein 2. Ring gefunden, so wird die GID vom 1. Ring in die Liste eigetragen und der 2. Ring weggeworfen.
Wird nun ein 3. Ring gefunden, so steht die GID ja in der Liste und der 3. Ring wird behalten und von nun an als "1. Ring" gef�hrt.

Sprich eigentlich wirft der Bot einfach jedes 2. Item weg?

Oder hab ich schon wieder einen Denkfehler?

edit: Ok der Denkfehler war: das wird ja jeweils pro Item gemacht �ber alle Rezepte und nicht �ber alle Items...

edit2: Warum nat�rlich das Init das Problem ist, weil die Items nicht korrekt initialisiert werden. Wobei ich die Init Funktion gar nicht kapier. Eigentlich muss man doch einfach alle Items die in der Kiste / Inventar liegen rasch �berpr�fen. Oder hab ich schon wieder nen Denkfehler?

Code:

_items = me.GetItems();

if(!_items)
	return false;

for(i = 0; i < _items.length; i++)
{
	if(_items[i].mode == 0)
	{
		if(_items[i].itemloc == 0 || _items[i].itemloc == 4)
		{
				NTSI_CheckItem(_items[i], true);
		}
	}
}

Ok, evtl. m�sste man statt dem Inventar den Cube dazu nehmen, weil das Inventar wird ja eh danach �berpr�ft...

04/30/2011 14:59 jentownhh#1615

Quote:

Originally Posted by ldevil

Das muss auch so heissen:

Code:

MWConfig_Script.push([[COLOR="DimGray"]"NTPindleskin.ntj"[/COLOR], [COLOR="Red"]10[/COLOR]]); NTConfig_NihlathakExtension = [COLOR="Navy"][B][I]false[/I][/B][/COLOR]

Jo die h�tte geklappt wenn das rote Portal noch da gewesen w�hre :D

K�nnte man die Standart Nihlathak.ntj noch so �ndern, das er immer Nihlathak �berspringt :D ?

04/30/2011 15:07 Muddy Waters#1616

Wollte gerade sagen, irgendwas siehst du da falsch. :D

Ich nenne hier ja gerne das Schubladen Modell, weil das die aus meiner Sicht anschaulichste M�glichkeit ist, sich das vorzustellen.

Jedes aktive Rezept ist eine Schublade.
Alle Items auf dem Char werden durchlaufen, wobei ein Item, dass f�r ein Rezept geeignet ist, erstmal auf s�mtliche Schubladen verteilt wird, in die es passt.
Meine Erweiterung kontrolliert nun diese Schubladen und pr�ft dabei, ob ein Item bereits in einer anderen Schublade liegt. Wenn ja, besteht die M�glichkeit das derzeit betrachtete Item in die derzeit betrachtete Schublade abzulegen und so das alte, mehrfach vorkommende Item, zu ersetzen.
Letztenendes werden so auch Items f�r verschiedene Rezepte aufgehoben, als beispielsweise nur ein Jewel f�r s�mtliche Rezepte vorzusehen.

Trotzdem k�nnen so aber nicht mehr Items aufgehoben werden, als tats�chlich auch f�r Rezepte ben�tigt werden, der Bug entsteht nicht dadurch.
Ich habe den Fehler zwar eben nicht reproduzieren k�nnen, dennoch glaube ich es soweit analysiert zu haben, um sagen zu k�nnen, was das Problem ausl�st.

Dazu muss man sich zun�chst die Initialisierung des Cubing Arrays respektive der "Schubladen" anschauen. Dort f�llt auf, dass zun�chst Items aus dem Inventar einsortiert und erst danach Items aus der Truhe betrachtet werden.

Das habe ich so umgesetzt, weil es sonst Probleme gibt, in Verbindung mit Items, die nur in begrenzter Menge aufgehoben werden und der Festlegung, nur solche Items zum Cuben zu verwenden, welche nicht aufgehoben werden sollen.

Grunds�tzlich findet die Initialisierung des Cubing Arrays zu Beginn eines jeden Bot Scripts statt. Aus irgendeinem Grund wird aber in der NTTMGR_TownManager() Funktion nochmal initialisiert.
Wobei man dazu sagen muss, dass dies nur unter gewissen Annahmen zu dem Problem f�hren kann.

Es m�ssen n�mlich die identifizierten und nicht ben�tigten Items bzw. Ringe noch im Inventar liegen und dieser Fall tritt gerade dann ein, wenn die nicht ben�tigten Items aufgrund entsprechender Einstellungen gedroppt werden oder aber aufgrund des gerade behobenen Bugs w�hrend der der Identifizierung, im Inventar verbleiben.

Findet nun eine erneute Initialisierung statt, werden s�mtliche Items aus dem Inventar, die f�r irgendwelche Rezepte geeignet sind, auch f�r diese eingeplant und entsprechend gestasht.
Die bereits gestashten Ringe verbleiben einfach wo sie sind, auch wenn sie nicht direkt f�r Rezepte eingeplant sind, einfach weil gestashte Items nicht mehr anger�hrt werden.

L�sung des Problems ist einfach das Entfernen der entsprechenden zus�tzlichen Initialisierung. Bisher sehe ich noch keine dadurch entstehenden Probleme, aber ich schaue mir die Sache trotzdem nochmal an. :)

LG
Muddy

04/30/2011 15:11 -Nemesis1337-#1617

Hey Muddy, hab da grad nen Fehler im Itemparcer.
Bei Diablo is nen Ormus Gew�nder gedroppt, jedoch hat der Bot sofort gestoppt und hat den Fehler hier ausgegeben:
[Only registered and activated users can see links. Click Here To Register...]

Gibts da was von Muddys?

Achja:

Code:

{
	var i;
	var _identified;
	var _result = 0;
	var _quantity, _quantityString, _replacedCheckListItem, _specs;

	_identified = item.itemflag & 0x10;
	
	if(arguments.length < 2)
		checkQuantity = false;
	if(arguments.length < 3)
		nipDebug = false;
	
	for(i = 0 ; i < _NTIP_CheckList.length ; i++)
	{
		if(_NTIP_CheckList[i][0].length > 0)
		{
			if(_NTIP_CheckList[i][0].indexOf("maxquantity") == -1)
			{	
				if(eval(_NTIP_CheckList[i][0]))
				{
					if(_NTIP_CheckList[i][1].length > 0)
					{
->Zeile 93:					if(eval(_NTIP_CheckList[i][1]))
						{
							if(nipDebug)
								return new Array(1, _NTIP_CheckList[i][2], _NTIP_CheckList[i][3], _NTIP_CheckList[i][4], _NTIP_CheckList[i][0] + " && " + _NTIP_CheckList[i][1]);
														
							return 1;
						}
						else if(!_identified && _result == 0)
							_result = -1;
					}
					else
					{
						if(nipDebug)
							return new Array(1, _NTIP_CheckList[i][2], _NTIP_CheckList[i][3], _NTIP_CheckList[i][4], _NTIP_CheckList[i][0]);
												
						return 1;
					}
				}
			}

04/30/2011 15:16 Muddy Waters#1618

In dem Fall sind deine .nips fehlerhaft - zumindest teilweise.

Versuchs mal mit der .nip Check Erweiterung aus dem zweiten Post. ;)
Zusatzinformationen dazu sollten sich per Suchfunktion finden lassen.

LG
Muddy

04/30/2011 15:46 Gaterrrr#1619

@Muddy: Wenn ich den Rusher in Richtung Wizard move, dann macht er manchmal automatisch weiter, wenn nicht, dann hilft es, den Quester zu moven / sterben zu lassen.

Darf man in naher oder ferner Zukunft mit einer Verbesserung des Rush-Scripts @ Wizard rechnen? Denn es w�re wirklich besser, wenn man nichts machen m�sste, beim Rush-Bot. ;-)

Und um dann nochmal auf den D2NT Log zur�ck zu kommen: Mit dem Cuben hattet ihr recht, da ist es normal, dass die Essence nicht angezeigt werden. Allerdings ist mir aufgefallen, dass der Log zeitversetzt geschrieben wird. Teilweise 20 Minuten - so lange dauert ein Run bei meinem Bot. Darf ich deshalb annehmen, dass der Bot erst nach dem Beenden aller Scripts und nach dem Verlassen des Games in die Log beim D2NT Manager die aufgehobenen Items eintr�gt?

04/30/2011 15:51 ldevil#1620

Quote:

Originally Posted by jentownhh

Jo die h�tte geklappt wenn das rote Portal noch da gewesen w�hre :D

K�nnte man die Standart Nihlathak.ntj noch so �ndern, das er immer Nihlathak �berspringt :D ?

Hast du das Nila Script mit "Skip on Tomb Vipers"? wenn ja, setzt den ausdruck da im if(...) einfach auf if(false) ist etwas ineffizient aber schnell umgestellt. dann geht er runter sagt "skip" und geht danach wieder hoch zu pindle ;)

@Muddy: Ok, ez hab ich's glaubs einigermassen kapiert. Auch wenn ich noch nicht erkannt hab wieso die Reihenfolge der Initialisierung wichtig ist. Aber das hab ich mir auch net angeschaut um ehrlich zu sein.

Mein "fix" verhindert, dass mein Bot alle 2 Games mult weil alles voller Ringe ist, das passt f�r mich f�r's erste :)

Page 108 of 495

108