HamaIndustries / FEMultiPlayer-V2

Reupload & revamp of FEMP with fresh bugfixes... usually. Check "releases" for download.
http://eliatlarge.github.io/FEMultiPlayer-V2/
GNU General Public License v3.0
27 stars 31 forks source link

Unnerf Staves / Make Staves use correct healing formula #85

Closed ghost closed 8 years ago

ghost commented 8 years ago

Issue type: problem

Description:

As per here, here, here, and here it seems that the sub in consensus that staves should be using their correct healing formulas of 10 + Mag for Heal, 20 + Mag for Mend, and 10 + Mag for Physic (range on Physic btw is (1 ~ Mag/2).

As per @eliatlarge 's comment on https://www.reddit.com/r/FEMP/comments/4hm2mm/why_does_mend_larachel_only_heal_17_not_34/ I am opening this as an issue.


References:

http://fireemblem.wikia.com/wiki/Heal http://fireemblem.wikia.com/wiki/Mend http://fireemblem.wikia.com/wiki/Physic

HamaIndustries commented 8 years ago

It appears that the healing value was mag/2 + healvalue/2, for some reason.

rayrobdod commented 8 years ago

Damage formulae: 8a4854c86e3e91821644f4e8cbe7f7d44c7d87d7

ghost commented 8 years ago

@rayrobdod Does this also fix Physic's range to be from 1 to Mag/2 (rounded up to the next whole number if a fraction), so it won't be considered OP to some?

rayrobdod commented 8 years ago

Was that also a part of this issue? The commit does not appear to change Physic's range, but the relation of the first half probably justifies linking this issue and that commit.

HamaIndustries commented 8 years ago

I didn't know that was requested. I'll look into it.

ghost commented 8 years ago

Range correction wasn't requested yet, but probably will be given Physic's potential. Can always ask the sub to make sure but I think they'll be on board with it so that it doesn't overshadow Mend and Recover too easily.

HamaIndustries commented 8 years ago

oh.

well, i guess its a beautiful new feature now then. I can see this messing with 1 level healers, since they'll have significantly debuffed range.

ghost commented 8 years ago

I can see this messing with 1 level healers, since they'll have significantly debuffed range.

Good; If they want to make use of Physic, they need to properly invest in their healer. That was a cheap strat.

HamaIndustries commented 8 years ago

now I need to make it actually work

I really need to get back to testing before committing

ghost commented 8 years ago

Sorry, I would've tested that for you but I've been busy testing out #87

HamaIndustries commented 8 years ago

ok well it works now so lets pretend that nothing happened, and ray can keep pretending like I'm a good coder