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

rework read time formula #65280

Conversation

anoobindisguise
Copy link
Contributor

@anoobindisguise anoobindisguise commented Apr 23, 2023

Summary

Balance "Adjust reading speed formula"

Purpose of change

Reading times had an issue where high intelligence values caused reading speed to linearly approach 1 second. This lead to weird gameplay of waiting to read books until you hit 28 intelligence from various buffs (like cephalopod/slime primers, artifacts, etc) and then nearly instantly devouring your entire library.

Describe the solution

Reading time was changed from [ 1 minute - 3 seconds * (int-8) ] to [ 3_minutes / ( 2 + int/8 ) ]. So reading time is normal (1 minute) at int 8, 20% slower at int 4 (72 seconds) and about 15% faster at int 12 ( 51 seconds). At 28 int it is now almost twice as fast, at 33 seconds, instead of being 60x faster (1 second). Overall int has a much less pronounced effect on your ability to speed read books which should be fine since int affects book learning in multiple other ways too. The main problem was the absurdly fast turbo reading that came from critical mass int levels.

Describe alternatives you've considered

This is the result after a bit of tweaking and I'm happy with it.

Testing

Additional context

@anoobindisguise anoobindisguise marked this pull request as draft April 23, 2023 20:18
@github-actions github-actions bot added [C++] Changes (can be) made in C++. Previously named `Code` Game: Balance Balancing of (existing) in-game features. json-styled JSON lint passed, label assigned by github actions astyled astyled PR, label is assigned by github actions labels Apr 23, 2023
@github-actions github-actions bot removed the astyled astyled PR, label is assigned by github actions label Apr 25, 2023
@github-actions github-actions bot added the astyled astyled PR, label is assigned by github actions label Apr 25, 2023
@anoobindisguise anoobindisguise marked this pull request as ready for review May 10, 2023 19:19
@github-actions github-actions bot added Code: Tests Measurement, self-control, statistics, balancing. BasicBuildPassed This PR builds correctly, label assigned by github actions labels May 11, 2023
@github-actions github-actions bot removed the BasicBuildPassed This PR builds correctly, label assigned by github actions label May 12, 2023
@github-actions github-actions bot added the BasicBuildPassed This PR builds correctly, label assigned by github actions label May 17, 2023
@bombasticSlacks bombasticSlacks merged commit 18fe8ad into CleverRaven:master May 18, 2023
@anoobindisguise anoobindisguise deleted the anoobindisguise-reading-fixes branch June 13, 2023 19:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
astyled astyled PR, label is assigned by github actions BasicBuildPassed This PR builds correctly, label assigned by github actions [C++] Changes (can be) made in C++. Previously named `Code` Code: Tests Measurement, self-control, statistics, balancing. Game: Balance Balancing of (existing) in-game features. json-styled JSON lint passed, label assigned by github actions
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants