Jump to content

  •  

Bug Tracker Migration

June 3rd
Good news everyone! The staff has decided that it is time to slowly kill off this Bug Tracker. We will begin the process of slowly migrating from this Bug Tracker over to our Github Issues which can be found here: https://github.com/HerculesWS/Hercules/issues

Over the next couple of days, I will be closing off any opportunity to create new reports. However, I still will keep the opportunity to reply to existing Bug Reports. Doing this will allow us to slowly fix any bug reports we have listed here so that we can easily migrate over to our Issue Tracker.

Update - June 7th 2015: Creating new bug posts has been disabled. Please use our https://github.com/HerculesWS/Hercules/issues tracker to post bugs. Users are still able to reply to existing bug posts.

- Administration

Issue Information

  • #006506

  • 4 - High

  • Confirmed

Issue Confirmations

  • Yes (5)No (2)
Photo

EDP + Cross Impact

Posted by Hercules Bot on 19 August 2012 - 05:32 PM

Originally posted by mofo
Updated to r16667, now EDP boosts damage of cross impact, sonic blow, soul breaker, etc. Was this change intentional? TIA

Originally posted by VodkaMartini
confirmed on Cross Ripped Slasher, Magnum Break, Meteor Assault and Phantom Menace Skill. (r16666)

Attachment shows Magnum Break + EDP. w/o EDP damage = ~< 2k

Edited by VodkaMartini, 19 August 2012 - 07:26 PM.


Originally posted by mofo
it's actually worse now because it raises ATK prior to boosting final damage by 400%. I'l try to get screenshots later. i made sure i had renewal.h set to renewal edp.

Originally posted by Poseid0n
I confirm this bug. EDP do too much damage. It one shoot all players.

Do you can, please, fix it ?

Originally posted by zamorasantos
yes its a bug

Originally posted by kyeme
Yeah too much damage :( .. i disabled Renewal EDP on my server :(

Originally posted by Tepoo
Tested on official Server. Cross Impact doesnt increase the dmg when edp is activated.

Originally posted by RyokoMVP
but here edp give 400% final atk lol...i already posting about this before...but they dont care...

Originally posted by Euphy
Is anyone working on this?  This is a major issue on Renewal servers...

Originally posted by kyeme
Thank you for confirming this. :)

Originally posted by GM Takumirai
I dont know if this is the correct fix http://rathena.org/b...694#entry135694

Originally posted by malufett

Is anyone working on this?  This is a major issue on Renewal servers...

ok let me do it..I'll do a remedy even though renewal atk is not yet fully implemented..

:meow:

Originally posted by Angezerus
I don't mean to push anyone, but why does nobody start that renewal atk implementation? It's a problem since 3-4 months, I already posted formulas and links about the problem, but noone does anything. This should be the first thing... There's no point in fixing skills all the time if the base attack is wrong. This edp thingie is starting to really piss me off. There's about 10 to 20 reports about this. Please do something, or revert to the old 400% atk. Anything would be better!

But theres a problem with the skill as well:
Currently we increase the weapon atk and base atk as well (as intended), but in the meantime we increase the final attack by 400% (as it was done in the old formula). THIS is the problem wit the skill.... This is a duplicate bonus. Pz make the 400% bonus only pre re.
Of course the renewal formula needs renewal attack, but as you see it's an error in the skill as well not only the missing re atk...

Here's the source:

Renewal bonus part:
#ifdef RENEWAL_EDP
// renewal EDP increases your weapon atk by watk x Skill Level - 1
if( sc->data[SC_EDP] && sc->data[SC_EDP]->val1 > 1 )
  watk = watk * (sc->data[SC_EDP]->val1 - 1);

This applies to renewal as well, making the bonus extremely OP
case SC_EDP: // [Celest]
   val2 = val1 + 2; //Chance to Poison enemies.
   val3 = 50*(val1+1); //Damage increase (+50 +50*lv%) <<<< SHOULD BE ONLY PRE RE

Edited by Angezerus, 10 September 2012 - 09:29 AM.


Originally posted by malufett
this should be the fixed [rev='16770']..
however due to the current atk formula damage is still quite huge...but not that big as before...

I don't know If I should flag all reports as fixed...anyway bug me if its ok to change all to fixed...

:meow:

Originally posted by Angezerus
Thanks a lot :3

Originally posted by kyeme
Thanks for d fixed :). May I ask something?
My normal attack is = 60k and my Cross Impact = 109K is this normal? or there was something wrong about bDefRatioAtkRace (Pierce Defense) ? Im using Super Latest Revision. Here's my Screenshot

Screenshot 1: Sura with Angelus and Revitalize Alive Total Def = 433 + 5 and Cranial Shield. And Venon Impression

Screenshot 2: Equipment of GX = Ice Pick with Turtle General Card

Screenshot 3: Normal Attack of GX = 60k

Screenshot 4: Cross Impact of GX = 109k each

Edited by kyeme, 10 September 2012 - 07:11 PM.


Originally posted by malufett
@kyeme
its normal in our current formula..but its not normal as we can see..
anyway this is how it makes the damage still huge..
are current formula or the PRE flows

Base Damage->Skill Modifiers->Damage Reduction->Masteries->Elemental->Card Bonuses


therefor the 'bDefRatioAtkRace' bonus is amplified(suppose to be wrong) where it should be added as fixed damage..

:meow:

Originally posted by donkeyg
any solution on this?

Originally posted by donkeyg
this was happened in my server. 150 max level. and no god item at all but they still able to cross impact 99999999999 dmg on every MVP.... and i don think even if i got MG/ mjolnir can deal such a over damage



Edited by donkeyg, 17 November 2012 - 03:29 AM.


Originally posted by donkeyg
bump!

Originally posted by GM Takumirai
any further solution