Skip to content
This repository has been archived by the owner on Dec 4, 2020. It is now read-only.

Remove automatic multipliers when mob is marked as an NM #756

Merged
merged 2 commits into from Jun 24, 2020

Conversation

ibm2431
Copy link
Contributor

@ibm2431 ibm2431 commented Jun 22, 2020

This was entirely made up.

NMs which have been CONFIRMED to have HP values that aren't in line with the standard growth scale should have their HP manually set instead.

I affirm:

  • that I agree to Project Topaz's Limited Contributor License Agreement, as written on this date
  • that I've tested my code since the last commit in the PR, and will test after any later commits

…s entirely made up. NMs which have been CONFIRMED to have HP values that aren't in line with the standard growth scale should have their HP manually set instead.
@TeoTwawki
Copy link
Contributor

There was a 1.5x stat multiplier that was made up too, dunno if still exists

@cocosolos
Copy link
Contributor

I didn't see any stat bonuses but there is also an MP bonus applied right after this, dunno if you want to get that as well.

@ibm2431
Copy link
Contributor Author

ibm2431 commented Jun 23, 2020

I was originally tempted to remove the stat multiplier, but expected its removal to be met with resistance, so stuck with just the HP since there's a specific config setting I could point to.

But if we all want to remove the stat multiplier too, I will happily delete it.

I didn't see the MP bonus, I'll delete that too. (Although "mob MP" will be a discussion for another time.)

edit: Oh, there's an NM stat multiplier in map.conf already. So out the vanilla stat multiplier goes.

@ibm2431 ibm2431 changed the title Remove automatic HP multiplier when a mob is marked as an NM Remove automatic multipliers when mob is marked as an NM Jun 23, 2020
@cocosolos
Copy link
Contributor

Didn't even see those multipliers next to the conf settings, good eye!

@cocosolos cocosolos added the merge ready reviewed and deemed ready to merge label Jun 23, 2020
@ibm2431 ibm2431 merged commit ae2583f into release Jun 24, 2020
@zircon-tpl zircon-tpl deleted the remove-false-hp-boost branch July 1, 2020 20:29
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
merge ready reviewed and deemed ready to merge
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants