Cataclysm-DDA icon indicating copy to clipboard operation
Cataclysm-DDA copied to clipboard

REMOVE SKILL RUST FROM GAME OPTIONS

Open sonphantrung opened this issue 2 years ago • 15 comments

Summary

Infrastructure "REMOVE SKILL RUST FROM GAME OPTIONS"

Purpose of change

Resurrect #55173 Original desc: Since skill rust was transformed into a useful and important gameplay mechanic in https://github.com/CleverRaven/Cataclysm-DDA/pull/50176, there is no good reason to keep it in the game debug options.

Describe the solution

Basically the same thing as the former repo (can't playtest rn, machine has no good capability of compilation)

Describe alternatives you've considered

NONE

Testing

The code is practically the same, so test results should be the same

Additional context

  • Doing git apply/ git am is a PAINFUL process, so I basically just copied the same code ;)

sonphantrung avatar Aug 04 '22 14:08 sonphantrung

What's a reason to cut out stuff out of debug options?

RelMayers avatar Aug 04 '22 22:08 RelMayers

It's a name of the section of the options, not legitimate debug options.

anothersimulacrum avatar Aug 04 '22 22:08 anothersimulacrum

"What's a reason to cut out a option out of debug options?" Do we want player unable to customise their gameplay in any other way than just making a fork?

RelMayers avatar Aug 04 '22 23:08 RelMayers

"What's a reason to cut out a option out of debug options?" Do we want player unable to customise their gameplay in any other way than just making a fork?

If someone cared enough they could just make a mutation that allows you to not have skill rust. It's not something you need to make a whole fork over

Holli-Git avatar Aug 04 '22 23:08 Holli-Git

What's the reason to have 3 different methods for a system to work when we only actually care about and want one, and so the others only impose a maintenance burden?

This option existed because skills rust was a thing we wanted, but was not implemented in a satisfactory way. This mechanic is implemented in a satisfactory way. There is no reason for this option to exist, and it existing just means it diverts problems to people changing it instead of reporting the problem.

anothersimulacrum avatar Aug 04 '22 23:08 anothersimulacrum

INT based skill rust was nice for some runs, currently it has been removed completely. Removing option might cause people diverting from DDA to BN instead of them reporting problems. Also is there need to test everything in experimental at once?

RelMayers avatar Aug 05 '22 00:08 RelMayers

  • Unless you’re playing w/ StatsThroughSkills, you don’t necessarily have to constantly regrind your skills constantly to unrust it: Why bother constantly regrinding skills when you don’t need to? Tbf, you’ll never forget the ability to craft things, interacting w/ vehicles w/ the new system
  • My only issue w/ Skill Rust here is that: Many actions aren’t calculated properly w/ theoretical skills (Computers for example, hacking on computers while having 0 practical and 6 theoretical should take theoretical skills into calculation, instead of just calculating practical skills, since I got some ideas on Hacking after all!)

sonphantrung avatar Aug 05 '22 04:08 sonphantrung

But i like the capped option 😥

Mo-rang avatar Aug 05 '22 05:08 Mo-rang

And finally, after #58851 got merged into the base game, you now have LOTS of time to regain your skills if you want to

sonphantrung avatar Aug 05 '22 08:08 sonphantrung

Could you address concerns rather than ignoring them all together?

RelMayers avatar Aug 05 '22 15:08 RelMayers

  • Unless you’re playing w/ StatsThroughSkills, you don’t necessarily have to constantly regrind your skills constantly to unrust it: Why bother constantly regrinding skills when you don’t need to? Tbf, you’ll never forget the ability to craft things, interacting w/ vehicles w/ the new system
  • My only issue w/ Skill Rust here is that: Many actions aren’t calculated properly w/ theoretical skills (Computers for example, hacking on computers while having 0 practical and 6 theoretical should take theoretical skills into calculation, instead of just calculating practical skills, since I got some ideas on Hacking after all!)

I do think both of those issues should be addressed in any fix being merged. Stats through skills because it's a mod that is packaged with the game and if this causes wonky behavior in it that's an issue. And honestly computers is currently broken by skill rust in my playthroughs. I'm fine with removing options but those two items should be fixed before or at the same time we are removing the options.

An easy fix for stat's through skills would be to shift the stat gain from practical to theoretical.

Maleclypse avatar Aug 05 '22 18:08 Maleclypse

Is the skill training speed option being removed as well? I tend to like it as I find current skill gain unrealistically fast.

chrispikula avatar Aug 05 '22 18:08 chrispikula

I'm not sure that's necessarily incorrect behavior for stats through skills, though it may warrant nicer handling.

anothersimulacrum avatar Aug 05 '22 19:08 anothersimulacrum

As the originator of the rust change, I would suggest that we do this after stable. Doing it now raises the possibility of having to deal with presently minor rust issues that then may get elevated to blocker level. Since all the listed issues are not severe enough to block IMO, I'd prefer to let people disable them with the option and not have them slow down release.

@chrispikula digressing, but I do plan on dramatically slowing skill gain after stable. I don't want people to conflate the rust change with skill gain changes like they conflated crafting time changes with proficiencies.

I-am-Erk avatar Aug 06 '22 03:08 I-am-Erk

I play without skill rust becaue I disagree with the concept entirely.

No amount of modifications to this mechanic (making it faster/slower/capped/uncapped/statbased/...) will make this change.

I fully intend to continue doing so for the entire observable future, regardless of any misguided game design decisions.

If you make it harder or impossible for me to do so, I will absolutely find a technical way to overcome whatever you come up with, so feel free to try.

x-qq avatar Aug 06 '22 19:08 x-qq