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

  • #004718

  • 3 - Medium

  • Confirmed

Issue Confirmations

  • Yes (5)No (0)
Photo

MaxHP & MaxSp - Expanded Classes

Posted by Hercules Bot on 18 January 2011 - 11:33 PM

Originally posted by Alair
http://www.eathena.w...er&showbug=4718

In eAthena MaxHP and MaxSP for TAEKWON,  STAR GLADIATOR, LINKER, GUNSLINGER and NINJA calculating linearly, but:

Aegis HP table: http://up.5cm.ru/u/zJNN/maxhp3.scp
Aegis SP table: http://up.5cm.ru/u/z3QK/maxsp3.scp

does not match.

Sorry my english. Please, fix.

Originally posted by Ind
moving to db bug tracker -- not a core issue.

Originally posted by Kenpachi
The current database layout doesn't allow to realize this because the HP is calculated by HPFactor and HPMultiplicator in job_db1.txt.

Originally posted by Kenpachi
We have to create a new layout. I can't accept that we're not able to be officia in such an important thing.

Originally posted by Daegaladh
I did the whole system some time ago :3
I'll post it later because I have to leave now, sorry

Originally posted by Daegaladh
Here's the diff for status.c (rev r15734) http://pastebin.com/qzyFrkNb

And here the db files:
The real only difference is on ninja & guns HP, the rest is on Munak/Bongun classes (yep, they are on aegis files XD) and aren't relevant, so you can delete those classes and join job_maxsp files into one.

pre-re/job_maxhp.txt http://pastebin.com/PHSB3F5T
pre-re/job_maxsp.txt http://pastebin.com/C3QVMgts

re/job_maxhp.txt http://pastebin.com/CvyN34hF
re/job_maxsp.txt http://pastebin.com/EycnqLff

Edit: Oops! I forgot: You have to remove HPFactor,HPMultiplicator & SPFactor fields from job_db1.txt too

Edited by Daegaladh, 20 March 2012 - 12:18 AM.


Originally posted by Kenpachi
I'll check this. But since we're in a major organzation discussion it will may take some time.
Thanks, Daegaladh. :)

Originally posted by Kenpachi
The DB files Daegaladh provided look good to me.
Please check the source diff and implement this update, (Or your version if you don't like the provided one.)

-> Moving this to Core.

Originally posted by Ind
this have to be thought a bit further, this certainly takes changing the max level to a higher level of difficulty

Originally posted by Kenpachi
We could provide custom values that goes up to level 999/1000 like we did in exp.txt and simply remove the level column from the layout and just read as many values as max level is set to in exp.txt.
(Putting those values within exp.txt and rename it to a more fitting name is a solution, too but somehow I don't like it.)

Originally posted by Daegaladh
Here's an updated (and fixed) version of my patch: http://pastebin.com/2sDnEnk2 (rev 16347)
Tested and working :3

Also I added Expanded Super Novice, Expanded Super Baby, Kagerou and Oboro to the tables, but since I have no info about their HP and SP, I used Rune Knight's values as dummy data, so you can do the same or add the real data if you have it.

Originally posted by Protimus
It seems to be functional, but maybe we can improve the code.

Originally posted by michieru
bump

there is some info too:
http://rathena.org/b...ray-vs-rathena/

Edited by michieru, 17 July 2012 - 05:30 PM.