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

Increase tall lizard density #1103

Closed
wants to merge 1 commit into from

Conversation

Leander-0
Copy link
Contributor

About the PR

Makes lizards with the tall trait have more "mass" again.

Why / Balance

The mass was originally removed due to mass-contest issues and how it affected mostly combat with shoves insta stunning people with 1 or 2 shoves, but since it was removed it will not longer affect players with less mass.

On terms of what will affect on balance.

Pros:
More easy to pull crates and objects.

Cons:

  • Harder to fit on tight spaces (already with the bigger size).
  • Harder to move on low gravity areas or space since inertia will affect them more due to larger mass.
  • When on critical or dead condition will be much harder to pull around and drag.
  • Easier target to shoot at.

Technical details

2 lines of YML

Changelog

🆑 Leander

  • tweak: Tall lizards got into a new diet and all of them have gained more weight.

@dvir001
Copy link
Contributor

dvir001 commented Mar 9, 2024

No
Trait should never be a "click for buff"
Also this is bullshit as the hitbox stay the same

@Leander-0
Copy link
Contributor Author

Leander-0 commented Mar 9, 2024

No Trait should never be a "click for buff" Also this is bullshit as the hitbox stay the same

What is the buff? moving crates more easily? then honestly at this rate remove all the trait sizes then and make all species equal, tell me why then vulpkanins can "click for buff" for being small fit inside dufflebags and backpacks too since thats pretty much a real buff.

@Mnemotechnician
Copy link
Contributor

Mnemotechnician commented Mar 9, 2024

Also this is bullshit as the hitbox stay the same

False. All people have circular fixtures, whose mass depends on their density and radius: (Pi * radius^2 * density) kg. The tall trait affects both the radius of the fixture and its density.

@dvir001
Copy link
Contributor

dvir001 commented Mar 9, 2024

Also this is bullshit as the hitbox stay the same

False. All people have circular fixtures, whose mass depends on their density and radius: (Pi * radius^2 * density) kg. The tall trait affects both the radius of the fixture and its density.

So the small trait buff you with a smaller hitbox?

@Mnemotechnician
Copy link
Contributor

So the small trait buff you with a smaller hitbox?

you pay a big price for it, and it's hardly a buff :p

or at least used to, before mass contests were removed - the attempt to "improve balance" broke the balance

@MagnusCrowe
Copy link
Contributor

Does this not also increase the amount of biomass needed for cloning?

@github-actions github-actions bot added the Status: Needs Review This PR is awaiting reviews label Mar 28, 2024
@VividPups
Copy link
Contributor

Does this not also increase the amount of biomass needed for cloning?

I remember Bigger things required more for cloning in the past

@dvir001
Copy link
Contributor

dvir001 commented Apr 14, 2024

I'm hopping, we can get size slider added then have some actual code based around it for mass later.
For now, I think we should not mess with any size related stuff till we can add a system that isn't trait based to replace it.
Said system will also need support to enable the "felinid" go into bags under X scale for vulps if its still needed.

@dvir001
Copy link
Contributor

dvir001 commented Apr 14, 2024

Closing this till someone add a new size system or just use the Cosmatic Drift one since we got Lank approval to do so anyway.

@dvir001 dvir001 closed this Apr 14, 2024
@Leander-0 Leander-0 deleted the lizard-mass branch June 14, 2024 03:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants