discordia-space / CEV-Eris

Death is our destination
https://discord.gg/QfEg7K7
Other
152 stars 496 forks source link

Restrict combat mech users from entering with any suits/armors #8471

Closed XElectricX closed 6 months ago

XElectricX commented 7 months ago

About The Pull Request

What it says on the tin. Anything in the suit slot will prevent the player from entering a mech. This only affects mechs with combat and reinforced chassis.

Why It's Good For The Game

Mech pilots should not be able to get away with wearing armor inside a bigger suit of armor.

Testing

It just works.

Changelog

:cl: balance: Cannot board mechs with combat or reinforced chassis if wearing armor. /:cl:

Firefox13 commented 7 months ago

It just works.

Please provide a proper list and explanation of tests you have conducted. What armors you generally can and cannot wear? LIGHT_SLOWDOWN restricts mech users to only basic vests, which are really, really bad.

XElectricX commented 7 months ago

It just works.

Please provide a proper list and explanation of tests you have conducted.

Quite literally just entering a mech to see if the restriction works. And it does.

What armors you generally can and cannot wear? LIGHT_SLOWDOWN restricts mech users to only basic vests, which are really, really bad.

You just answered your question, homie.

assortedbeads commented 7 months ago

This pr allows something like the NT void (a high tech combat void) to be used in a mech while a hazard suit with 7, 7, 10 is denied because it has higher slowdown.

Maybe have a second look at individual slow down values because they can vary a lot.

assortedbeads commented 7 months ago

This pr allows something like the NT void (a high tech combat void) to be used in a mech while a hazard suit with 7, 7, 10 is denied because it has higher slowdown.

Maybe have a second look at individual slow down values because they can vary a lot.

Going further on this, rather than using slowdown maybe you should look as how much armor the voidsuit has and use that as a cut off.

TheLeopold commented 7 months ago

Go further, restrict mech pilots from wearing anything in suit slot. That seems more logical And I'm dead serious

XElectricX commented 7 months ago

This pr allows something like the NT void (a high tech combat void) to be used in a mech while a hazard suit with 7, 7, 10 is denied because it has higher slowdown. Maybe have a second look at individual slow down values because they can vary a lot.

Going further on this, rather than using slowdown maybe you should look as how much armor the voidsuit has and use that as a cut off.

I have looked through the files. It was why I did it with LIGHT to begin with, because even that category has some really extreme armors. But that's a balance issue with the armors; not about to touch every single one of them as that's out of scope and I've rarely experienced PvP here, so I'm not qualified.

I was using armor values at first but those are also all over the place. Slowdown seemed to be the best way to generalize.

Go further, restrict mech pilots from wearing anything in suit slot. That seems more logical And I'm dead serious

This was what I was going to do but Wasteland told me the paramedics needed their suits so it'd screw them over. 🤷

Although since it only affects combat mechs now, might be viable to do.

Humonitarian commented 7 months ago

It just works.

Please provide a proper list and explanation of tests you have conducted. What armors you generally can and cannot wear? LIGHT_SLOWDOWN restricts mech users to only basic vests, which are really, really bad.

That's a good thing.

By forcing mech users to use light armors, we prevent mechs from functioning as a second health bar.

Using slowdown however can be considered questionable, it will prevent the use of most if not all space-worthy suits from being worn, and will allow the crusader armor to be worn despite being top tier.

TheLeopold commented 7 months ago

This was what I was going to do but Wasteland told me the paramedics needed their suits so it'd screw them over.

If paramedics are the only problem with this, then maybe snowflaking it is the best solution? As in "you can't wear armor except paramedic suit". Done.

assortedbeads commented 7 months ago

This was what I was going to do but Wasteland told me the paramedics needed their suits so it'd screw them over.

If paramedics are the only problem with this, then maybe snowflaking it is the best solution? As in "you can't wear armor except paramedic suit". Done.

Paramedics are not the only issue

Humonitarian commented 6 months ago

PR will be approved once the following test cases are checked and results provided:

You can use a mining mech without suffocating/dying to pressure RIG cannot be deployed while in a mech

assortedbeads commented 6 months ago

PR will be approved once the following test cases are checked and results provided:

You can use a mining mech without suffocating/dying to pressure RIG cannot be deployed while in a mech

Heat can also fuck you in mechs last I checked even when closed and atmos on

I still think weak voidsuits should be fine.

githubuser4141 commented 6 months ago

PR will be approved once the following test cases are checked and results provided:

You can use a mining mech without suffocating/dying to pressure RIG cannot be deployed while in a mech

First part isn't necessary because only combat/heavy mechs have this restriction, which do in fact work in nil pressure, and you should be able to put a suit on in light, powerlifter, walker, pod chests.

XElectricX commented 6 months ago

PR will be approved once the following test cases are checked and results provided: You can use a mining mech without suffocating/dying to pressure RIG cannot be deployed while in a mech

First part isn't necessary because only combat/heavy mechs have this restriction, which do in fact work in nil pressure, and you should be able to put a suit on in light, powerlifter, walker, pod chests.

What this fella said. I've added a proc to the deployment and toggling process in RIGs to check if the wearer is inside a mech that restricts armors. Works without issue as I tested on a powerloader, walker, and a combat mech.