Noita
Anvil of Destiny
Reviewing wand buffing formula
  1. (bug?) About the limit_buff subfunc, instead of
    value = threshold + value * scale
    , I think what you meant to do is
    value = threshold + (value - threshold) * scale
    to smooth the curve. Currently if the bonus_mana is 51, it will get "limited" to 60.2.
  2. (suggestion) I think buff_amount should be taken into account in manaChargeSpeed calc as well. For example as simple as
    wand.manaChargeSpeed = wand.manaChargeSpeed + Random(45, 60) * 2 * buff_amount
    . Great for those who tweak config values
  3. (documentation) Maybe you should clarify that during wand merging, the fact that one of the castDelay, rechargeTime or spread is getting nerfed while other two are getting buffed is by design (and that this feature is not applied when buffing the wand with tables - no stat nerf when tablet buffing). This was the reason why I looked into the code as well, as I got surprised that my wands with both 0.7 cast delay got merged into a 0.12 cast delay (even though I've tweaked config_regular_wand_buff value for myself).
Last edited by Gentlemad Scientist; 8 Nov, 2020 @ 8:45am
< >
Showing 1-3 of 3 comments
Horscht  [developer] 8 Nov, 2020 @ 10:30am 
You're right, I totally missed that, thank you.

I also don't remember why I didn't include mana charge speed in the buff amount scaling.

The reason for the reduction of 1 stat was to make it a little more balanced instead of just straight up better in every way. I decided to leave it out of the mod description because I thought it would make the explanation too long and complicated. Do you think it should stay that way to get more variation, or should I get rid of the reduction?
Hmm its hard to say if you should get rid of the reduction or not. On one hand, it's definitely a good balancing mechanic since the mod is already being somewhat op, but on the other hand some people may expect to have their wands strictly upgraded. This is why I mentioned that it's maybe just a documentation problem.

Also, knowing about stat reduction may discourage merging in favor of tablet buffing, once you get a good main wand
Last edited by Gentlemad Scientist; 8 Nov, 2020 @ 7:49pm
Horscht  [developer] 9 Nov, 2020 @ 8:41am 
Okay I pushed an update and added that little info into the description (for the people who actually read it).
< >
Showing 1-3 of 3 comments
Per page: 1530 50