Fehler #16854
geschlossen[Krieger] Sturmangriff - Falsche Endposition
Der Sturmangriff des Kriegers sollte nun immer die richtige Endposition berechnen.
Beschreibung
Ich bin im Source zufällig auf etwas gestoßen was mir beim spielen gar nicht aufgefallen ist.
Sturmangriff sollte den Charakter auf maximaler melee reichweite abstellen.
Dies tut es auf RG nicht.
Das Problem liegt wahrscheinlich an folgendem code in der SpellEffect.cpp:
void Spell::EffectCharge(SpellEffIndex /*effIndex*/) {
if (!unitTarget)
return;if (effectHandleMode SPELL_EFFECT_HANDLE_LAUNCH_TARGET) {
// Spell is not using explicit target - no generated path
if (m_preGeneratedPath.GetPathType() PATHFIND_BLANK) {
//unitTarget->GetContactPoint(m_caster, pos.m_positionX, pos.m_positionY, pos.m_positionZ);
Position pos = unitTarget->GetFirstCollisionPosition(unitTarget->GetObjectSize(), unitTarget->GetRelativeAngle(m_caster));
m_caster->GetMotionMaster()->MoveCharge(pos.m_positionX, pos.m_positionY, pos.m_positionZ);
}
else
m_caster->GetMotionMaster()->MoveCharge(m_preGeneratedPath);
}if (effectHandleMode SPELL_EFFECT_HANDLE_HIT_TARGET) {
// not all charge effects used in negative spells
if (!m_spellInfo->IsPositive() && m_caster->GetTypeId() TYPEID_PLAYER)
m_caster->Attack(unitTarget, true);
}
}
In diesem Video sieht man das der Spieler immer auf maximaler melee Range positioniert wird.
Von Franchizer vor fast 9 Jahren aktualisiert
Mist Video vergessen
-> https://www.youtube.com/watch?v=DXOCjIATv1c
Von Rushor vor fast 9 Jahren aktualisiert
- Zielversion wurde auf Klassen gesetzt
wotlk-quellen gibts auch dazu?
Von Franchizer vor fast 9 Jahren aktualisiert
-> Patch 2.3 https://www.youtube.com/watch?v=k_GGW3JiMu4
-> Pre Cata https://www.youtube.com/watch?v=DYAj_a_TjZc
-> Das bisher beste Video das ich gefunden habe^^ https://www.youtube.com/watch?v=tMNvYC8igWQ
Von Rushor vor fast 9 Jahren aktualisiert
float WorldObject::GetObjectSize() const { return (m_valuesCount > UNIT_FIELD_COMBATREACH) ? m_floatValues[UNIT_FIELD_COMBATREACH] : DEFAULT_WORLD_OBJECT_SIZE; }
nunja, für UNIT_FIELD_COMBATREACH einen wert finden der immer die maximale range zurückgibt
Von Rushor vor fast 9 Jahren aktualisiert
- Status wurde von Neu zu Testbereit geändert
neue distanz
meleerange wird nun mit einbezogen
auch gegen spieler testen. wenn ihr nicht wisst ob charge an stelle xy genauso buggy war wie nun auf dem master-testserver > austesten aufm live
Von sindorei vor fast 9 Jahren aktualisiert
Im PvP laggen sich die angechargten Targets eh in 1/3 der Fälle aus den Charges, spielt das auf und Krieger wird noch verkorkster :X
Von Rushor vor fast 9 Jahren aktualisiert
laggs haben nichts mit klassenmechaniken zu tun
Von Mollestus vor fast 9 Jahren aktualisiert
- Status wurde von Testbereit zu In Bearbeitung geändert
- Zugewiesen an wurde auf Rushor gesetzt
Auf normale Mobs etc scheint es zu passen, genauso wie Spieler.
Problematisch sind Bosse wie Prof/Bloodqueen etc:
zB: https://www.youtube.com/watch?v=HKzatWtnL8U&feature=youtu.be
Von Mollestus vor fast 9 Jahren aktualisiert
Sollte wieder öffentlich sein.
Vllt wegen der Hitbox einen festen wert nehmen? 5m oder so von der Mitte der Hitbox ausgesehen? kA ob das sonst machbar wäre oder man passt die Boss Hitboxen an :D
Von Rushor vor fast 9 Jahren aktualisiert
wenn wir die bosshitboxen anpassen gibts wieder flame, dass man an diese zu nah ranlaufen muss um meleehitten zu können
Von Franchizer vor fast 9 Jahren aktualisiert
Die Frage ist doch, was läuft in dem Video schief?
Du hast Prof angestürmt, doch konntest nicht angreifen,
dass heißt das Charge bei fetten Hitboxen nicht an MeleeRange rangeht.
Andererseits chargst du nie einen neutralen Raidboss an ;)
Das sieht dann wiederum so aus:
https://www.youtube.com/watch?v=lcphD8xcnPU
(Raidboss ist StunImmun und kommt deswegen noch ein Stück auf dich zu.)
Das ist somit schon ziemlich am Original aber doch noch nicht ganz.
Ein Test mit Archavon in Neutral zeigte, das Charge den Krieger 10m vor der Hitbox abstellt, und nicht 5m.
(Vielleicht hilft dir der Screen: http://www2.pic-upload.de/img/29355912/WoWScrnShot_010716_161943.jpg)
Was mich gerade Irritiert ist, das diese Änderung doch dann auch Abfangen des Furor Krieger beeinflusst.
Der Spell greift doch auf den selben Spelleffect zurück.
Fury's könnten also ziemlich angepisst sein wenn sie noch 5m zum Boss laufen müssten.
Von Mollestus vor fast 9 Jahren aktualisiert
Schon klar, dass die Bosse noch zu einem angelaufen kommen. Doch ging es mir vor allem darum, dass man dennoch nicht in Meelerange ist, was für PvP zB in BGs mit Fahrzeugen oÄ oder eben im PvE als DD bei solchen Bossen ziemlich doof ist, wenn man noch weiter vorlaufen muss.^^
Von Rushor vor fast 9 Jahren aktualisiert
unitTarget->GetObjectSize() only, ist also zu nah am ziel
unitTarget->GetMeleeReach() only ist leider auch zu am ziel
deshlab wird atm beides verwendet unitTarget->GetObjectSize()+ unitTarget->GetMeleeReach()
bei großen hitboxen gibts dann probleme
spontan würde mir nur einfallen:
uint32 Chargerange unitTarget->GetObjectSize() + unitTarget->GetMeleeReach()
Position pos = unitTarget->GetFirstCollisionPosition(Chargerange > xx ? 5 : Chargerange, unitTarget->GetRelativeAngle(m_caster));
demnach wäre es wie von euch beschrieben, wenn die gesamtrange größer als xx wäre würde der wert 5 genommen werden, ansonsten das resultat aus unitTarget->GetObjectSize()+ unitTarget->GetMeleeReach()
frage ist nun auf welchen wert wir xx festlegen sollen. bei welchen bossen ab welcher hitboxgröße kommt es überall zu problemem?
Von Rushor vor fast 9 Jahren aktualisiert
- Status wurde von In Bearbeitung zu Testbereit geändert
- Zugewiesen an
Rushorwurde gelöscht
kontaktpoint berechnung:
1.) unitTarget->GetObjectSize() + unitTarget->GetMeleeReach()
2.) wenn (unitTarget->GetObjectSize() + unitTarget->GetMeleeReach()) > = 9 yards entfernung == > 9 yards entfernung
das sollte die enorme distanz bei npcs mit großen hitboxen entfernen
Von Mollestus vor fast 9 Jahren aktualisiert
- Status wurde von Testbereit zu Ready geändert
- Update Text Deutsch aktualisiert (Vergleich)
Passt.
Von Rushor vor mehr als 8 Jahren aktualisiert
Bedeutung Live:
Der Bug wurde erfolgreich behoben und der Fix dazu befindet sich auf dem Spielserver. Sollte der Fehler weiterhin bestehen sollte ein neues Ticket ( https://redmine.rising-gods.de/projects/live/issues/new ) erstellt werden. Anmerkungen in dem alten abgewiesenen oder live genommenen Ticket werden ignoriert und gelöscht.
Von Daejiv vor mehr als 7 Jahren aktualisiert
- Zielversion wurde von Klassen zu Klasse - Krieger geändert
Von Left4Info vor mehr als 6 Jahren aktualisiert
- Tags wurde auf Classes, Warrior gesetzt