Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

NPC: I am already talking to you... En 7x #102

Closed
brewsterl opened this issue Aug 1, 2016 · 41 comments
Closed

NPC: I am already talking to you... En 7x #102

brewsterl opened this issue Aug 1, 2016 · 41 comments
Assignees

Comments

@brewsterl
Copy link
Contributor

brewsterl commented Aug 1, 2016

Bien, esto si lo confirmo desde hace mucho y pasa de la siguiente manera: Si estan 3 npcs muy cerca (A unos 5 SQM por ejemplo) y dices Hi. Uno te saluda, y los otros te dicen NPC: I am already talking to you... Como por ejemplo al comprar promotion en Thais, Carlin, etc. O tambien si estas en Carlin Depot, y hablas con el npc de los parcel y terminas de hablar con el y rapidamente vas a otro npc y hablas te dice I am already talking to you... cuando le dices hi

Tambien ocurre que si hay un player y un npc cerca a unos 3 SQM por ejemplo? y hablas con el npc y le dices bye, hasta ahi todo bien, luego cuando vienen otro player y dice HI y esta aun el player parado cerca del npc, el mismo npc esta ocupado porque esta hablando con el player que esta afk parado cerca de el.. ¿?¿?¿?¿?¿?

Problema obviamente solo se encuentra en 7x
Me explico? no encontre en donde está el error a ver si me puedes ayudar

Aca se explica un poco mejor el bug:
https://gfycat.com/BrightFluidLabradorretriever

@brewsterl brewsterl added the bug label Aug 1, 2016
@mattyx14
Copy link
Owner

mattyx14 commented Aug 1, 2016

Probaste con los scripts únicos del servidor? - Eso para ver si los NPC no tienen ningún problema por que veo que se mueven muy rápido.

Pero para que el sistema de NPC responda a muchos players a la vez ocupas ir a:
\data\npc\lib\npcsystem\modules.lua

-- Constants used for shop mode. Notice: addBuyableItemContainer is working on all modes
SHOPMODULE_MODE_TALK = 1 -- Old system used before Tibia 8.2: sell/buy item name
SHOPMODULE_MODE_TRADE = 2 -- Trade window system introduced in Tibia 8.2
SHOPMODULE_MODE_BOTH = 3 -- Both working at one time

-- Used in shop mode
SHOPMODULE_MODE = `SHOPMODULE_MODE_TALK``

Eso debe solucionar le hablado múltiple.

@brewsterl
Copy link
Contributor Author

Si claro el problema se reproduce igualmente yo revise todo los libs y cambie y bueno no consegui la falla y si se reproduce con los npcs de otx te invito a que lo intentes para que veas que ocurre tal cual, el problema de la speed es solo el walkinterval = 1000 y es una vieja update de RealOTX eso ya lo arregle pero el problema es ese, y tambien lo tengo en mi servidor personal tibiana. La solucion temporal que le consegui fue separar los npcs que estan "pegados" como en rl (Paladin store en carlin) promotions en kings, etc..

Aqui te dejo unas fotos con el otx clean y npcs:

sin titulo

Aca en la segunda foto le digo Hi a Josue y rapidamente camino hacia Yasmin y me aparece el mensaje I am already talking to you...
2

@mattyx14
Copy link
Owner

mattyx14 commented Aug 1, 2016

Trata ver si el error se reproduce en la OTX 3 si eso es asi el problema es ligado con el client.
Si no es así el problema estará en el código de la OTX 2.

@brewsterl
Copy link
Contributor Author

Testeado y en otx3 no se reproduce este error, aca dejo una foto:

Milton está en un piso superior, Le digo hi, bajo rapidamente y le digo Hi a Britany y funciona perfectamente como deberia ser.

sin titulo

EL problema esta ligado al codigo de otx2 x) pero exactamente no se que es..

@brewsterl
Copy link
Contributor Author

Aca otro ejemplo de este bug de un amigo que utiliza otx2, tal cual como te explique anteriormente este lugar en carlin.. que me vi forzado a separar los npc y eliminar a legola por esa misma razon

13898617_930064387102555_284550016_o png

@mattyx14
Copy link
Owner

mattyx14 commented Aug 2, 2016

voy a ver que puedo hacer si lo puedo solucionar

@mattyx14
Copy link
Owner

mattyx14 commented Aug 6, 2016

no pude jejeje

@brewsterl
Copy link
Contributor Author

xD Pasé por lo mismo, llevo tiempo buscando una solucion pero aun no se que pueda ser :v

@mattyx14 mattyx14 changed the title NPC: I am already talking to you... En 7x NPC: I am already talking to you... En 7x (OTX 2) Sep 9, 2016
@disu5
Copy link

disu5 commented Oct 17, 2016

i have that same problem :/
what i can do with that?

@Felipe1993
Copy link

Felipe1993 commented Oct 25, 2016

aun no se puede? probe con distintas versiones de otx editar distintos protocoles y editar sources aun sin exito ... otx 2 ya no sera mas trabajado?

@brewsterl
Copy link
Contributor Author

De momento tengo compromisos personales y problemas en mi país que no me permiten continuar ayudando en otx2, cuando tenga algo de tiempo continuare

@Felipe1993
Copy link

Felipe1993 commented Dec 23, 2016

encontre una solucion /parche no editando los sources
a los npc problematicos o que estan cercanos unos de otros usen
talkradius = 3 <<< significa que hablara cuando te acerques a 3 sqm del npc puede ser cualquier numero
saludos ! grande otx

@mattyx14
Copy link
Owner

Es una buena opción pero debe reparar directamente en código ya que otx 3 no tiene ese problema

@brewsterl
Copy link
Contributor Author

Es solo un parche de agua fria y yo la verdad aun no consigo este error :P

@Felipe1993
Copy link

Felipe1993 commented Jan 7, 2017

si lo se pero ayuda.... para quienes no quieran dejar otx 2.x etc. saludos me gustaria saber que cambios hay en el otx 2.8?

@mattyx14
Copy link
Owner

mattyx14 commented Jan 7, 2017

Sólo nombre y la compilación del code que no se realizó ni en 2.6 ...Ni 2.7 ,... Revisa el log

@Felipe1993
Copy link

Felipe1993 commented Jan 25, 2017

gracias

@brewsterl brewsterl added the OTX2 label Jan 27, 2017
@brewsterl brewsterl changed the title NPC: I am already talking to you... En 7x (OTX 2) NPC: I am already talking to you... En 7x Jan 27, 2017
@brewsterl brewsterl reopened this Mar 19, 2017
@tarantonio
Copy link

¿any advances with this issue?

@brewsterl
Copy link
Contributor Author

@tarantonio nope

@tarantonio
Copy link

tarantonio commented Apr 25, 2017

@mattyx14 habría que darle un empujón a este bug, es básico para montar un server old school en OTX

@brewsterl
Copy link
Contributor Author

@tarantonio baja el talkradius a 3 y separa algunos npca y listo. Otx2 está descontinuado desde hace mucho, queda mejor trabajar en otx3 x).

@tarantonio
Copy link

@mattyx14 otx esta bien pero no es viable para versiones 7.x de momento

@mattyx14
Copy link
Owner

otx 3 funciona perfectamente para 7.x y es mas funcional la otx 2

@brewsterl
Copy link
Contributor Author

De casualidad probaste otx3 en 7.7? Yo lo veo bastante estable y me va muy bien, sol pues cuestión de probar, en otx2 yo a medida que lo usaba cuando salió fui reportando errores. Fueron pocos. Lo mismo debería ser para esta.. y es algo más nuevo. X) @tarantonio dale un voto de confianza.

@tarantonio
Copy link

tarantonio commented Apr 26, 2017

Tiene varios problemas el pasar a OTX3, para empezar no es compatible retroactivamente, con lo cual todo mi trabajo es en vano.
Luego tiene bugs, puedo vivir con los bugs que quedan en otx2 e intentar solucionarlos, pero no me apetece empezar de nuevo con OTX3 y un montón de sorpresas sin resolver.

Es el único bug importante que queda en OTX2, pero es realmente molesto y te impide crear un server en condiciones. Es como tener un server perfecto y con este bug no puedes usarlo.
¿porque no tratamos de resolverlo?

@brewsterl
Copy link
Contributor Author

Con las oop classes si es compatible muchas cosas pero igual no me importa eso, en unas semanas igual empezaré a trabajar un datapack al estilo de realotx pero con otx3, y ya verás como muchos se van a cambiar.. solo es cuestión de e dedicar algunas horas y se que tengo varias personas que me ayudaran. @tarantonio

@tarantonio
Copy link

Yo es que ya tengo el datapack para otx 2 casi al 100% y es un tedio volver a empezar en otra distribución.
No creo que sea algo insalvable poder resolver esto. Es más fácil que empezar de cero.

@brewsterl
Copy link
Contributor Author

@tarantonio ya conseguí una solucion para esto, cuando tenga la pc al 100% ya que perdí todo, subire el fix.. requiero hacer muchos cambios pero si funcionó lo que pensé x) espera unos dias..

@mattyx14
Copy link
Owner

mattyx14 commented May 5, 2017

Muy bien

@tarantonio
Copy link

Espero el fix @brewsterl , gran trabajo!

@Felipe1993
Copy link

creo qu esta solucion que postie tmb deberian subirla son solo unos cuantos npcs... si quieres yo te los paso... lo mismo paso con el cavethings y fue un buen parche no se porque no toman mi solucion en cuenta.... saludos

@mattyx14
Copy link
Owner

Dame la referencia de el ajuste para probarlo

@Felipe1993
Copy link

Probaste con los scripts únicos del servidor? - Eso para ver si los NPC no tienen ningún problema por que veo que se mueven muy rápido.

Pero para que el sistema de NPC responda a muchos players a la vez ocupas ir a:
\data\npc\lib\npcsystem\modules.lua

-- Constants used for shop mode. Notice: addBuyableItemContainer is working on all modes
SHOPMODULE_MODE_TALK = 1 -- Old system used before Tibia 8.2: sell/buy item name
SHOPMODULE_MODE_TRADE = 2 -- Trade window system introduced in Tibia 8.2
SHOPMODULE_MODE_BOTH = 3 -- Both working at one time

-- Used in shop mode
SHOPMODULE_MODE = `SHOPMODULE_MODE_TALK``
Eso debe solucionar le hablado múltiple.

@mattyx14

pero como hacer para que muchos npcs le respondan a 1 solo player correctamente??

vamos que ya casi no quedan bugs

@Cjaker
Copy link
Collaborator

Cjaker commented Mar 30, 2018

Hmmm.. seems interesting.

@Felipe1993
Copy link

@mattyx14 A los npcs problema en lua se le pone
local keywordHandler = KeywordHandler:new() local npcHandler = NpcHandler:new(keywordHandler) NpcSystem.parseParameters(npcHandler) npcHandler.talkRadius = 1

evita lo del rango parche de agua
pero aun queda esto de hi hay que decirle buy o sell y asi no deberia ser

@pasturryx
Copy link

@tarantonio ya conseguí una solucion para esto, cuando tenga la pc al 100% ya que perdí todo, subire el fix.. requiero hacer muchos cambios pero si funcionó lo que pensé x) espera unos dias..

hola tambien tengo este error, como solucionaste esto?

@pasturryx
Copy link

bump @brewsterl
bump

mattyx14 added a commit that referenced this issue Oct 8, 2018
@mattyx14
Copy link
Owner

mattyx14 commented Oct 8, 2018

ya hice un correctivo a ver si ello soluciona este problema

@pasturryx
Copy link

@mattyx14 17:26 GM Legacy: hi
17:26 Xodet: Hello GMLegacy. I sell mana potion, life potion, blank runes.
17:26 Sam: GM Legacy! I am already talking to you...

el error aun se mantiene

@mattyx14
Copy link
Owner

El error donde se quedaba si decir bye aunque estuvieras cambiando de pisos era el importe ... Voy a ver eso del multiablado

@pasturryx
Copy link

okas re-abre este error para que si alguien lo ve y sabe como, comente como lo arreglo

@mattyx14 mattyx14 reopened this Oct 16, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

7 participants