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

[MOD] INT Based Learning #37075

Merged
merged 2 commits into from Jan 20, 2020
Merged

[MOD] INT Based Learning #37075

merged 2 commits into from Jan 20, 2020

Conversation

ghost
Copy link

@ghost ghost commented Jan 15, 2020

Summary

SUMMARY: Mods "add new mod rebalancing learning mechanics around int stat"

Purpose of change

INT stat has very low value in Cata, its impact is marginal and virtually only one noticeable is slight reading speed change. This mod aims to ameliorate that - it is a mod mostly due to long legacy of established mechanics.

Describe the solution

  • Make base INT provide effective focus modifier for learning (based on 8 int being the average, and 5 effective focus point lost/gained for point), this is done in adjust_for_focus() function
  • Blacklist Fast Learner, Slow Learner traits
  • Scale reading skill gains with adjust_for_focus - this provides relatively little gain for highly intelligent characters due to leaving min/max caps in place (since there is only so much one can learn from one chapter), but lower INT characters will find drops in focus quickly crippling their learning rate.
  • Practice skill gains were already scaling using that function - but added effective focus modifier makes INT manifest significant changes in practice rates.

Describe alternatives you've considered

not releasing

Testing

create word with mod enabled and a character
use debug menu to play around with int and focus
see how stuff works by reading or practicing

Planned for sometime in future

Improved disassembly learn chance for high int characters

@Brian-Otten
Copy link
Contributor

I would probably enjoy playing with this and would love to see it as an option. I think mainlining this, maybe with smaller effect, would also not be a bad idea. High INT characters having a bit of an easier time maintaining focus makes sense to me, both gameplay and realism wise.

@curstwist curstwist added [JSON] Changes (can be) made in JSON Mods Issues related to mods or modding labels Jan 16, 2020
@ZhilkinSerg ZhilkinSerg added the [C++] Changes (can be) made in C++. Previously named `Code` label Jan 20, 2020
@ZhilkinSerg ZhilkinSerg merged commit 90bd793 into CleverRaven:master Jan 20, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
[C++] Changes (can be) made in C++. Previously named `Code` [JSON] Changes (can be) made in JSON Mods Issues related to mods or modding
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants