-
Notifications
You must be signed in to change notification settings - Fork 292
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
feat(balance): round 101~119g -> 100g #6014
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'd probably round to the nearest multiple of 5 instead of the nearest multiple of 100 myself. |
that'd also work, however these numbers were originally either
thus it'd cause no harm in e.g rounding 107g to 100g rather than 105g. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
119 and the like make much more sense to round to 120, and in general I think that nearest 10 is the more sensible approach for most items.
Additionally: M33 ball ammo is genuinely about that heavy, and thus would arguably be best served at 115 (an entirely realistic value).
again, there's no real reason plant marrow is 119g, no tangible benefit to rounding to arbitrary weight value.
this would be whether to choose realism vs gameplay abstraction (simpler weights). i'm indifferent on this one, could revert them if we want this level of precision but i doubt players want it |
I don't see how rounding to the nearest 10 is all that arbitrary. The entire basis of the metric system is base 10. It still makes things far more metric to round to that, and the numbers would likely be just as pleasing when displayed ingame. |
rounding from arbitrary values doesn't make sense. are there any real reason plant marrow must be 119g? it could have been any value, 118g, 104g, 93g... then why not just use 100g which rounds up nicely.
but 100g is better on eyes than 110g; 23 of 100g plant marrows is 2.30kg wheras 23 of 110g plant marrow is 2.53kg. again, to reiterate: if it doesn't really matter how heavy plant marrow should be, then it'd better be 100g which stacks nicely. |
There's no tangible benefit to doing any rounding. You're doing this to nitpick because you specifically dislike pounds and non-rounded numbers, Which is generally why I don't accept this, I leave it up to you to talk someone else into this. |
By that standard about 80% of the game's weights are arbitrary and rounding would never make sense except for the things that have a specifically defined weight such as chemicals... except then it doesn't because they have a specifically defined weight as it is. So when would you accept rounding other than to intervals of 100? |
there exists UI improvement, see #2505 and PR body:
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
After lengthy discussion in dev chat, and overall examining what's going on,
I don't approve of any rounding, but if we for some reason have to round for any reason, 10's works. I.E. 114 g goes to 110 g. I see no reason to round at all and upend weight as it subtly messes with balance especially on low strength / bad back players.
Purpose of change (The Why)
metricate item weights, part 2 (remember #4568?)
It's kind of the same reason they were 1/4 pounds etc. in the old days
lbs
was the default unit; so it made sense to use 1/4 pounds for weight since it would nicely display as0.25 lbs
on UI.now it's the opposite; since #2510 the default is kg, so
0.1 kg
display betterDescribe the solution (The How)
round 101 grams from 119 grams to 100 grams.
Describe alternatives you've considered
but again, no real reason they can't be nice and simple 0.1kg
Testing
Additional context
Checklist
Mandatory
closes #1234
in Summary of the PR so it can be closed automatically.main
so it won't cause conflict when updatingmain
branch later.