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

Disabling skill rust #2

Closed
random-cdda-modder opened this issue Apr 15, 2023 · 6 comments · Fixed by #3
Closed

Disabling skill rust #2

random-cdda-modder opened this issue Apr 15, 2023 · 6 comments · Fixed by #3

Comments

@random-cdda-modder
Copy link

random-cdda-modder commented Apr 15, 2023

You can take the Good Memory trait, call it "Disable skill rust", make it 0 points and make the multiplier 0.

If that won't work because someone is dividing by the multiplier, make the multiplier 0.00000000000000000000000001

I have not tested this at all.

https://github.com/CleverRaven/Cataclysm-DDA/blob/13f8d81b4830226ad24345d0dcf1b00fc3fd5dad/data/json/mutations/mutations.json#L855

{
    "type": "mutation",
    "id": "GOODMEMORY",
    "name": { "str": "Disable Skill Rust" },
    "points": 0,
    "description": "Your skills will not erode.",
    "types": [ "MEMORY" ],
    "skill_rust_multiplier": 0,
    "starting_trait": true,
    "valid": false,
    "cancels": [ "FORGETFUL" ]
}
@BananaBonanza
Copy link

I made the above edits to a copy of Experimental Build 2023-04-15-0610. Initial testing looks like this works without any 'divide by zero' errors (that I've seen so far, anyway).

Made a new character, did not select the new trait and began playing with 1 point in every skill. After waiting two days everything had started to degrade. Skills were now at level 0 with percentages in then 90s. Debug'd the character to have the trait, trained up athletics via the "practice" craft, and then waited several more days. No skill rust after that point.

@neonspectra
Copy link
Owner

Thanks for this! I think the path forward with this one is to try merging into CleverRaven as a debug option, since that would make it easier on everyone involved imo.

I created a new PR in CleverRaven as a debug option...

  • If it gets merged, I'll include steps on how to activate it in-game in the guide.
  • If it doesn't, I'll add a mod to add this mutation in and document how it works

BTW, @random-cdda-modder, I was going to add you as a co-author, but it seems you just created your GitHub account and do not have any commits that I can grab your committer signature from.

If you provide your GitHub NoReply commit address, I'd be happy to add a commit with you as a co-author.

@captainsawbones
Copy link

Hey! I saw your commit got shot down, so I figured I'd take @random-cdda-modder 's idea and package it into its little own baby standalone mod to make life a little bit easier for you guys. I made it clear that RCM' was the one who came up with the idea in the modinfo and the readme, and i'll link you to my repository. I hope that's okay (contributing properly to things /collaborating on github is new to me and I don't really know the proper way to do things yet.) here's that repository

@random-cdda-modder
Copy link
Author

I hereby relinquish all authorship and copyright for anything related to this issue, feel free to do what you want without even attribution or anything. Ideas are free, execution is golden

neonspectra added a commit that referenced this issue Apr 19, 2023
Add documentation based on #2 for how to disable skill rust.
@neonspectra neonspectra linked a pull request Apr 19, 2023 that will close this issue
@neonspectra
Copy link
Owner

@captainsawbones Thanks! I really appreciate your contribution. I added some documentation to this repo pointing to your mod along with the explanation of how it works.

Welcome to collaborating on GitHub-- it's really daunting and unintuitive and I had a hard time with it too when I started. There's a reason they call git the stupid content tracker 😉 .

BTW I sent over a pull request to your repo with some suggestions. Please feel free to review at your convenience if they're things that you want to put in.

neonspectra added a commit that referenced this issue Apr 19, 2023
Add documentation based on #2 for how to disable skill rust.
@neonspectra
Copy link
Owner

Thanks everyone for all the contributions on getting this one documented!

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

Successfully merging a pull request may close this issue.

4 participants