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

Alcohols unintentionally have only very brief potion effects #1

Closed
eggdropsoap opened this issue Apr 17, 2020 · 1 comment
Closed
Assignees

Comments

@eggdropsoap
Copy link
Owner

eggdropsoap commented Apr 17, 2020

Before commit 61d2f3e, binnie.core.liquid.AlcoholEffect.makedrunk() was being passed a hardcoded 2.1f for "strength" (at binnie/extratrees/items/ItemDrink.java, line 143) regardless of the alcohol item used. After 61d2f3e, alcohol fluids' ABV are passed instead, which range from 0.05f to 0.8f. The code in makedrunk() was tuned for that strength of 2.1f, which is the current equivalent 210% ABV. Passing fluid ABVs now instead means the time and amplifiers for potion effects used to represent drunkenness are too brief to cause visible effects in most cases, varying from 1s for beer, with a max of 4s for neutral grain spirit (the minimum for the Nausea filter to briefly appear).

The code in makedrunk() can be roughly rescaled for the ABVs now passed in by multiplying ABV by 100 to get effective strength, before plugging strength into the existing potion effect time calculations in AlocholEffect.makedrunk(). This results in a time range for effects from 11s (ABV 0.05) to 45s (ABV 0.8).

@eggdropsoap eggdropsoap self-assigned this Apr 17, 2020
@eggdropsoap
Copy link
Owner Author

Fixed in 49a2379

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