Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

updateLustValues is getting out of control with multiplications #275

Open
Linulanielly opened this issue Aug 1, 2020 · 0 comments
Open

Comments

@Linulanielly
Copy link

updateLustValues function heavily uses multiplication of factors to compute the prodFactor by which the new lust addition will be multiplied.
Sadly with addition of more and more multipliers the actual numbers are getting ridiculously huge. For example:
if PC is hasStatusEffect("Ludicrously Endowed"), hasStatusEffect("Red Myr Venom"), hasItemByClass(Savicite) (assume 3 items), hasStatusEffect("Dzaan Withdrawal") together they all give:
1.5 * 1.5 * (1.2 * 3) * 1.5 = 12.15 i.e. +1115% to the original productFactor
While if these numbers were to be added up:
0.5 + 0.5 + (0.2 * 3) + 0.5 = 2.1 i.e. +210% to the original productFactor

Could you please replace multiplications with additions? So the statement like this perk will result in +50% lust gain would be applied to the base lust gain? (right now it is applied to all lust gains combined)
Otherwise the characters with more and more different multipliers of their lust quickly get out of possible way to control their lust levels and I assume that there will be only more additions to the ways the lust can be increased in the future.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant