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

Remove Int/IntCap skill rust options and make Vanilla default #51327

Merged
merged 2 commits into from
Sep 3, 2021

Conversation

eltank
Copy link
Contributor

@eltank eltank commented Sep 3, 2021

Summary

Interface "Remove Int/IntCap skill rust options and make Vanilla default"

Purpose of change

The Int and IntCap rust options were not doing anything meaningful, other than influence Character::rust_rate() which actually has no impact on gameplay.
Also, skill rust provides benefits now so players should be encouraged to use Vanilla rather than Off.

Describe the solution

  • Delete Character::rust_rate()
  • Remove Int and IntCap rust options (migrating config files to Vanilla and Capped respectively)
  • Update the description of the skill rust option.
  • Make Vanilla the default for new games game installations

Describe alternatives you've considered

Testing

Load a save in which skill rust was set to Int. Observe that it migrated to Vanilla.
Start the game with a clean user profile. Open config/options.json and verify that SKILL_RUST is set to "vanilla".

Additional context

Before:

image

After:

image

The default value change will not affect players in the middle of a game or players starting a new game with the same "config" directory. It will only affect players installing the game for the first time or players who choose to delete their old config or change the installation directory in order to get a fresh start with all options reset to default.

@actual-nh actual-nh added <Bugfix> This is a fix for a bug (or closes open issue) [C++] Changes (can be) made in C++. Previously named `Code` Info / User Interface Game - player communication, menus, etc. labels Sep 3, 2021
@eltank eltank changed the title Remove Int/IntCap skill rust options Remove Int/IntCap skill rust options and make Vanilla default Sep 3, 2021
@ZhilkinSerg ZhilkinSerg merged commit 9684812 into CleverRaven:master Sep 3, 2021
@eltank eltank deleted the rust_option branch September 3, 2021 05:50
@linonetwo
Copy link
Contributor

Hi, we have a question:

skill rust provides benefits now

what benefit can we get from it? I know it encourages us to play in party-based gameplay, but that feature still requires debug privilege, and is not available in normal gameplay. And that is not a real benefit I think.

@I-am-Erk
Copy link
Member

Rust doesn't have anything to do with party based gameplay really. It provides a benefit because you'll learn skills faster overall if you give them a little time to rust and then resume practicing them.

@linonetwo
Copy link
Contributor

Thanks! I didn't know that, I'm going to share this with my friends.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
<Bugfix> This is a fix for a bug (or closes open issue) [C++] Changes (can be) made in C++. Previously named `Code` Info / User Interface Game - player communication, menus, etc.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants